Skip to content
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

Run testing repo tests against PR #3545

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

mulkieran
Copy link
Member

No description provided.

@mulkieran
Copy link
Member Author

Next time, add temporary commit to not run Packit stuff, for that is a waste.

Copy link

Cockpit tests failed for commit 06426fd. @martinpitt, @jelly, @mvollmer please check.

@mulkieran
Copy link
Member Author

mulkieran commented Feb 26, 2024

Cockpit tests failed for commit 06426fd. @martinpitt, @jelly, @mvollmer please check.

Plz ignore. We should be skipping Packit tests anyway...for this PR.

@mulkieran mulkieran force-pushed the testing-test branch 2 times, most recently from d3fc6b2 to 30b3d6f Compare February 26, 2024 19:34
@martinpitt
Copy link
Contributor

@mulkieran FTR, that test failure actually wasn't statis' fault, but an (unrelated) kernel regression. We are currently investigating that. Sorry for the noise on PRs, this is a hard one to debug.

rawhide even passed.

What I want to say is: Feel free to run such tests also against cockpit-storage, if it helps you/you want additional coverage.

@mulkieran mulkieran force-pushed the testing-test branch 4 times, most recently from d293d76 to a5e6a83 Compare March 11, 2024 16:13
@mulkieran
Copy link
Member Author

To test stratis-storage/ci#510

@mulkieran
Copy link
Member Author

Testing stratis-storage/ci#511

@mulkieran mulkieran force-pushed the testing-test branch 3 times, most recently from 36b0e6b to 3ff9038 Compare March 12, 2024 00:29
@mulkieran mulkieran force-pushed the testing-test branch 3 times, most recently from e1bb325 to 9367b37 Compare June 25, 2024 18:57
@mulkieran
Copy link
Member Author

mulkieran commented Jun 25, 2024

We are trying to exactly duplicate the behavior of the successful manual test in the auto-test: stratis-storage/testing@35b083c .

@mulkieran
Copy link
Member Author

We are trying to exactly duplicate the behavior of the successful manual test in the auto-test: stratis-storage/testing@35b083c .

But alas, the test succeeds.

@mulkieran mulkieran force-pushed the testing-test branch 2 times, most recently from c30709a to 9367b37 Compare June 28, 2024 02:02
@mulkieran
Copy link
Member Author

I believe the test will fail now...

@mulkieran
Copy link
Member Author

Likely this will pass, good.

@mulkieran
Copy link
Member Author

This one should realize a single signal failure with adding a cache device

@mulkieran
Copy link
Member Author

We expect tests to succeed now.

jbaublitz and others added 16 commits December 23, 2024 18:13
Specify default integrity journal size in parser as string value.

Signed-off-by: mulhern <[email protected]>
Define an enum to represent the tag size.

Store the IntegrityTagSpec value in the pool-level metadata.
Nothing is gained by turning the enum into bits before storing it in the
pool-level metadata.

Compute using the bytes value that is at least as large as the number of
bits the tag spec represents.

Use tag_spec instead of tag_size for the name of the field in the
CreatePool method.

Signed-off-by: mulhern <[email protected]>
Signed-off-by: mulhern <[email protected]>
Combine both current specs in the struct.

The default of both fields is None.

Signed-off-by: mulhern <[email protected]>
Use ValidatedIntegritySpec for serialization and to pass to methods
invoked by engine's create_pool method.

Signed-off-by: mulhern <[email protected]>
This is an option in the D-Bus, the predict script and the pool-level
metadata.

Signed-off-by: mulhern <[email protected]>
Have the string representation match that for D-Bus API.

Signed-off-by: mulhern <[email protected]>
Signed-off-by: mulhern <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants