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 VM Retirement Error Message #19710

Merged
merged 1 commit into from Jan 22, 2020

Conversation

tinaafitz
Copy link
Member

Issue error log message only if there are invalid src_ids.

The following error log message is unconditional. It should only be issued when src_ids contains at least one invalid vm id.
[----] E, [2020-01-13T12:50:10.880878 #10833:46b5e3c] ERROR -- : MIQ(Vm.make_retire_request) Retirement of [Vm] IDs: [] skipped - target(s) does not exist

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

@miq-bot miq-bot added the wip label Jan 13, 2020
@tinaafitz tinaafitz force-pushed the fix_retirement_message branch 2 times, most recently from 6b8b51f to 0e9fa7d Compare January 14, 2020 12:16
@tinaafitz tinaafitz force-pushed the fix_retirement_message branch 3 times, most recently from 1710d77 to e342d53 Compare January 21, 2020 20:36
@miq-bot
Copy link
Member

miq-bot commented Jan 21, 2020

Checked commit tinaafitz@7435dce with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

@tinaafitz tinaafitz changed the title [WIP] Fix VM Retirement Error Message Fix VM Retirement Error Message Jan 22, 2020
@miq-bot miq-bot removed the wip label Jan 22, 2020
@chessbyte chessbyte merged commit 685e678 into ManageIQ:master Jan 22, 2020
@simaishi
Copy link
Contributor

@tinaafitz can this be ivanchuk/yes and hammer/yes?

@miq-bot
Copy link
Member

miq-bot commented Jan 24, 2020

@tinaafitz Cannot apply the following label because they are not recognized: changelog/yes hammer/yes ivanchuk/yes

@tinaafitz
Copy link
Member Author

@miq-bot add_label bug, hammer/yes, ivanchuk/yes, changelog/yes

@tinaafitz
Copy link
Member Author

Hi @simaishi Yes, I added the labels, thanks.

@simaishi
Copy link
Contributor

@tinaafitz The backport conflicts on spec file as initiated_by exists only in master branch. Can you create a separate PR with appropriate test for ivanchuk/hammer?

@Fryguy Fryguy added this to the Sprint 129 Ending Feb 3, 2020 milestone Jan 31, 2020
@simaishi
Copy link
Contributor

simaishi commented Feb 6, 2020

Backported to hammer via #19821

@simaishi
Copy link
Contributor

Backported to ivanchuk via #19820

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.

None yet

5 participants