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

Properly dissassociate capsule lifecycle environments #363

Merged
merged 1 commit into from Jul 8, 2019

Conversation

jlsherrill
Copy link
Contributor

@jlsherrill jlsherrill commented Jul 5, 2019

The previous code would grab the last listed org for the capsule
and then attempt to lookup all associated lifecycle environments
by that organization. The result is that if a capsule is associated
with 5 orgs, but only associated with the Library of one or two of them,
it may randomally decide to try to remove the lifecycle env from another org

Closes: #363

The previous code would grab the last listed org for the capsule
and then attempt to lookup all associated lifecycle environments
by that organization.  The result is that if a capsule is associated
with 5 orgs, but only associated with the Library of one or two of them,
it may randomally decide to try to remove the lifecycle env from another org
Copy link
Contributor

@johnpmitsch johnpmitsch left a comment

Choose a reason for hiding this comment

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

Worked well with the four backups in automation, including one I added with a capsule, which restored and disassociated for lifecycle env correctly.

PLAY RECAP *********************************************************************
192.168.121.154            : ok=21   changed=16   unreachable=0    failed=0
192.168.121.173            : ok=21   changed=16   unreachable=0    failed=0
192.168.121.22             : ok=21   changed=16   unreachable=0    failed=0
192.168.121.38             : ok=21   changed=16   unreachable=0    failed=0
Monday 08 July 2019  14:25:32 -0400 (2:58:26.977)       3:11:56.122 ***********

Thanks @jlsherrill!

@johnpmitsch johnpmitsch merged commit fca4b1f into RedHatSatellite:master Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants