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

FAPI Test: Add failure test no root certificate. #2750

Merged

Conversation

JuergenReppSIT
Copy link
Member

@JuergenReppSIT JuergenReppSIT commented Jan 15, 2024

  • It is now checked whether the provisioning fails if the appropriate root certificate does not exist.
    For this purpose the EK certificate is generated with a self signed certificate which is not in the list of the FAPI cerificates.
  • In the cirrus CI due to a race condition cause by gmake -j distcheck The test now is executed with gmake -j check.

@JuergenReppSIT JuergenReppSIT force-pushed the fapi-test-no-root-cert branch 2 times, most recently from 329e1e7 to ba60345 Compare January 15, 2024 10:08
@JuergenReppSIT JuergenReppSIT marked this pull request as draft January 15, 2024 10:09
@JuergenReppSIT JuergenReppSIT force-pushed the fapi-test-no-root-cert branch 5 times, most recently from 2ea2f52 to 8d1f471 Compare January 15, 2024 12:04
It is now checked whether the provisioning fails if the appropriate
root certificate does not exist.
For this purpose the EK certificate is generated with a self
signed certificate which is not in the list of the FAPI cerificates.

Signed-off-by: Juergen Repp <juergen_repp@web.de>
@JuergenReppSIT JuergenReppSIT marked this pull request as ready for review January 15, 2024 14:27
@JuergenReppSIT JuergenReppSIT force-pushed the fapi-test-no-root-cert branch 5 times, most recently from 68ca08f to 17dc77f Compare January 17, 2024 12:18
@JuergenReppSIT JuergenReppSIT marked this pull request as draft January 17, 2024 12:18
@JuergenReppSIT JuergenReppSIT force-pushed the fapi-test-no-root-cert branch 4 times, most recently from af82c86 to a09960f Compare January 17, 2024 13:55
@JuergenReppSIT JuergenReppSIT marked this pull request as ready for review January 17, 2024 14:14
The removing of the ca files is added to clean-local and
distclean-local in Makefile.am
The deletion in clean-hook and uninstall-local is removed.
"make -j check" is used instead of "make -j distcheck".

Signed-off-by: Juergen Repp <juergen_repp@web.de>
@AndreasFuchsTPM AndreasFuchsTPM merged commit 32ff82e into tpm2-software:master Jan 24, 2024
25 checks passed
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