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

Fix script logic issue & add error handling for tpm2_engine test #9956

Merged
merged 1 commit into from Apr 7, 2020

Conversation

rfan1
Copy link
Contributor

@rfan1 rfan1 commented Apr 3, 2020

There is an issue with wrong parameter within "expect" script. at the same time, add some error handling.

@lilyeyes
Copy link
Contributor

lilyeyes commented Apr 3, 2020

The "commit message" (SLES15SP2 security test:tpm2_enigne, fix poo#65220) is not proper.
Please revise it.

Copy link
Contributor

@lilyeyes lilyeyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rfan1 rfan1 changed the title SLES15SP2 security test:tpm2_enigne, fix poo#65220 Fix script logic issue & add error handling for tpm2_engine test Apr 3, 2020
@coolgw coolgw merged commit 7db71f4 into os-autoinst:master Apr 7, 2020
Copy link
Contributor

@jouyingbin jouyingbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after confirmed.

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