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

[SLE15-SP6] Better summary for the SLE_HPC => SLES upgrade (jsc#PED-7841) #652

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

lslezak
Copy link
Member

@lslezak lslezak commented Feb 22, 2024

Problem

Solution

  • Update the product rename mapping

Technical Details

Originally it was "SLE_HPC" => ["SLES", "sle-module-hpc"] mapping. But because the old SLE_HPC required the HPC module (a hard dependency, you cannot have SLE_HPC without the HPC module) then the real upgrade is not "SLE_HPC => SLES + HPC" but rather "SLE_HPC + HPC => SLES + HPC". And in that case we can remove the HPC module from the mapping because it will be automatically upgraded to the new available version.

If the HPC module was optional for SLE_HPC we would have to add it to the mapping, but as it was mandatory we can leave it out.

That mapping is used when finding the product upgrades for the upgrade summary. The extra HPC value confuses the algorithm and it thinks the SLE_HPC is upgraded to the HPC module.

Testing

  • Tested manually

hpc_upgrade_sp6_better

@lslezak lslezak changed the title Better summary for the SLE_HPC => SLES upgrade (jsc#PED-7841) [SLE15-SP6] Better summary for the SLE_HPC => SLES upgrade (jsc#PED-7841) Feb 22, 2024
@coveralls
Copy link

Coverage Status

coverage: 37.385%. remained the same
when pulling 3246db1 on better_summary_sp6
into 65b1196 on SLE-15-SP6.

@lslezak lslezak merged commit 1637a8f into SLE-15-SP6 Feb 22, 2024
12 checks passed
@lslezak lslezak deleted the better_summary_sp6 branch February 22, 2024 13:28
@lslezak
Copy link
Member Author

lslezak commented Feb 22, 2024

Note: submitted manually in https://build.suse.de/request/show/322405

@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #9 successfully finished

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

4 participants