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

Fix test-agents-lxd #14338

Merged
merged 1 commit into from
Jul 20, 2022
Merged

Fix test-agents-lxd #14338

merged 1 commit into from
Jul 20, 2022

Conversation

barrettj12
Copy link
Contributor

The gating test test-agents-lxd was failing because it was trying to deploy a very old revision of mysql, which runs on trusty. Juju 3.0 didn't seem to like the trusty machine.

Newer versions of the mysql charm currently aren't working on focal (e.g. these issues), so I've decided to use postgresql instead.

QA steps

cd tests
./main.sh -v agents

@benhoyt
Copy link
Member

benhoyt commented Jul 20, 2022

Per Mattermost chat, this doesn't seem to work on 3.0 - either the 15s polling (test override) is not working, or the revision or revision updater is not doing what we think it should. Seems to pass on 2.9 though.

Copy link
Member

@benhoyt benhoyt left a comment

Choose a reason for hiding this comment

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

Per Jordan on Mattermost:

anyway my point is that the status polling seems to work on Jenkins even if it doesn't work on my machine. My PR should fix the other issue (can't tear down model) so the test should pass on Jenkins

@barrettj12
Copy link
Contributor Author

/merge

@jujubot jujubot merged commit 21e9724 into juju:develop Jul 20, 2022
@barrettj12 barrettj12 deleted the test-agents-lxd branch July 20, 2022 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants