Skip to content

Use os-installer coming with metal-hammer instead from metal-images#179

Open
majst01 wants to merge 4 commits intomasterfrom
with-os-installer
Open

Use os-installer coming with metal-hammer instead from metal-images#179
majst01 wants to merge 4 commits intomasterfrom
with-os-installer

Conversation

@majst01
Copy link
Copy Markdown
Contributor

@majst01 majst01 commented Mar 3, 2026

Description

With this, the install-go which is inside every metal-image is not used, instead metal-hammer comes with os-installer and executes that.
This enables us to make breaking changes to what the installer expects without breaking existing images.

Part of metal-stack/releases#236

Tested with:

  • debian-12.0.20260210
  • debian-11.0.20230505 (older versions panic because of frr < v8.5)
  • firewall-ubuntu-3.0.20260210
  • firewall-ubuntu-3.0.20230505 (first version with frr >= v8.5)

@majst01 majst01 requested a review from a team as a code owner March 3, 2026 09:53
@metal-robot metal-robot bot added this to Development Mar 3, 2026
@majst01 majst01 self-assigned this Mar 3, 2026
@majst01 majst01 force-pushed the with-os-installer branch 4 times, most recently from d794f97 to e152c11 Compare March 3, 2026 10:49
@Gerrit91
Copy link
Copy Markdown
Contributor

Gerrit91 commented Mar 4, 2026

Requires docs update as this changes the image contract.

@majst01 majst01 force-pushed the with-os-installer branch from e152c11 to 0bb4b00 Compare March 4, 2026 11:57
@simcod simcod moved this to In Progress in Development Mar 9, 2026
@iljarotar iljarotar added the requires docs update For merging this pull request, an update in metal-stack.io/docs is required. label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires docs update For merging this pull request, an update in metal-stack.io/docs is required.

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants