Skip to content
This repository was archived by the owner on Jul 17, 2024. It is now read-only.

Allow creating stopped instance.#265

Merged
luqmana merged 5 commits into
mainfrom
luqmana/create-stopped
Sep 9, 2022
Merged

Allow creating stopped instance.#265
luqmana merged 5 commits into
mainfrom
luqmana/create-stopped

Conversation

@luqmana

@luqmana luqmana commented Sep 9, 2022

Copy link
Copy Markdown
Contributor

cli side updates for oxidecomputer/omicron#1653

This adds a -s/--start flag to the create operation. If it's omitted, it defaults to true which preserves the current behaviour. You can also explicitly specify true or false. If the flag is passed with no value, it treats that as true.

@luqmana luqmana force-pushed the luqmana/create-stopped branch from c36b573 to 38fff44 Compare September 9, 2022 06:40

@karencfv karencfv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks like you just need to generate the docs!

cargo run -- generate json -D docs or run test_generate_json with EXPECTORATE=overwrite set please :)

@luqmana luqmana force-pushed the luqmana/create-stopped branch from 38fff44 to 1645019 Compare September 9, 2022 06:54
@luqmana

luqmana commented Sep 9, 2022

Copy link
Copy Markdown
Contributor Author

ok, looks like it's just the auth/ssh tests failing now (which is a known issue in CI). All pass locally against latest omicron.

@karencfv karencfv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🙇‍♀️

@luqmana luqmana merged commit ce0735a into main Sep 9, 2022
@luqmana luqmana deleted the luqmana/create-stopped branch September 9, 2022 07:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants