Created attachment 1330986 [details] log in foreman-maintain.log file Description of problem: While running foreman-maintain upgrade run, if upgrade failed in repository setup, firewall set in maintenance mode should turn off. Version-Release number of selected component (if applicable): Foreman-maintain snap 2.0 How reproducible: always Steps to Reproduce: 1. Run #foreman-maintain upgrade run --target-version 6.3 without setting up repositories Actual results: Upgrade run will fail due to Error: 'rhel-7-server-satellite-6.3-rpms' does not match a valid repository ID and can not access satellite as maintenance mode on. Expected results: Maintenance mode should turn off if upgrade run failed. Additional info:
@Nikhil, If I understand the issue correctly, foreman-maintain set its own firewall rules before upgrade and once upgrade completed successfully, it auto removes those firewall rules right ? But it doesn't happen when upgrade fails at some step. Do you see any issue if it doesn't clean the rules ? I mean in next run or re-run do you see any issue w/ firewall rules ?
Most likely in this scenario the satellite admin will fix the repo issue and try upgrading again. It is better to leave the satellite in maintenance mode - may be there can be a message saying that satellite continues to be in maintenance mode?
Based on https://bugzilla.redhat.com/show_bug.cgi?id=1495575#c2 Adding need info from Nikhil.
(In reply to Swapnil Abnave from comment #4) > Based on https://bugzilla.redhat.com/show_bug.cgi?id=1495575#c2 Adding need > info from Nikhil. This is not causing any issue in next-run or rerun and yes I agree with comment #2, There should be message saying Satellite is in maintenance mode and use 'foreman-maintain advanced procedure run maintenance-mode-disable' to turn off maintenance mode.
Upstream bug assigned to apatel
Thank you for your interest in Satellite 6. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in the product in the foreseeable future. This is due to other priorities for the product, and not a reflection on the request itself. We are therefore closing this out as WONTFIX. If you have any concerns about this, please do not reopen. Instead, feel free to contact Rich Jerrido or Bryan Kearney. Thank you.