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

[JUJU-1663] Drop Python 3.5 support from python-libjuju #722

Merged
merged 2 commits into from Sep 1, 2022

Conversation

cderici
Copy link
Member

@cderici cderici commented Aug 24, 2022

Description

Python-libjuju no longer supports Python 3.5 (:tada:), following the Juju dropping support for xenial.

QA Steps

The PR on juju-qa-jenkins to remove the Python3.5 tests is up juju/juju-qa-jenkins#67, and I ran the job builder with the changes, so we shouldn't see any (unit, integration) tests spawn for Python 3.5.

Even if they spawn somehow, we can ignore them, but we need to make sure all the other tests are passing for other Python versions. (Modulo, of course we might have some known intermittent bugs that are still under investigation.)

@cderici
Copy link
Member Author

cderici commented Aug 24, 2022

Two initial observations on the CI:

  • Unit tests didn't spawn, probably because we're modifying the .github/workflow/tox.yaml (which is what is used to spawn the unit tests)
  • We seem to be getting all kinds of weirdness from Jenkins for the integration tests (Not related to the PR). There was a recent maintenance on that, might be the cause of what's going on. We should probably wait to land this until we see the tests running and passing. (except the github-integration-tests-pylibjuju-python3.5)

@cderici
Copy link
Member Author

cderici commented Aug 24, 2022

/build

cderici added a commit to cderici/python-libjuju that referenced this pull request Aug 29, 2022
The PR for that (juju#722) is
yet to have landed
@cderici
Copy link
Member Author

cderici commented Aug 31, 2022

/build

3 similar comments
@cderici
Copy link
Member Author

cderici commented Aug 31, 2022

/build

@cderici
Copy link
Member Author

cderici commented Aug 31, 2022

/build

@cderici
Copy link
Member Author

cderici commented Aug 31, 2022

/build

Copy link
Member

@jack-w-shaw jack-w-shaw left a comment

Choose a reason for hiding this comment

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

LGTM

@cderici
Copy link
Member Author

cderici commented Sep 1, 2022

/merge

@cderici
Copy link
Member Author

cderici commented Sep 1, 2022

/merge

2 similar comments
@cderici
Copy link
Member Author

cderici commented Sep 1, 2022

/merge

@cderici
Copy link
Member Author

cderici commented Sep 1, 2022

/merge

@jameinel jameinel merged commit 00cb2c0 into juju:master Sep 1, 2022
jujubot added a commit that referenced this pull request Oct 6, 2022
#739

## What's Changed

* Model name can now be accessed through model.name by @jack-w-shaw in #702
* [JUJU-1593] Fix `unit.run()` and update the old client codes by @cderici in #710
* Add py.typed marker by @sed-i in #709
* [JUJU-1664] Add force, no-wait, destroy-storage params to app.destroy by @cderici in #714
* snapcraft.io access should use https requests by @addyess in #715
* [JUJU-1680] Add issue and PR templates by @cderici in #718
* [JUJU-1681] Add --attach-storage parameter to model.deploy by @cderici in #720
* [JUJU-1706] Allow waiting for `wait_for_exact_units=0` by @cderici in #723
* [JUJU-1663] Drop Python 3.5 support from python-libjuju by @cderici in #722
* [JUJU-1671] Charmhub url from model config by @cderici in #724
* [JUJU-1733] Revisit unitrun example by @cderici in #725
* [JUJU-1800] Revise the `application.upgrade_charm()` (refresh) by @cderici in #729
* [JUJU-1893] Revisit `charmhub.info()` by @cderici in #737

[JUJU-1593]: https://warthogs.atlassian.net/browse/JUJU-1593?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1664]: https://warthogs.atlassian.net/browse/JUJU-1664?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1680]: https://warthogs.atlassian.net/browse/JUJU-1680?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1681]: https://warthogs.atlassian.net/browse/JUJU-1681?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1706]: https://warthogs.atlassian.net/browse/JUJU-1706?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1663]: https://warthogs.atlassian.net/browse/JUJU-1663?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[JUJU-1671]: https://warthogs.atlassian.net/browse/JUJU-1671?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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.

None yet

3 participants