Version 3.7.0 #3691
Merged
Version 3.7.0 #3691
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
mulkieran
force-pushed
the
version-3.7.0
branch
from
525627f to
9f927a3
Compare
|
Congratulations! One of the builds has completed. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
Signed-off-by: mulhern <[email protected]>
Signed-off-by: mulhern <[email protected]>
mulkieran
force-pushed
the
version-3.7.0
branch
3 times, most recently
from
27feadd to
3ead336
Compare
|
Failures with testing repo and with stratis-cli are expected... |
mulkieran
force-pushed
the
version-3.7.0
branch
from
3ead336 to
44287ea
Compare
|
should pass stratis-cli tests |
mulkieran
force-pushed
the
version-3.7.0
branch
from
cd79e70 to
d00880a
Compare
|
success |
mulkieran
force-pushed
the
version-3.7.0
branch
from
d00880a to
21cbe0e
Compare
|
TMT failures can be ignored, so everything is passing. |
jbaublitz
approved these changes
Sep 27, 2024
mulkieran
force-pushed
the
version-3.7.0
branch
from
21cbe0e to
807c337
Compare
|
With the removal of the TEMPORARY commit stratis-cli tests will fail. We actually need to make that redirect permanent, though. |
Signed-off-by: mulhern <[email protected]>
(cherry picked from commit a196c6f)
Instead, allow create_archive.py to read the version from the package. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 5b289b6)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 5222537)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit fa4a036)
(cherry picked from commit 254add9)
…s enabled (cherry picked from commit fa2807a)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit 729ffa8)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit f0c0910)
Currently, data allocations from the cap device are data block aligned. Metadata allocations, however, are not. This can lead to data allocations after a metadata allocation to become unaligned as well. This has performance implications. This commit addresses this in a backwards compatiable way by using our previous metadata size calculation but rounds the total desired metadata device size up to the nearest data block size multiple. This has a few very desirable properties: 1. The metadata allocations will always be aligned to the data block size boundary. 2. The metadata allocations may appear a bit larger, but the overall growth of the step function tracks the linear growth of the original metadata space calculation function. 3. While this commit does not resolve alignment issues for pools that have already bumped into this problem, this code should theoretically cause future allocations on affected pools to be in alignment, minimizing performance impact for future allocations on affected pools. (cherry picked from commit 361f13f)
Signed-off-by: mulhern <[email protected]>
It gets really crufty, as we work hard to improve our tests and never think about the README when we change them. Also, I very much doubt that many people are reading that README. Signed-off-by: mulhern <[email protected]> (cherry picked from commit ba23a31)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 29603f2)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 9c76eac)
We haven't maintained pedantic lints for a very long time, but since we still obey these ones, we might as well hold onto them. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 123d176)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit 5127b8c)
Some of the lints listed as pedantic were at some point promoted to suspicious or style lints, which are already denied since those lint categories are implied by the all category, which is denied. Add further notations about the included pedantic and nursery lint deny entries to indicate that they can be removed without very deep consideration. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 97892ae)
Signed-off-by: mulhern <[email protected]>
Signed-off-by: mulhern <[email protected]> (cherry picked from commit c9daa0b)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit af7f9ab)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit f71b59e)
The default level is "off" since this option will be mostly for debugging. Signed-off-by: mulhern <[email protected]> (cherry picked from commit f6bd343)
Use HashSet type to enforce the set property. This allows more precise algorithms than the Vec type. Signed-off-by: mulhern <[email protected]> (cherry picked from commit d4c4e7b)
Thinpool's snapshot_filesystem() is protected from being called with a value for snapshot_name which is already in use by Pool.snapshot_filesystem. The assertion detects if thinpool's snapshot_filesystem() method is invoked improperly from thinpool test code. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 5a9e3e1)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit 88e9685)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 5e3b0cb)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 6cadfea)
Signed-off-by: Bryan Gurney <[email protected]> (cherry picked from commit b10f8bf)
Remove an unnecessary lint. Reduce the scope of a necessary lint. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 51209e9)
Do just one important retry code sequence: starting stratisd and ensuring that the D-Bus is available. We can do others later, as needed and opportunity presents itself. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 5cf3f4f)
Remove the related pylint disable Signed-off-by: mulhern <[email protected]> (cherry picked from commit 1212008)
Panic always if no argument for pool_mutating_actions. Panic if more than one of the designated attribute found. Signed-off-by: mulhern <[email protected]> (cherry picked from commit 1081905)
Signed-off-by: mulhern <[email protected]> (cherry picked from commit 25a4996)
mulkieran
force-pushed
the
version-3.7.0
branch
from
807c337 to
6736d79
Compare
|
Redirect to stratis-cli rebase branch now permanently part of this PR. No other changes. testing changes are still just a PR, so expect failures on those. |
|
TMT framework errors again... |
mulkieran
merged commit ae7b357
into
stratis-storage:rebase-3.6.0
46 of 51 checks passed
This was referenced Oct 4, 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.
Related stratis-storage/project#729