Skip to content

Commit

Permalink
ci: Fix OWNER variable in netperf workflow for push events
Browse files Browse the repository at this point in the history
The network performance workflow has a step named `vars` which
determines an appropriate value for an environment variable named OWNER.
This environment variable is added as a label into the GKE cluster the
workflow creates. This commit fixes a bug where, if the branch the
workflow is ran from contained a forward-slash, cluster creation would
fail, as GKE cluster labels cannot contain forward slashes.

Signed-off-by: Ryan Drew <ryan.drew@isovalent.com>
  • Loading branch information
learnitall authored and ldelossa committed May 7, 2024
1 parent 1dd302b commit 49ca139
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/net-perf-gke.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ jobs:
else
SHA="${{ github.sha }}"
OWNER="${{ github.ref_name }}"
OWNER="${OWNER/./-}"
OWNER="${OWNER//[.\/]/-}"
fi
CILIUM_INSTALL_DEFAULTS="--chart-directory=install/kubernetes/cilium \
Expand Down

0 comments on commit 49ca139

Please sign in to comment.