Skip to content

Simplify in-place refresh component script - #591

Open
Nguyen Vuong (normanvuong) wants to merge 1 commit into
masterfrom
nguyenvuong/inplace_refresh_retrofit
Open

Simplify in-place refresh component script#591
Nguyen Vuong (normanvuong) wants to merge 1 commit into
masterfrom
nguyenvuong/inplace_refresh_retrofit

Conversation

@normanvuong

@normanvuong Nguyen Vuong (normanvuong) commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Limit in-place manifest refresh to components that can change during apt upgrade or dnf update.
  • Preserve install-time versions for pinned and source-installed components.
  • Record the active kernel after reboot.
  • Keep APT’s native DKMS hook responsible for rebuilding modules in dependency order.
  • Simplify refresh logic by removing broad best-effort detection and strict tier handling.

Validation

https://dev.azure.com/hpc-platform-team/hpc-image-val/_build/results?buildId=36810&view=results

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@ColtonPaul

Copy link
Copy Markdown
Contributor

The simplification is good, but it does seem like we're losing the confidence that we have a "detector" for components that can upgrade during an in-place upgrade. Is there a compromise option where we get the best of both worlds:

  1. We don't have to write version discovery code for components that can't upgrade
  2. If the list of upgradeable components in any component_versions.json changes, we're not at risk of forgetting to update refresh_component_versions.sh accordingly?

I'm curious about something like an in-place-upgradeable attribute in versions.json that could help solve this and possibly also help lock down intentions about which package versions can move (for example, I see that pmix still has a detector, so I checked the code and saw that only Ubuntu holds it, and RHEL doesn't. Is that intentional?).

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.

2 participants