Skip to content

[installinator] write out mupdate-override.json #8042

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

sunshowers
Copy link
Contributor

@sunshowers sunshowers commented Apr 24, 2025

Write out a file containing a UUID which indicates that installinator wrote out an updated dataset. This is a signal for Nexus to assume that installinator wrote out a new install dataset behind its back.

I'm not entirely sure if writing out the JSON file at the beginning vs at some later point matters a huge amount here -- if installinator fails for some reason, we assume the system is broken anyway.

We also record the host phase 2 and control plane artifact hashes, and there's space to record other hashes if we'd like to do so in the future.

Tested on dublin:

Screenshot 2025-04-24 150911

BRM42220026 # ls -al /pool/int/*/install/mupdate-override.json
-rw-------   1 root     root          55 Dec 28  1986 /pool/int/2196ba5d-e211-4495-9c0b-e0948ba0afbd/install/mupdate-override.json
-rw-------   1 root     root          55 Dec 28  1986 /pool/int/f3673b2a-0cdc-4315-ac53-7889918cee4c/install/mupdate-override.json

See RFD 556 for more information.

Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1

[skip ci]
Created using spr 1.3.6-beta.1
@sunshowers sunshowers changed the base branch from main to sunshowers/spr/main.installinator-write-out-mupdate-overridejson April 24, 2025 06:39
@sunshowers sunshowers changed the base branch from sunshowers/spr/main.installinator-write-out-mupdate-overridejson to main April 24, 2025 06:39
Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant