search

RedHatSatellite / satellite-clone

Utility to assist cloning an existing Satellite 6 server to a new host
GNU General Public License v3.0
37 stars 31 forks source link

Disables hammer-cli-csv module and ignore warnings in hammer commands #367

Closed johnpmitsch closed 4 years ago

johnpmitsch commented 4 years ago

This disables the hammer-cli-csv module by removing it's configuration file from the hammer config. This is an issue on some 6.6 installations, where the file is still present.

A fix could still be needed in the installer, but this commit will fix things for any current backups by ensuring the file is removed.

This also updates the disassociate_capsules script to exclude warnings for all hammer commands where we are parsing the output. It also DRYs up our parsing into one function to make updating this area easier.

You can test just the "disassociate_capsules" script on a Satellite that includes capsules associated with a lifecycle environment. The script will disassociate them and give instructions for reversal.

johnpmitsch commented 4 years ago

This did pass 3/4 automated installs

PLAY RECAP *********************************************************************
192.168.121.150            : ok=20   changed=15   unreachable=0    failed=1
192.168.121.183            : ok=21   changed=16   unreachable=0    failed=0
192.168.121.203            : ok=21   changed=16   unreachable=0    failed=0
192.168.121.6              : ok=21   changed=16   unreachable=0    failed=0

The one failing install is a known issue on cloning with no pulp data backup, which is being worked on here https://github.com/theforeman/foreman_maintain/pull/296 so is unrelated to this PR