-
Notifications
You must be signed in to change notification settings - Fork 497
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-1453] Test charm cleanup #14375
Conversation
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.
A couple of cleanups
@@ -1,15 +1,15 @@ | |||
series: bionic | |||
applications: | |||
lxd-profile: | |||
charm: cs:~juju-qa/bionic/lxd-profile-without-devices-2 | |||
charm: juju-qa-lxd-profile-without-devices |
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.
we need to ensure we retain in a test bundle somewhere (if not already thought about):
- revision pinning
- deploy from cs
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'll add it into the todo. I'll construct a test specifically for this purpose.
605ca50
to
fa65bb1
Compare
/merge |
1 similar comment
/merge |
Cleaning up local copies of the Ubuntu charm in favour of Charmhub.
Also made deployments of test charm from charmhub instead of locally.
- Updated for packing and uploading with charmcraft.
- Moved usages of lxd-profile-without-devices over to charmhub.
- Consolidates the lxd profile subordinate charm. - Updates charm for use with charmhub and charmcraft.
/merge |
Continued work that is stable for cleaning up charms in our testing infra.
Checklist
QA steps
Documentation changes
N/A
Bug reference
N/A