-
Notifications
You must be signed in to change notification settings - Fork 115
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
Add tests for VMware secureboot provisioning #17214
base: master
Are you sure you want to change the base?
Conversation
|
PRT Result
|
da5653f
to
169d55e
Compare
|
PRT Result
|
169d55e
to
c3b2c63
Compare
|
PRT Result
|
PRT Result
|
c3b2c63
to
50bf798
Compare
|
PRT Result
|
50bf798
to
022dd95
Compare
|
PRT Result
|
022dd95
to
110eb8d
Compare
|
PRT Result
|
PRT Result
|
110eb8d
to
be6a7fc
Compare
|
PRT Result
|
|
PRT Result
|
be6a7fc
to
a86f487
Compare
|
PRT Result
|
a86f487
to
8b96ccb
Compare
Waiting on RedHatQE/wrapanapi#501 |
@pytest.mark.rhel_ver_match('[8]') | ||
@pytest.mark.rhel_ver_list('[9, 10]') |
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.
why did we remove rhel 8 here?
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.
We're now running EL9 and EL10 provisioning tests with API, since its faster compared to CLI/UI
@@ -98,6 +109,7 @@ def test_positive_provision_end_to_end( | |||
'controller_key': 1001, | |||
}, | |||
}, | |||
'virtual_tpm': 'false' if pxe_loader.vm_firmware == 'bios' else 'true', |
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.
This should work, but in the hammer PR we mentioned that the value should be 0 or 1 theforeman/hammer-cli-foreman#630
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.
yes, both works, but I think both makes sense, so do we really need to change it?
@pytest.mark.rhel_ver_match(r'^(?!.*fips).*$') | ||
@pytest.mark.rhel_ver_match('[7]') |
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.
@Gauravtalreja1 Could you please explain this change?
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.
We're just running EL7 provisioning tests with CLI
26fec4d
to
1d5c6ea
Compare
|
PRT Result
|
|
PRT Result
|
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
Signed-off-by: Gaurav Talreja <gtalreja@redhat.com>
1d5c6ea
to
84028c4
Compare
|
PRT Result
|
Problem Statement
Missing test coverage for new feature of VMware secureboot provisioning
Solution
Update existing tests to include VMware UEFI secureboot provisioning
Related Issues