Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: integration tests CI #13

Open
wants to merge 5 commits into
base: 6/edge
Choose a base branch
from
Open

feat: integration tests CI #13

wants to merge 5 commits into from

Conversation

Gu1nness
Copy link
Collaborator

@Gu1nness Gu1nness commented Feb 12, 2025

🏷 Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation Update
  • Tooling and CI changes
  • Dependencies upgrade or change

📝 Description

Introduction of a CI workflow that listens on comments on PRs and opens PR on the remote repositories to trigger the CI.
It then posts the PR URL so we can fetch the results.

This is run on comment on a PR, following this syntax:

  • /test to run on 4 charms.
  • /test/<mongod | mongos>/<vm | k8s> to run on a specific charm.
  • /test/*/<vm | k8s> to run for both charms on a specific substrate.

📑 Motivation and Context

This is the final step required for the single kernel charm.
A way of easily running the CI tests on the remote charms.

🧪 How Has This Been Tested?

This needs to be tested.

✅ Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

username: ${{ github.event.comment.author.login }}
org: "Canonical"
team: "data-ai-engineers"
github_token: ${{ secrets.GITHUB_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

```shell
* /test to run on 4 charms.
* /test/<mongod | mongos>/<vm | k8s> to run on a specific charm.
* /test/*/<vm | k8s> to run for both charms on a specific substrate.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are a rockstar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants