Disable AppArmor before running Molecule tests #223
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🗣 Description
This pull request modifies the
build.yml
workflow to leverage cisagov/action-disable-apparmor to disable AppArmor before running the Molecule tests.💭 Motivation and context
This is a better (less janky) solution to the AppArmor woes than #216. We are also already taxing the free GitHub runners, and I have seen failures in the steps added in #216. This solution should tax the runners a bit less.
It is also worth noting that lot of folks (me included) are of the opinion that AppArmor shouldn't be enabled in the GitHub runners.
Resolves #215.
🧪 Testing
All automated tests pass.
✅ Pre-approval checklist
✅ Pre-merge checklist
v1
tag of cisagov/action-disable-apparmor once First commits action-disable-apparmor#1 is approved and merged.