-
Notifications
You must be signed in to change notification settings - Fork 15
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 a test to assert on /etc/machine-id permissions for AWS #266
Conversation
@narmaku can you tag a new container version once this is approved and merged ? See https://bugzilla.redhat.com/show_bug.cgi?id=2221269#c6 |
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.
Let's use testinfra modules for this. It's the pytest library we use to run tests in remote hosts. Let me know if you need help with that.
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.
Please see my comments in last review. Thanks!
93c277a
to
f7b5bdd
Compare
test_suite/cloud/test_aws.py
Outdated
@@ -29,6 +29,15 @@ def instance_data_aws_cli(host, instance_data_aws_web): | |||
|
|||
@pytest.mark.order(2) | |||
class TestsAWS: | |||
@pytest.mark.pub |
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.
I think we need to remove this marker since if the change will be present during the builds done by Image Builder, the test should be run.
This marker is used to skip the test case in case what's going to be tested only applies to images that are about to be published, or already published (e.g. Bath/Update ad GA image validations). Examples of "pub" test cases are the ones related to rhui.
Related: RHBZ #2221269
f7b5bdd
to
d6e970a
Compare
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.
LGTM!
in order to exercise test for RHBZ#2221269. See: osbuild/cloud-image-val#266
in order to exercise test for RHBZ#2221269. See: osbuild/cloud-image-val#266 NOTE: changing more than necessary in order to satisfy CI tools
Related: RHBZ #2221269