fix(ci): reorder gke tags and labels variable expansions #1522
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
If you're really unlucky, you can get a
.
at the end of thetags
variable, which will be substituted for-
after we replace the last-
withx
. GKE doesn't allow for-
at the end of the string, which is why we change the last-
.See openshift/release#47789 for a real-world example of this problem.
These changes reorders the shell expansion so we first replace all
.
with-
, then delete the last-
.Cherry-picked from #1513
Testing Performed
Testing the variable expansions: