Skip to content

Add room to machine and switch#156

Merged
majst01 merged 2 commits into
mainfrom
add-room-to-machine-and-switch
Jul 16, 2026
Merged

Add room to machine and switch#156
majst01 merged 2 commits into
mainfrom
add-room-to-machine-and-switch

Conversation

@majst01

@majst01 majst01 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Description

Retrofit metal-stack/metal-api#651

Intentionally break proto by re-using indices in favour of nice and aligned messages.
This is probably the last time such breaking changes can be made.

Bycatch was the removal of unnecessary package in the auditing annotation in the api package

Used AI-Tools ✨

  • none used for generation

@majst01
majst01 requested a review from a team as a code owner July 14, 2026 09:23
@majst01 majst01 self-assigned this Jul 14, 2026
@majst01
majst01 requested review from iljarotar and mwennrich July 14, 2026 09:24
@majst01
majst01 force-pushed the add-room-to-machine-and-switch branch from 4336cf9 to 96186f6 Compare July 14, 2026 11:17
@Gerrit91 Gerrit91 added the requires docs update For merging this pull request, an update in metal-stack.io/docs is required. label Jul 14, 2026
@Gerrit91

Copy link
Copy Markdown
Contributor

The room concept needs to be reflected in our official documentation describing what this is specifically used for, when it is optional and what effects it has if any. Here is the ongoing PR for this: metal-stack/website#278

@majst01
majst01 force-pushed the add-room-to-machine-and-switch branch from 96186f6 to 5c397a9 Compare July 14, 2026 11:33
@majst01
majst01 merged commit 2659dc2 into main Jul 16, 2026
10 checks passed
@majst01
majst01 deleted the add-room-to-machine-and-switch branch July 16, 2026 11:16
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: Done

Development

Successfully merging this pull request may close these issues.

3 participants