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

Tumbleweed: Introduce explicit {gnome,kde,textmode}+apparmor jobs #590

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

Vogtinator
Copy link
Member

With SELinux by default, we need explicit testing with AppArmor. Use the full testsuites instead of INSTALLONLY=1 for better coverage.

Depends on os-autoinst/os-autoinst-distri-opensuse#21078

@DimStar77
Copy link
Contributor

Do we need an explicit APPARMOR=1 setting there? instead of just SELINUX=0 implying it?

Other than that, absolutely agree that we need these tests

@DimStar77
Copy link
Contributor

we probably also need some create_hdd to run the current AppArmor tests that are run behind 'textmode' now

@Vogtinator
Copy link
Member Author

Do we need an explicit APPARMOR=1 setting there? instead of just SELINUX=0 implying it?

No. Those jobs are just "test TW with AppArmor", not "test the AppArmor stack". The latter is done by the SECURITY_TEST=apparmor ones.

@Vogtinator
Copy link
Member Author

we probably also need some create_hdd to run the current AppArmor tests that are run behind 'textmode' now

Indeed. I'll try to make that work.

With SELinux by default, we need explicit testing with AppArmor.
Use the full testsuites instead of INSTALLONLY=1 for better coverage.
@Vogtinator
Copy link
Member Author

we probably also need some create_hdd to run the current AppArmor tests that are run behind 'textmode' now

Indeed. I'll try to make that work.

Done. No idea if it works.

I added back the +selinux tests so we could merge this at any time to see if the explicit apparmor stuff works.

@Vogtinator Vogtinator merged commit 84557fa into os-autoinst:master Feb 6, 2025
2 checks passed
@Vogtinator Vogtinator deleted the explicitapparmor branch February 6, 2025 10:22
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