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.

Sign up for GitHub

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

Jump to bottom

Regression test for CryptHandle unlocking with both keyring and Clevis enabled #3486

Merged
merged 2 commits into from
Oct 31, 2023
Merged

Regression test for CryptHandle unlocking with both keyring and Clevis enabled #3486

merged 2 commits into from
Oct 31, 2023

Conversation

jbaublitz
Copy link
Member

@jbaublitz jbaublitz commented Oct 30, 2023 •
edited by mulkieran
Loading

@jbaublitz jbaublitz self-assigned this Oct 30, 2023
@mulkieran
Copy link
Member

Test failure in tang tests was caused by a coincidental failure to download dependencies so I'm running it again.

@jbaublitz
Copy link
Member Author

Confirmed test failure where it is expected:

---- engine::strat_engine::backstore::crypt::tests::clevis_loop_test_both_initialize stdout ----
thread 'engine::strat_engine::backstore::crypt::tests::clevis_loop_test_both_initialize' panicked at src/engine/strat_engine/backstore/crypt/mod.rs:386:22:
called `Result::unwrap()` on an `Err` value: Msg("The key description \"test-description-for-stratisd\" is not currently set.")

@jbaublitz jbaublitz force-pushed the regression-test branch 2 times, most recently from c9508e5 to 6a71169 Compare October 30, 2023 21:38
@jbaublitz jbaublitz requested a review from mulkieran October 30, 2023 22:00
@jbaublitz jbaublitz mentioned this pull request Oct 31, 2023
Closed
@mulkieran
Copy link
Member

/packit test

@mulkieran mulkieran requested a review from bgurney-rh October 31, 2023 14:36
@mulkieran mulkieran merged commit e8c9365 into stratis-storage:master Oct 31, 2023
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bgurney-rh bgurney-rh bgurney-rh approved these changes

@mulkieran mulkieran mulkieran approved these changes

Assignees

@jbaublitz jbaublitz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jbaublitz @mulkieran @bgurney-rh