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

Add option to enforce AppArmor profiles in tests #199

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Conversation

baierjan
Copy link
Member

@baierjan baierjan commented Aug 22, 2024

Introduce variable USE_APPARMOR to allow enforcing AppArmor profiles inside tests. Use the new variable inside nginx install variant of the test.

This needs os-autoinst/openQA#5883 first, otherwise the inner test will not pass.

Reference: https://progress.opensuse.org/issues/155413

Validation run (currently expected to fail): https://openqa.opensuse.org/tests/4423875

Copy link
Contributor

mergify bot commented Aug 22, 2024

This pull request is now in conflicts. Could you fix it? 🙏

Introduce variable `USE_APPARMOR` to allow enforcing AppArmor profiles
inside tests. Use the new variable inside nginx install variant of the
test.

Reference: https://progress.opensuse.org/issues/155413
@baierjan baierjan added the WIP label Aug 22, 2024
@mergify mergify bot merged commit 7c9d4ee into master Aug 22, 2024
6 checks passed
@mergify mergify bot deleted the apparmor branch August 22, 2024 18:47
@baierjan
Copy link
Member Author

Argh, mergify apparently ignores WIP label...

@baierjan baierjan removed the WIP label Aug 22, 2024
@perlpunk
Copy link
Contributor

perlpunk commented Aug 22, 2024

https://github.com/os-autoinst/os-autoinst-distri-openQA/blob/master/.mergify.yml#L6

- -label~=^acceptance-tests-needed|not-ready

@baierjan
Copy link
Member Author

Yes, I found out after. I created the missing not-ready label for this project and deleted the WIP to prevent similar confusion in the future.

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.

4 participants