-
Notifications
You must be signed in to change notification settings - Fork 55
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
base: master
Are you sure you want to change the base?
Conversation
ab59741
to
06426fd
Compare
Next time, add temporary commit to not run Packit stuff, for that is a waste. |
Cockpit tests failed for commit 06426fd. @martinpitt, @jelly, @mvollmer please check. |
Plz ignore. We should be skipping Packit tests anyway...for this PR. |
d3fc6b2
to
30b3d6f
Compare
@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. What I want to say is: Feel free to run such tests also against cockpit-storage, if it helps you/you want additional coverage. |
d293d76
to
a5e6a83
Compare
To test stratis-storage/ci#510 |
Testing stratis-storage/ci#511 |
36b0e6b
to
3ff9038
Compare
e1bb325
to
9367b37
Compare
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. |
c30709a
to
9367b37
Compare
I believe the test will fail now... |
41d2421
to
f0a1d25
Compare
Likely this will pass, good. |
9705aee
to
b63d6fe
Compare
b63d6fe
to
af3164c
Compare
af3164c
to
7651200
Compare
This one should realize a single signal failure with adding a cache device |
8eb2648
to
3e445ec
Compare
We expect tests to succeed now. |
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]>
Signed-off-by: mulhern <[email protected]>
Signed-off-by: mulhern <[email protected]>
3e445ec
to
710b27d
Compare
No description provided.