Project

General

Profile

Bug #22228

If upgrade errors, a message to re-run upgrade manually is printed

Added by Anurag Patel over 1 year ago. Updated over 1 year ago.

Status:
New
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

From Bugzilla:

Description of problem:

If you run foreman-maintain to upgrade a system and the "satellite-installer --upgrade" portion fails, you will be sent back to a shell prompt with instructions to re-run "satellite-installer --upgrade". If you do this, it may work, but the system is still in maintenance mode.

Instead, if the sat-installer command fails, it should print the usual output but also print output telling the user to re-run foreman-maintain. Otherwise, the user is left in maintenance mode and it looks like the upgrade failed when they try to access Satellite in the browser.

History

#1 Updated by Anurag Patel over 1 year ago

  • Tracker changed from Feature to Bug

#2 Updated by Amit Upadhye over 1 year ago

  • Assignee changed from Anurag Patel to Amit Upadhye

#3 Updated by Amit Upadhye over 1 year ago

Anurag Patel wrote:

From Bugzilla:

Description of problem:

If you run foreman-maintain to upgrade a system and the "satellite-installer --upgrade" portion fails, you will be sent back to a shell prompt with instructions to re-run "satellite-installer --upgrade". If you do this, it may work, but the system is still in maintenance mode.

Instead, if the sat-installer command fails, it should print the usual output but also print output telling the user to re-run foreman-maintain. Otherwise, the user is left in maintenance mode and it looks like the upgrade failed when they try to access Satellite in the browser.

Here foreman-maintain is handing over control to 'satellite-installer', I discussed with Kavita and as per that I think handling this with exception in foreman-maintain should be good approach. We are thinking to catch all exceptions and print that on terminal with message of re-running foreman-maintain. If there are any more thoughts on this one then those are welcome.

#4 Updated by Amit Upadhye over 1 year ago

Amit Upadhye wrote:

Anurag Patel wrote:

From Bugzilla:

Description of problem:

If you run foreman-maintain to upgrade a system and the "satellite-installer --upgrade" portion fails, you will be sent back to a shell prompt with instructions to re-run "satellite-installer --upgrade". If you do this, it may work, but the system is still in maintenance mode.

Instead, if the sat-installer command fails, it should print the usual output but also print output telling the user to re-run foreman-maintain. Otherwise, the user is left in maintenance mode and it looks like the upgrade failed when they try to access Satellite in the browser.

Here foreman-maintain is handing over control to 'satellite-installer', I discussed with Kavita and as per that I think handling this with exception in foreman-maintain should be good approach. We are thinking to catch all exceptions and print that on terminal with message of re-running foreman-maintain. If there are any more thoughts on this one then those are welcome.

Adding more details as per discussion with Kavita and as per my understanding of this. If we see implementation then we have run method in /definitions/procedures/installer/upgrade.rb which calls execute! method for actually do heavy lifting of satellite-installer, katello-installer and foreman-installer.

To avoid messages of this specific installers as per versions of Satellite and as per upstream we are thinking to wrap execute! method call in exception and then remove installer specific message and append foreman-maintain message there.

Also available in: Atom PDF