Skip to content

Machine State Command#132

Open
majst01 wants to merge 1 commit into
mainfrom
machine-condition-command
Open

Machine State Command#132
majst01 wants to merge 1 commit into
mainfrom
machine-condition-command

Conversation

@majst01
Copy link
Copy Markdown
Contributor

@majst01 majst01 commented Jun 4, 2026

Description

This was missing and enables an admin to taint/lock/untaint/unlock a machine

Used AI-Tools ✨

  • none used for generation

@majst01 majst01 requested a review from a team as a code owner June 4, 2026 11:59
@metal-robot metal-robot Bot added this to Development Jun 4, 2026
@majst01 majst01 self-assigned this Jun 4, 2026
@majst01 majst01 force-pushed the machine-condition-command branch from 3c4039d to f312b33 Compare June 5, 2026 06:42
Comment thread proto/metalstack/admin/v2/machine.proto Outdated
option (metalstack.api.v2.auditing) = AUDITING_INCLUDED;
}
// ConditionCommand sends a condition command to a machine.
rpc ConditionCommand(MachineServiceConditionCommandRequest) returns (MachineServiceConditionCommandResponse) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not really sure regarding the naming, especially the Command part. Maybe a simple SetMachineState?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better now ?

@majst01 majst01 changed the title Machine Condition Command Machine State Command Jun 5, 2026
@majst01 majst01 force-pushed the machine-condition-command branch from f312b33 to 4143b07 Compare June 5, 2026 06:50
@majst01 majst01 force-pushed the machine-condition-command branch from 4143b07 to 5f4ab5a Compare June 6, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants