Per-pool project limits (from Incus) #14078
Merged
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
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit adf17104edf5c87bb760a00fa97112f316f899a0) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 3c5bf91e1bad1e2e5606c696656ee25cf5fcdcb0) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 0ef36a6e786ad1dcf368135291db3cbad003c84a) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 6a099288b3502c4456a682e4dc8e7d7c03e07fe1) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Closes lxc/incus#906 Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit febce0851933037db19ebc7a26bf9982a1be4c84) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit e2df60ba28b599d4d6d1df5dfe88a5c0d6c5652b) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 80f0ceacf42c8b61b28d6f5f304dd4059927a801) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
Signed-off-by: Stéphane Graber <[email protected]> (cherry picked from commit 8d846de3beef3c78161b417cc78a304522ff0b75) Signed-off-by: Wesley Hershberger <[email protected]> License: Apache-2.0
github-actions
bot
added
Documentation
|
Heads up @mionaalex - the "Documentation" label was applied to this issue. |
MggMuggins
changed the title
MggMuggins
changed the title
simondeziel
changed the title
tomponline
reviewed
Sep 11, 2024
tomponline
reviewed
Sep 11, 2024
tomponline
reviewed
Sep 11, 2024
Signed-off-by: Wesley Hershberger <[email protected]>
Signed-off-by: Wesley Hershberger <[email protected]>
Signed-off-by: Wesley Hershberger <[email protected]>
tomponline
added a commit
that referenced
this pull request
Sep 13, 2024
From lxc/incus#1109. After #14078 got merged the following error started to appear on MicroCloud when forming clusters: ` Error: System "micro02" failed to join the cluster: Failed to update cluster status of services: Failed to join "LXD" cluster: Failed to configure cluster: Failed to initialize member: Failed to initialize storage pools and networks: Failed to update local member project "default": Failed loading storage pool names: Storage pool(s) not found` The DB`s `GetStoragePoolNames()` returns 404 in case of no storage pools. This has to be checked explicitly otherwise the error is returned back to the caller.
|
Bit late, but should Maybe at least under |
|
Good spot @MusicDin ! @MggMuggins please can you open a follow up PR, thanks. |
MggMuggins
added a commit
to MggMuggins/lxd
that referenced
this pull request
Sep 16, 2024
RE canonical#14078 (comment) Incus must have changed their metadata tool's syntax slightly. Signed-off-by: Wesley Hershberger <[email protected]>
tomponline
added a commit
that referenced
this pull request
Sep 17, 2024
RE #14078 (comment) I guess we changed the metadata tool's syntax slightly.
tomponline
pushed a commit
to tomponline/lxd
that referenced
this pull request
Dec 4, 2024
RE canonical#14078 (comment) Incus must have changed their metadata tool's syntax slightly. Signed-off-by: Wesley Hershberger <[email protected]> (cherry picked from commit 71f68de)
tomponline
pushed a commit
to tomponline/lxd
that referenced
this pull request
Dec 9, 2024
RE canonical#14078 (comment) Incus must have changed their metadata tool's syntax slightly. Signed-off-by: Wesley Hershberger <[email protected]> (cherry picked from commit 71f68de)
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.
Implements
limits.disk.pool.POOLNAMEconfig key on projects. When set, the key prevents a project using more than the configured amount of space within that pool.When
limits.disk.pool.POOLNAME: 0, the pool is excluded fromlxc storage listin that project.Contains cherry-picks from lxc/incus#1098