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

ci: run tpm2-tools test suite #1969

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

Conversation

williamcroberts
Copy link
Member

To ensure changes in the tpm2-tss don't break dependent projects, run
the tpm2-tools test suite as part of the CI. This also builds tpm2-abrmd
since the tpm2-tools test suite is dependent on that project.

Signed-off-by: William Roberts [email protected]

To ensure changes in the tpm2-tss don't break dependent projects, run
the tpm2-tools test suite as part of the CI. This also builds tpm2-abrmd
since the tpm2-tools test suite is dependent on that project.

Signed-off-by: William Roberts <[email protected]>
@williamcroberts
Copy link
Member Author

It appears that python is not installed on Fedora-32 image, just python3. It should be compatible for our purposes, so Ill need to symlink it or do alternatives. Also, no option --enable-integration for tpm2-tools.

@williamcroberts
Copy link
Member Author

Actually I should probably fix this so it works on Fedora-32 without Python 2.X installed, since it appears that python might not always be available if a python is installed and theirs some inconsistency over that executable name. Ill fix the tools up and respin this when ready.

@williamcroberts
Copy link
Member Author

I started the tools fixes here: tpm2-software/tpm2-tools#2332

@tstruk
Copy link
Contributor

tstruk commented Mar 5, 2021

Do we know how much this will increase the CI run time?

@williamcroberts
Copy link
Member Author

Do we know how much this will increase the CI run time?

No, but I would like to do is add it to a specific included run rather than all the test runs. So this way it's more isolated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants