Only create placeholder device when encryption is enabled#3856
Only create placeholder device when encryption is enabled#3856jbaublitz wants to merge 1 commit intostratis-storage:masterfrom
Conversation
|
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. |
|
Cockpit tests failed for commit a5b83bc. @martinpitt, @jelly, @mvollmer please check. |
e531f88 to
420832e
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. |
2 similar comments
|
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. |
|
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. |
2e908c2 to
19695b2
Compare
|
@mulkieran Okay, I believe this is ready for review. The only tests that seem to be failing are the rawhide tests. |
|
After we just discussed this, this will likely be demoted to 3.9.0 or 3.9.1 depending on how hard it is to evaluate the need for migration code. |
19695b2 to
cb9567f
Compare
|
Blocked by #3800 |
cb9567f to
96b24f6
Compare
2e94cf6 to
a3a2687
Compare
|
@mulkieran I believe this may be rendered obsolete by the performance testing. I think the only thing we may need to test is slower drives. If there's a large performance degradation in V2, this may be worth pursuing. |
|
I confirmed with @spetrovi that performance is consistent across slower and faster drives. I'm going to close the linked issue. |
Closes #3840