-
Notifications
You must be signed in to change notification settings - Fork 80
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
Maas upgrade play #512
Maas upgrade play #512
Conversation
Moving these to their own play allows us more independance in timing verification runs to ensure that the agent has time to competely sync the server side checks before we try to verify. Documents the new playbook in the README and adds a 30 second delay after setup_maas.yml before the new playbook is run, when deploying with scripts/deploy.sh with env value DEPLOY_MAAS=yes
7c589e1
to
0bb304d
Compare
This needs to rebased as well |
tags: | ||
- maas-verify | ||
vars_files: | ||
- "{{ rpc_repo_path }}/playbooks/roles/rpc_maas/defaults/main.yml" |
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.
This links to the openstack-ansible repo by default so fails out.
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.
odd, I didn't have an issue testing that.
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.
rpcd/playbooks/group_vars/all.yml:rpc_repo_path: /opt/rpc-openstack/openstack-ansible
That's the default so I think that would never work, since the role exists in /opt/rpc-openstack/rpcd
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.
Very odd, but @andymcc is correct:
rpc_repo_path: /opt/rpc-openstack/openstack-ansible |
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.
right, and it should have shown up in my testing.
I must have introduced a change somewhere along the way that broke this.
-- fixed in next patchset
0bb304d
to
fa61d61
Compare
Adds a playbook to be run as a one-time operation to convert maas to use template based configuration. The playbook records a fingerprint of when it runs under /etc/openstack_deploy/migrations and will not run again if this file is present. The play will generate .tab files on the deploy host under /tmp for reporting the state of checks before and after upgrade. Also fixed verify-maas play to use dash instead of underscore in filename and made appropriate change to README to reflect that.
fa61d61
to
315316c
Compare
closing this PR and opening a new one with cleaned up commits |
Cap tox at 2.4.0 to avoid env substitution bug
Blacklist tox 2.4.0 which breaks our tests
No description provided.