stratis-storage / stratisd

Easy to use local storage management for Linux.
https://stratis-storage.github.io
Mozilla Public License 2.0
794 stars 55 forks source link

Add password verification for stratis-min #3595

Open jbaublitz opened 5 months ago

jbaublitz commented 5 months ago

Closes #3593

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 37298c2614ebf4f2d251d5d226e3cd9cc3b40936. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 2621892035b15e855f00b560672d1b6d7cacfc18. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 028dc7e7a3c1eaeeba13631ed2dc6d02395a32ec. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 49667e251b7f17665174518cf9d373a75e87cda8. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 215f803f17c13a5a2cfc08f8c005e3d2a0549391. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 8dcbec9aff5f994cf9e24aa60381ae435d855be4. @martinpitt, @jelly, @mvollmer please check.

packit-as-a-service[bot] commented 4 months ago

Cockpit tests failed for commit 911fbb0bcb7a78eae5e8667c9f947f71e712977d. @martinpitt, @jelly, @mvollmer please check.

mvollmer commented 4 months ago

Cockpit tests failed for commit 37298c2. @martinpitt, @jelly, @mvollmer please check.

This failure is unrelated to Stratis, please ignore. I'll investigate. (@martinpitt, can we configure things so that only the storage tests run here? Would that make sense?)

mvollmer commented 4 months ago

This failure is unrelated to Stratis, please ignore. I'll investigate.

This caused by the switch to DNF5 in Rawhide, see https://github.com/cockpit-project/cockpit/pull/20467 for a fix that makes the tests green again.

jbaublitz commented 4 months ago

@mvollmer Thanks for the heads up!

jbaublitz commented 1 month ago

@mulkieran Is it worth doing manual testing on this or should we wait until we can automate the testing to merge it?