-
Notifications
You must be signed in to change notification settings - Fork 929
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
Preparation for 4.0.10 (stable-4.0) #13704
Merged
tomponline
merged 87 commits into
canonical:stable-4.0
from
simondeziel:preparation-for-4.0.10
Jul 10, 2024
Merged
Preparation for 4.0.10 (stable-4.0) #13704
tomponline
merged 87 commits into
canonical:stable-4.0
from
simondeziel:preparation-for-4.0.10
Jul 10, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This must have been around a long time but only became visible now that cloud-init is making use of /dev/lxd. Closes canonical#10231 Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 2c5b007) Signed-off-by: Simon Deziel <[email protected]>
spawn=allow is required when QEMU is asked to daemonize. Previous QEMU versions would incorrectly block `fork` when spawn=deny was passed while allowing clone to succeed. This was then making it possible for us to use daemnize thanks to most Linux distributions using the clone syscall to implement fork. Current QEMU has fixed their seccomp profile to block both fork and clone, preventing -daemonize when spawn=deny is passed. Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit af5c49e) Signed-off-by: Simon Deziel <[email protected]>
``` for f in $(git grep -lF github.com/lxc/lxd); do sed -i 's|github.com/lxc/lxd|github.com/canonical/lxd|g' "${f}" done ``` Signed-off-by: Simon Deziel <[email protected]>
github-actions
bot
added
Documentation
Documentation needs updating
API
Changes to the REST API
labels
Jul 5, 2024
simondeziel
changed the title
Preparation for 4.0.10
Preparation for 4.0.10 (stable-4.0)
Jul 5, 2024
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
2 times, most recently
from
July 5, 2024 18:38
c745c7d
to
e0807e8
Compare
markylaing
reviewed
Jul 8, 2024
ru-fu
previously approved these changes
Jul 8, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Great job getting the docs to build again! 👍
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
from
July 8, 2024 13:29
51eafbb
to
988ff40
Compare
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
4 times, most recently
from
July 9, 2024 04:07
baad25f
to
d55365f
Compare
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 033ae33)
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
5 times, most recently
from
July 9, 2024 06:17
6f32b6b
to
1b751ed
Compare
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]> (cherry picked from commit 5c60a70)
Signed-off-by: Simon Deziel <[email protected]>
uuidgen was used to generate a single placeholder remote name. If we ever want a UUID generator back, we should use: `systemd-id128 new -u` Which doesn't depend on an additional package. Signed-off-by: Simon Deziel <[email protected]> (cherry picked from commit 32b92f6)
Signed-off-by: Simon Deziel <[email protected]>
On LXD 4.0, the metadata LV is hardcoded to 1G so we need a bigger pool to have space for data LVs. Later LXD versions changed this to be dynamic and/or configurable. Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
from
July 9, 2024 22:23
a07f3d0
to
724e7ac
Compare
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
2 times, most recently
from
July 10, 2024 01:44
d7add48
to
f6ce644
Compare
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
…eric Signed-off-by: Thomas Parrott <[email protected]> (cherry picked from commit a13b571) Signed-off-by: Simon Deziel <[email protected]>
Fails on GitHub runners with: `ERROR: cannot find parent subvolume` Signed-off-by: Simon Deziel <[email protected]>
Signed-off-by: Simon Deziel <[email protected]>
simondeziel
force-pushed
the
preparation-for-4.0.10
branch
from
July 10, 2024 03:07
f6ce644
to
b31ae68
Compare
tomponline
approved these changes
Jul 10, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR should have a commit list very similar to that of #13643