Skip to content

Fix AppArmor configuration without daemon install #47

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 23, 2021
Merged

Fix AppArmor configuration without daemon install #47

merged 2 commits into from
Mar 23, 2021

Conversation

markgoddard
Copy link

@markgoddard markgoddard commented Mar 22, 2021 •
edited
Loading

  • If we are not installing the libvirtd daemon, we can't assume that there is a libvirt QEMU AppArmor template to configure.
  • The apparmor package may not always be installed.

@markgoddard markgoddard requested a review from priteau March 22, 2021 16:24
@markgoddard markgoddard self-assigned this Mar 22, 2021
@markgoddard markgoddard marked this pull request as draft March 23, 2021 11:04
@markgoddard
Ubuntu: Install apparmor package for apparmor_parser
17d1137 
In some cases AppArmor may be enabled, but the apparmor package that
provides apparmor_parser is not installed.
@markgoddard markgoddard force-pushed the fix-apparmor-no-daemon branch from f28fddb to 17d1137 Compare March 23, 2021 11:27
priteau
priteau reviewed Mar 23, 2021
View reviewed changes
@markgoddard markgoddard marked this pull request as ready for review March 23, 2021 15:08
@markgoddard markgoddard requested a review from priteau March 23, 2021 15:12
@markgoddard markgoddard merged commit 7964f3e into master Mar 23, 2021
@markgoddard markgoddard deleted the fix-apparmor-no-daemon branch March 23, 2021 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@priteau priteau priteau approved these changes

Assignees

@markgoddard markgoddard

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@markgoddard @priteau