Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: add machine Power On/Off Operations #38

Merged

Conversation

sachin120
Copy link
Contributor

@sachin120 sachin120 commented Dec 21, 2023

For Issue: #37 (comment)

What dose this implement

Added support for Machine Power On/Off operations

@sachin120 sachin120 force-pushed the machine-power-on-off-operation branch from 3cffc99 to 9948340 Compare December 21, 2023 08:29
@sachin120 sachin120 changed the title Add mahine Power On/Off Operations Add machine Power On/Off Operations Dec 21, 2023
@skatsaounis skatsaounis changed the title Add machine Power On/Off Operations feat: add machine Power On/Off Operations Jan 2, 2024
@sachin120 sachin120 force-pushed the machine-power-on-off-operation branch from 9948340 to 911ebba Compare January 2, 2024 12:56
Copy link
Collaborator

@skatsaounis skatsaounis left a comment

Choose a reason for hiding this comment

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

Hi @sachin120 and thank you for your contribution. It seems that both power on and off operations can accept optional arguments as per https://maas.io/docs/api. Could you please update your PR to include them?

  • In case of power on user can provide a comment and user_data in the format of base64 string.
  • In case of power off user can provide a comment and stop_mode

Copy link
Collaborator

@skatsaounis skatsaounis left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for taking care of my comment

@skatsaounis skatsaounis merged commit 299a5c1 into canonical:master Jan 2, 2024
3 checks passed
@sachin120 sachin120 deleted the machine-power-on-off-operation branch January 2, 2024 15:48
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