Backports (stable-5.0) #13659
Merged
Backports (stable-5.0) #13659
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
Additionally check the responses if the endpoint gets queried using the project parameter as this is producing different results for restricted and unrestricted metrics certificates. Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Eric Gelinas <eric.gelinas@canonical.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
tomponline
changed the title
Signed-off-by: Eric Gelinas <eric.gelinas@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
The vg_name is not a path and gives ENOENT when we try to open it. Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: hamistao <pedro.ribeiro@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
I noticed sometimes during shutdown we would see these errors in the log. Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Because `FillConfig` changes the meaning of d.config["source"], we want to check against the user's intent here, rather than the filesystem's UUID (which may not exist yet; eg a raw block device) Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Need to be able to check the block size associatd with a volume for zfs (see next commit) Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Fixes canonical#12578 Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Fixes canonical#12491 Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Din Music <din.music@canonical.com>
… the filesystem Signed-off-by: Din Music <din.music@canonical.com>
When refreshing a volume with its remote counterpart make sure to run GetVolume() which does not assign a new volatile.uuid as the volume on the target already exists. Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
This allows passing a custom timeout. Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com>
Signed-off-by: Din Music <din.music@canonical.com>
…P_NET_BIND_SERVICE) Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
…tens on TCP/53 Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
…volume Instead of giving a udev rules half a second to be applied, wait for the device up to 30 seconds to appear. This fixes the issue where zvol device cannot be located if the rules are not applied in time. Signed-off-by: Din Music <din.music@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com> (cherry picked from commit 288f85f)
In core24 AppArmor includes security fixes and our ruleset, although the source code remains unchanged become stricter. devpts was always available for unprivileged containers because of AppArmor bugs like [1]. Let's now allow it explicitly. [1] https://bugs.launchpad.net/apparmor/+bug/1597017 Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
Signed-off-by: Julian Pelizäus <julian.pelizaeus@canonical.com> (cherry picked from commit f0316ad)
tomponline
requested review from
simondeziel,
roosterfish,
MusicDin and
mihalicyn
MusicDin
approved these changes
Jun 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.