-
Notifications
You must be signed in to change notification settings - Fork 56
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
Require emulated TPM2 for Stratis tests #3617
Conversation
Cockpit tests failed for commit f78f27f. @martinpitt, @jelly, @mvollmer please check. |
All our tests failed w/ Artemis, guest couldn't be provisioned error. Cockpit tests failed with some error that is surely unrelated to stratisd changes. So we can be pretty sure that it is the configuration changes, for our tests only, that are breaking something. |
Ok. So this approach is guaranteed not to work. We would probably have to provision in the normal way and then use swtpm to emulate the TPM2 module on the provided virtual machine. |
e0006ab
to
81314d3
Compare
Cockpit tests failed for commit e0006ab. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 81314d3. @martinpitt, @jelly, @mvollmer please check. |
our tests seem to be stuck running with no information at all. |
Cockpit tests failed for commit 6fb6869. @martinpitt, @jelly, @mvollmer please check. |
I failed to background a step...and this induced an eventual timeout... |
Cockpit tests failed for commit f4acf6f. @martinpitt, @jelly, @mvollmer please check. |
shell syntax interpretation problems; I think this attempt may work |
Cockpit tests failed for commit b40b9ad. @martinpitt, @jelly, @mvollmer please check. |
Cockpit tests failed for commit 4845fa0. @martinpitt, @jelly, @mvollmer please check. |
rebased |
plans/all.fmf
Outdated
- name: Show TPM2 information | ||
how: shell | ||
script: tpm2_getrandom 16 | hexdump |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this go in the final version? I thought it was just for testing that the TPM was correctly provisioned during development of the PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't mind dropping it.
Signed-off-by: mulhern <amulhern@redhat.com>
@jbaublitz This is ready again. |
No description provided.