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

Add a tool to assist in reconnecting vms #18615

Merged
merged 2 commits into from
Apr 2, 2019

Conversation

agrare
Copy link
Member

@agrare agrare commented Apr 2, 2019

There are some cases where a refresh archives a number of VMs and
creates new ones based on our reconnect rules, but it is desirable to
continue to use the previous ones. This tool assists in that process by
finding pairs of Vms which have the same unique key and the older of the
two is archived and the newer is active, and re-activates the older vm.

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

@agrare agrare force-pushed the add_tool_to_reconnect_vms branch from c8084b3 to b06b342 Compare April 2, 2019 17:04
@miq-bot
Copy link
Member

miq-bot commented Apr 2, 2019

Checked commits agrare/manageiq@186d69d~...b06b342 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 1 offense detected

tools/reconnect_vms.rb

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.

LGTM 👍

Copy link
Contributor

@Ladas Ladas left a comment

Choose a reason for hiding this comment

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

👍 awesome work

@gtanzillo gtanzillo added this to the Sprint 109 Ending Apr 15, 2019 milestone Apr 2, 2019
@gtanzillo gtanzillo merged commit 68b3237 into ManageIQ:master Apr 2, 2019
@agrare agrare deleted the add_tool_to_reconnect_vms branch April 2, 2019 20:15
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