-
Notifications
You must be signed in to change notification settings - Fork 103
ci: drop Juju 3.3 #1269
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
ci: drop Juju 3.3 #1269
Conversation
|
Discussion link: https://matrix.to/#/!wJiiHsLipVywuWOyNi:ubuntu.com/$q1dUM0LCcnffzykmp1uEXGlgGQ-QWOo_fQGfJCPwlIw?via=ubuntu.com&via=matrix.org (no comments at this time) |
benhoyt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we remove 3.1/stable while we're here?
|
Well if we went all in, we'd remove Ref: #1270 |
Yeah, I reckon we should just do that. |
|
Ahh, actually we can't just do that. The naive approach didn't work, I've tried in dimaqq#3 |
james-garner-canonical
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm on board with just dropping 3.3 from CI as a start, since it's EOL and many integration tests are failing, which makes development more painful.
I think dropping the facades can wait till a separate PR given that python-libjuju currently supports some operations only through older facades.
Dropping other bases can wait too imo. 3.1 in particular seems to be more stable for some integration tests (e.g test_relate #1277), so it's probably useful to keep it in CI for now.
Juju 3.3 has reached end of life.
https://documentation.ubuntu.com/juju/latest/reference/juju/juju-roadmap-and-releases/index.html?dfghjkl=#juju-3-3
Meanwhile, 8 integration tests are failing against Juju 3.3 specifically #1267
This PR removed Juju 3.3 from CI to close #1267