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

Enable functional test with appropriate series #103

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

chanchiwai-ray
Copy link
Contributor

@chanchiwai-ray chanchiwai-ray commented Nov 15, 2023

See PR title.

@chanchiwai-ray chanchiwai-ray changed the title Enable functional test for Jammy and Focal Enable functional test with appropriate series Nov 15, 2023
Copy link
Contributor

@rgildein rgildein left a comment

Choose a reason for hiding this comment

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

I'm not sure if focal only on 2.9 will be enough, but I think for now it's ok.

@rgildein rgildein mentioned this pull request Nov 15, 2023
Copy link

@agileshaw agileshaw left a comment

Choose a reason for hiding this comment

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

Also was wondering whether we want to test focal for both juju 2.9 and 3.1. Otherwise LGTM

@chanchiwai-ray
Copy link
Contributor Author

It was originally four tests, [Focal, Jammy] x [2.9/stable, 3.1/stable], but we think it's too much. In fact, this charm does not depends on juju channel, so having focal and jammy in which channel should not be important, as long as they are there.

@chanchiwai-ray chanchiwai-ray merged commit 035bba6 into canonical:master Nov 16, 2023
4 checks passed
@chanchiwai-ray chanchiwai-ray deleted the fix-workflow branch November 16, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants