Skip to content

Commit b777e77

Browse files
authored
Merge pull request #3 from cisagov/lineage/skeleton
⚠️ CONFLICT! Lineage pull request for: skeleton
2 parents 7c80e08 + be81a0c commit b777e77

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

action.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,14 @@ branding:
66
description: Disables AppArmor on the GitHub Actions runner.
77
name: Disable AppArmor
88

9+
# We use a shell argument for each step that provides the same guardrails we
10+
# would have in our bash scripts. The `-Eueo pipefail` sets errtrace, nounset,
11+
# errexit, and pipefail. The `-x` will print all commands as they are run. We
12+
# are currently unable to set this as a default (like we can in workflows) so
13+
# this must be added to each step that runs shell commands. Please see the
14+
# following GitHub discussion that requests this feature for any status
15+
# updates: https://github.com/orgs/community/discussions/18597
16+
917
runs:
1018
using: composite
1119
steps:
@@ -16,9 +24,9 @@ runs:
1624
# https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/2093797
1725
# for more details.
1826
run: sudo aa-teardown || true
19-
shell: bash
27+
shell: bash -Eueo pipefail -x {0}
2028

2129
- id: disable-apparmor
2230
name: Disable and stop AppArmor systemd service
2331
run: sudo systemctl disable --now apparmor.service
24-
shell: bash
32+
shell: bash -Eueo pipefail -x {0}

0 commit comments

Comments
 (0)