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.

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mulkieran
Copy link
Member

No description provided.

@mulkieran mulkieran self-assigned this May 13, 2024
@mulkieran mulkieran added this to In progress (long term) in 2024May via automation May 13, 2024
@mulkieran mulkieran moved this from In progress (long term) to In progress in 2024May May 13, 2024
Copy link

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

@mulkieran
Copy link
Member Author

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.

@mulkieran
Copy link
Member Author

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.

@mulkieran mulkieran force-pushed the require-tpm2 branch 2 times, most recently from e0006ab to 81314d3 Compare May 13, 2024 20:50
Copy link

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

Copy link

Cockpit tests failed for commit 81314d3. @martinpitt, @jelly, @mvollmer please check.

@mulkieran
Copy link
Member Author

our tests seem to be stuck running with no information at all.

Copy link

Cockpit tests failed for commit 6fb6869. @martinpitt, @jelly, @mvollmer please check.

@mulkieran
Copy link
Member Author

mulkieran commented May 14, 2024

I failed to background a step...and this induced an eventual timeout...

Copy link

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

@mulkieran
Copy link
Member Author

shell syntax interpretation problems; I think this attempt may work

Copy link

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

Signed-off-by: mulhern <amulhern@redhat.com>
Copy link

Cockpit tests failed for commit 4845fa0. @martinpitt, @jelly, @mvollmer please check.

@mulkieran mulkieran moved this from In progress to Pending in 2024May May 14, 2024
@jbaublitz jbaublitz mentioned this pull request May 15, 2024
@mulkieran mulkieran removed this from Pending in 2024May May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Pending
Development

Successfully merging this pull request may close these issues.

None yet

1 participant