8000
Skip to content

lxd: Add support for starting instances on creation (from Incus)#13695

Merged
tomponline merged 7 commits intocanonical:mainfrom
boltmark:instance-start-on-creation
Jul 25, 2024
Merged

lxd: Add support for starting instances on creation (from Incus)#13695
tomponline merged 7 commits intocanonical:mainfrom
boltmark:instance-start-on-creation

Conversation

@boltmark
Copy link
Copy Markdown
Contributor
@boltmark boltmark commented Jul 1, 2024 •
edited
Loading

This PR adds support for starting instances on creation.

Includes cherry-picks from lxc/incus#410.

Closes #13322

@boltmark boltmark added the API Changes to the REST API label Jul 1, 2024
@github-actions github-actions bot added the Documentation Documentation needs updating label Jul 1, 2024
@github-actions
Copy link
Copy Markdown
github-actions bot commented Jul 1, 2024

Heads up @ru-fu - the "Documentation" label was applied to this issue.

@ru-fu
Copy link
Copy Markdown
ru-fu commented Jul 2, 2024

This will also need a doc update to the API tab for creating an instance: https://documentation.ubuntu.com/lxd/en/latest/howto/instances_create/
The other tabs say that you have the choice between init & launch / create & create and start, while the API says you have to start the instance separately.

You can find a short intro to how the docs work here - get in touch if anything is unclear. :)

@tomponline
Copy link
Copy Markdown
Member
tomponline commented Jul 2, 2024

Hi @boltmark looks like not all of the commits have been cherry-picked correctly as the CLA test is failing.

Also please can you sign your commits, see https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Thanks!

@tomponline tomponline changed the title lxd: Add support for starting instances on creation -- from Incus lxd: Add support for starting instances on creation (from Incus) Jul 2, 2024
@boltmark boltmark force-pushed the instance-start-on-creation branch 2 times, most recently from 932518a to 4cbddb4 Compare July 2, 2024 19:54
@boltmark
Copy link
Copy Markdown
Contributor Author
boltmark commented Jul 2, 2024

Thanks for the comments.
@tomponline -- CLA test is now passing.
@ru-fu -- I updated the documentation to include information + an example on starting containers upon creation.

@boltmark boltmark marked this pull request as ready for review July 2, 2024 20:51
@boltmark boltmark requested a review from tomponline as a code owner July 2, 2024 20:51
ru-fu
ru-fu reviewed Jul 3, 2024
View reviewed changes
Copy link
Copy Markdown
@ru-fu ru-fu left a comment

Choose a reason for hiding this comment

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

Thanks, this looks good! Two small comments (but none are blocking).

Comment thread doc/howto/instances_create.md Outdated
Comment thread doc/api-extensions.md Outdated
tomponline
tomponline reviewed Jul 3, 2024
View reviewed changes
Comment thread lxd/instances_post.go
tomponline
tomponline reviewed Jul 3, 2024
View reviewed changes
Comment thread lxd/instances_post.go Outdated
@tomponline tomponline mentioned this pull request Jul 3, 2024
Closed
@boltmark boltmark force-pushed the instance-start-on-creation branch from 941d643 to 86ad93a Compare July 18, 2024 01:43
@boltmark
Copy link
Copy Markdown
Contributor Author
boltmark commented Jul 18, 2024

@tomponline Ready for review. Looks like the documentation test is failing due to criu.org being down, but hopefully this is temporary.

tomponline
tomponline reviewed Jul 24, 2024
View reviewed changes
Comment thread lxc/init.go Outdated
tomponline
tomponline reviewed Jul 24, 2024
View reviewed changes
Comment thread doc/api-extensions.md
tomponline
tomponline reviewed Jul 24, 2024
View reviewed changes
Comment thread lxd/instances_post.go Outdated
tomponline
tomponline requested changes Jul 24, 2024 8000
View reviewed changes
Copy link
Copy Markdown
Member
@tomponline tomponline left a comment

Choose a reason for hiding this comment

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

Couple of nits and question about the migration fix

stgraber added 5 commits July 24, 2024 10:47
@stgraber @boltmark
api: instance_create_start
090e6b7 
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sponsored-by: Lanes & Planes GmbH (https://www.lanes-planes.com)
(cherry picked from commit a00b1aa4a713c7eb1460701ac6127c151fe72548)
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
License: Apache-2.0
@stgraber @boltmark
shared/api: Add Start to InstancesPost
22b2b58 
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sponsored-by: Lanes & Planes GmbH (https://www.lanes-planes.com)
(cherry picked from commit d85d6843ed5631660ca2375e797597174715c83f)
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
License: Apache-2.0
@stgraber @boltmark
doc/rest-api: Refresh swagger YAML
7d42e41 
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sponsored-by: Lanes & Planes GmbH (https://www.lanes-planes.com)
(cherry picked from commit b8c293da5a3399b8b16dcd2b5875dc5b4dd5323f)
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
License: Apache-2.0
@stgraber @boltmark
lxd/instance: Add support for Start property
81f758b 
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sponsored-by: Lanes & Planes GmbH (https://www.lanes-planes.com)
(cherry picked from commit 2405063803f5cc79198910014c48f4497b2ac091)
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
License: Apache-2.0
@stgraber @boltmark
lxc/launch: Use the Start property
48dc9c4 
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sponsored-by: Lanes & Planes GmbH (https://www.lanes-planes.com)
(cherry picked from commit aa15dd4d73dd3fee6d7abf19df307051d4ebfee7)
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
License: Apache-2.0
@boltmark boltmark force-pushed the instance-start-on-creation branch from 86ad93a to 2cddfe6 Compare July 24, 2024 18:45
boltmark added 2 commits July 24, 2024 14:26
@boltmark
doc/howto: Update documentation on start field
1428e4a 
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
@boltmark
i18n: Update translation templates
846fe00 
Signed-off-by: Mark Bolton <mark.bolton@canonical.com>
@boltmark boltmark force-pushed the instance-start-on-creation branch from 2cddfe6 to 846fe00 Compare July 24, 2024 21:27
@boltmark
Copy link
Copy Markdown
Contributor Author
boltmark commented Jul 24, 2024

Couple of nits and question about the migration fix

Thanks again @tomponline! I've resolved the nits and provided an alternative fix for the migration issue. Let me know what you think.

tomponline
tomponline approved these changes Jul 25, 2024
View reviewed changes
Copy link
Copy Markdown
Member
@tomponline tomponline left a comment

Choose a reason for hiding this comment

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

Thanks!

@tomponline tomponline merged commit 0021632 into canonical:main Jul 25, 2024
MggMuggins added a commit to MggMuggins/lxd that referenced this pull request Oct 24, 2024
@MggMuggins
lxc/launch: Start console if instance start was performed on server
535478e 
Fixes a regression introduced in canonical#13695

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
MggMuggins added a commit to MggMuggins/lxd that referenced this pull request Oct 24, 2024
@MggMuggins
lxc/launch: Start console if instance start was performed on server
18f4c42 
Fixes canonical#14339

Fixes a regression introduced in canonical#13695

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
@MggMuggins MggMuggins mentioned this pull request Oct 24, 2024
Merged
tomponline added a commit that referenced this pull request Oct 25, 2024
@tomponline
lxc/launch: Start console if instance start was performed on server (#…
b15f666 
…14340)

Fixes #14339

Fixes a regression introduced in #13695
tomponline pushed a commit to tomponline/lxd that referenced this pull request Dec 4, 2024
@MggMuggins @tomponline
lxc/launch: Start console if instance start was performed on server
9ef29f9 
Fixes canonical#14339

Fixes a regression introduced in canonical#13695

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
(cherry picked from commit 18f4c42)
tomponline pushed a commit to tomponline/lxd that referenced this pull request Dec 9, 2024
@MggMuggins @tomponline
lxc/launch: Start console if instance start was performed on server
4ce9f4a 
Fixes canonical#14339

Fixes a regression introduced in canonical#13695

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
(cherry picked from commit 18f4c42)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tomponline tomponline tomponline approved these changes

+1 more reviewer

@ru-fu ru-fu ru-fu left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

API Changes to the REST API Documentation Documentation needs updating

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add create and start endpoint 3CFE

4 participants

@boltmark @ru-fu @tomponline @stgraber
0