-
Notifications
You must be signed in to change notification settings - Fork 126
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
Drop Amulet support #618
Drop Amulet support #618
Conversation
Amulet has been replaced by Zaza. https://launchpad.net/bugs/1828424
The only problem with dropping it from charm-helpers is that maybe, somewhere, someone else is using it. We should probably put a deprecation notice in this cycle and then remove it the next. Thoughts? |
Actually there is urgency to remove this code as it's making use of |
Could we just drop the OpenStack related paths? |
|
Is ‘get_ubuntu_releases’ really only used for amulet? That’s quite surprising to me |
Yes. Please grep the repo recursively for |
No I totally agree; let's get rid it of it. |
+1, I was about to merge this but I did not find that the PR removes the offending It is worth noting that while the |
Change-Id: Ib53fe4766c69fd25c3046121f7e338bf9d78b505
Change-Id: I6a5298e7d45c0851a7330e6b3d898c54bee4591e
Since 2021-07-20 something has changed in the tox/pip stack and this has become necessary. See: openstack-charmers/zaza-openstack-tests#608 openstack-charmers/release-tools#151 Co-authored-by: Alex Kavanagh <alex.kavanagh@canonical.com>
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.
LGTM, thanks!
Change-Id: Ie2692c9bdea1037fcdf603e43543fa07402cbf7c
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.
LGTM, thanks!
Amulet has been replaced by Zaza.
https://launchpad.net/bugs/1828424