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

Run the setup playbook if we see that an upgrade has happened #15482

Merged

Conversation

carbonin
Copy link
Member

The playbook needs to run on upgrades and ansible tower will refuse to start if it isn't.

https://bugzilla.redhat.com/show_bug.cgi?id=1465590

The playbook needs to run on upgrades and ansible tower will refuse
to start if it isn't.

https://bugzilla.redhat.com/show_bug.cgi?id=1465590
@carbonin
Copy link
Member Author

@simaishi It's likely that this will conflict with #15423 so I'll look at a direct-to-fine PR.

@miq-bot
Copy link
Member

miq-bot commented Jun 29, 2017

Checked commit carbonin@72320a3 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@gtanzillo gtanzillo added this to the Sprint 64 Ending Jul 3, 2017 milestone Jun 30, 2017
@gtanzillo gtanzillo merged commit d418207 into ManageIQ:master Jun 30, 2017
@simaishi
Copy link
Contributor

@carbonin marking as fine/conflict, please remove the label when you have a PR for Fine branch. Thanks!

@carbonin carbonin deleted the run_the_setup_playbook_for_tower_upgrades branch June 30, 2017 15:26
@simaishi
Copy link
Contributor

Backported to Fine via #15487

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants