-
Notifications
You must be signed in to change notification settings - Fork 91
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
Service retirement should use make retire as a request #1453
Service retirement should use make retire as a request #1453
Conversation
@miq-bot add_label bug, blocker |
411631c
to
0de7e2f
Compare
Checked commit d-m-u@0de7e2f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
I can't get this to fail locally, so that's cool. 🎉 |
@d-m-u don't sweat it, travis loves to play games with our hearts, (well its not travis.. but.. its an easy blame so 🤷♂️ ) So can this be unwipped, ready for review? |
@miq-bot remove_label wip |
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.
LG2M, Thanks for bringing the update to the SUI @d-m-u !!
🌮 🐙
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.
Looks good to me.
Since the API call for service retirement changed in ManageIQ/manageiq-api#380 for the new retire as a request, the SUI ought to use the new call.
For https://bugzilla.redhat.com/show_bug.cgi?id=1599299