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-1468] Fixed test agents lxd #14333
Conversation
Charmhub is not the same as charmstore. So we cannot drop a charmstore tests and claim that there is still coverage because there's an equivalent charmhub tests. PR #14332 switched fro mysql to postgesql so that's probably an option here too. |
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.
Can you please update the PR description now that the implementation has changed
/merge |
1 similar comment
/merge |
#14350 Merge 2.9 #14329 [JUJU-1466] Filter by model uuid when querying azure instances #14318 [JUJU-1196] refactor nw-constraints-aws test #14333 [JUJU-1468] Fixed test agents lxd #14339 Fix cli/local_charms test due to deleted GitHub repo #14342 Update deploy_revision tests to actually check revision #14337 [JUJU-1475] Added lxd constraints tests #14335 More unit charm url work #14343 [JUJU-1480] Add support for retry-provisioning --all #14345 [JUJU-1480] Fix display of upgrading-from for subordinate units ``` # Conflicts: # api/client/client/client.go # cmd/juju/model/retryprovisioning.go # migration/precheck.go # migration/precheck_test.go # rpc/params/internal.go # tests/suites/agents/key_workers_run.sh ``` ## QA steps See PRs [JUJU-1466]: https://warthogs.atlassian.net/browse/JUJU-1466?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ [JUJU-1196]: https://warthogs.atlassian.net/browse/JUJU-1196?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ [JUJU-1468]: https://warthogs.atlassian.net/browse/JUJU-1468?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ [JUJU-1475]: https://warthogs.atlassian.net/browse/JUJU-1475?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ [JUJU-1480]: https://warthogs.atlassian.net/browse/JUJU-1480?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ [JUJU-1480]: https://warthogs.atlassian.net/browse/JUJU-1480?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
Dropped the charmstore test, which used the old MySQL charm. There is no need to force MySQL charm to switch to another channel because we have yet another test for charmhub in the same suite.
Checklist
QA steps
cd tests ./main.sh -v -p lxd -c lxd agents
Documentation changes
None
Bug reference
None