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.

Sign up for GitHub

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

Jump to bottom

Fixes #30652 - Fix upgrade procedure #388

Merged
merged 2 commits into from
Aug 31, 2020
Merged

Fixes #30652 - Fix upgrade procedure #388

merged 2 commits into from
Aug 31, 2020

Conversation

patilsuraj767
Copy link
Contributor

No description provided.

@theforeman-bot
Copy link
Member

Issues: #30652

@kgaikwad kgaikwad added the 0.6.x changes targeted for 6.8 label Aug 18, 2020
@jameerpathan111
Copy link
Contributor

@patilsuraj767 Tried this PR, issue seems to be fixed.

@vsedmik
Copy link

vsedmik commented Aug 31, 2020

Tested three upgrade scenarios, with and without --assumeyes option:
a) Satellite 6.6 -> 6.7
b) Satellite 6.7 -> 6.8
c) Capsule 6.7 -> 6.8

All scenarios succeeded, all services were running ok.

a)

Running Checks after upgrading to Satellite 6.7
================================================================================
Clean old Kernel and initramfs files from tftp-boot:                  [OK]
--------------------------------------------------------------------------------
Check number of fact names in database:                               [OK]
--------------------------------------------------------------------------------
Check whether all services are running:                               [OK]
--------------------------------------------------------------------------------
Check whether all services are running using the ping call:           [OK]
--------------------------------------------------------------------------------
Check for paused tasks:                                               [OK]
--------------------------------------------------------------------------------
Check to verify no empty CA cert requests exist:                      [OK]
--------------------------------------------------------------------------------
Check whether system is self-registered or not:                       [OK]
--------------------------------------------------------------------------------
Check if only installed assets are present on the system: 
/ Checking for presence of non-original assets...                     [OK]      
--------------------------------------------------------------------------------


--------------------------------------------------------------------------------
Upgrade finished.

b)

Running Checks after upgrading to Satellite 6.8
================================================================================
Clean old Kernel and initramfs files from tftp-boot:                  [OK]
--------------------------------------------------------------------------------
Check number of fact names in database:                               [OK]
--------------------------------------------------------------------------------
Check whether all services are running:                               [OK]
--------------------------------------------------------------------------------
Check whether all services are running using the ping call:           [OK]
--------------------------------------------------------------------------------
Check for paused tasks:                                               [OK]
--------------------------------------------------------------------------------
Check to verify no empty CA cert requests exist:                      [OK]
--------------------------------------------------------------------------------
Check whether system is self-registered or not:                       [OK]
--------------------------------------------------------------------------------
Check if only installed assets are present on the system: 
/ Checking for presence of non-original assets...                     [OK]      
--------------------------------------------------------------------------------


--------------------------------------------------------------------------------
Upgrade finished.

c)

Running Checks after upgrading to Capsule 6.8
================================================================================
Check to verify no empty CA cert requests exist:                      [OK]
--------------------------------------------------------------------------------
Check whether all services are running:                               [OK]
--------------------------------------------------------------------------------
Check whether all services are running using the ping call:           [OK]
--------------------------------------------------------------------------------
Check whether system is self-registered or not:                       [OK]
--------------------------------------------------------------------------------


--------------------------------------------------------------------------------
Upgrade finished.

@kgaikwad
Copy link
Member

Tested three upgrade scenarios, with and without --assumeyes option:
a) Satellite 6.6 -> 6.7
b) Satellite 6.7 -> 6.8
c) Capsule 6.7 -> 6.8

All scenarios succeeded, all services were running ok.

Awesome!! 👍
Thank you so much @vsedmik for your feedback and covering all the possible cases.
Now I can hit merge button for this pull-request :)

kgaikwad
kgaikwad approved these changes Aug 31, 2020
View reviewed changes
Copy link
Member

@kgaikwad kgaikwad left a comment

Choose a reason for hiding this comment

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

Merging this pull-request.

@kgaikwad kgaikwad merged commit 43f0c9e into theforeman:master Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kgaikwad kgaikwad kgaikwad approved these changes

Assignees
No one assigned
Labels
0.6.x changes targeted for 6.8 Needs testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@patilsuraj767 @theforeman-bot @jameerpathan111 @vsedmik @kgaikwad