Skip to content

Commit fbb79d3

Browse files
evallespbrjackma
authored andcommitted
Fix CI for check-role-prefix script
We're skipping Single quotes in sed as we want literal $ in the sed pattern, not shell expansion. Added this file to irrelevant-files. Signed-off-by: Enrique Vallespi Gil <[email protected]>
1 parent edbe5fa commit fbb79d3

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

scripts/check-role-prefix.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ MSG=$(head -n 1 "$TMP_MSG_FILE")
2424
ROLE_COUNT=$(echo "$CHANGED_ROLES" | tr '|' '\n' | wc -l)
2525

2626
if [ "$ROLE_COUNT" -eq 1 ]; then
27-
# shellcheck disable=SC1087
27+
# shellcheck disable=SC2016
2828
ESCAPED_ROLE=$(printf '%s\n' "$CHANGED_ROLES" | sed 's/[]\.*^$()+?{|]/\\&/g')
29-
PATTERN="^[[(]$ESCAPED_ROLE[])]"
29+
PATTERN="^[[(]${ESCAPED_ROLE}[])]"
3030
else
3131
PATTERN="^[[(](multiple)[])]"
3232
fi

zuul.d/base.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353
- .pre-commit-config.yaml
5454
- .readthedocs.yaml
5555
- .spellcheck.yml
56+
- scripts/check-role-prefix.sh
5657
- roles/dlrn_report
5758
- roles/dlrn_promote
5859
- roles/devscripts

zuul.d/project-templates.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,8 @@
5757
requires:
5858
- cifmw-pod-pre-commit
5959
- cifmw-molecule
60+
irrelevant-files:
61+
- scripts/check-role-prefix.sh
6062
- podified-multinode-edpm-deployment-crc: *content_provider
6163
- cifmw-crc-podified-edpm-baremetal: *content_provider
6264
- podified-multinode-hci-deployment-crc: *content_provider

0 commit comments

Comments
 (0)