New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
envoy: Support more envoy image tag formats #24750
Conversation
This commit is to add the support for below image tags Different envoy image tag formats: ``` quay.io/cilium/cilium-envoy:f195a0a836629ceca5d7561f758c9505d9ebaebfa262647a2d4 quay.io/cilium/cilium-envoy:v1.23-f195a0a836629ceca5d7561f758c9505d9ebaebfa262647a2d4 ``` Testing was done as per below, kindly note the existing format should be working as usual. ```bash $ test=quay.io/cilium/cilium-envoy:014ceeb312a4d18dcf0ea219143f099fa91f2f28@sha256:1a3020822e8fb10b5f96bf45554690c411c2f48d8ca8fcf33da871dad1ce6b53 $ echo $test | sed -E -e 's/[^/]*\/[^:]*:([^:@]*).*/\1/p;d' | sed -E -e 's/.*-([^-]*)$/\1/' 014ceeb312a4d18dcf0ea219143f099fa91f2f28 $ test=quay.io/cilium/cilium-envoy:v1.24-014ceeb312a4d18dcf0ea219143f099fa91f2f28@sha256:1a3020822e8fb10b5f96bf45554690c411c2f48d8ca8fcf33da871dad1ce6b53 $ echo $test | sed -E -e 's/[^/]*\/[^:]*:([^:@]*).*/\1/p;d' | sed -E -e 's/.*-([^-]*)$/\1/' 014ceeb312a4d18dcf0ea219143f099fa91f2f28 ``` Fixes: cilium#24749 Signed-off-by: Tam Mach <tam.mach@cilium.io>
Full CI is not required, the current format is verified in GHA. |
/test |
/ci-awscni |
/ci-external-workloads |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
non blocking nit: perhaps we can merge the 2 seds commands?
Something like:
➜ ~ cat test.sh
test=quay.io/cilium/cilium-envoy:014ceeb312a4d18dcf0ea219143f099fa91f2f28@sha256:1a3020822e8fb10b5f96bf45554690c411c2f48d8ca8fcf33da871dad1ce6b53
echo $test | sed -E -e 's/[^/]*\/[^:]*:(.*-)?([^:@]*).*/\2/p;d'
test=quay.io/cilium/cilium-envoy:v1.24-014ceeb312a4d18dcf0ea219143f099fa91f2f28@sha256:1a3020822e8fb10b5f96bf45554690c411c2f48d8ca8fcf33da871dad1ce6b53
echo $test | sed -E -e 's/[^/]*\/[^:]*:(.*-)?([^:@]*).*/\2/p;d'
➜ ~ ./test.sh
014ceeb312a4d18dcf0ea219143f099fa91f2f28
014ceeb312a4d18dcf0ea219143f099fa91f2f28
(please double test this as I'm a bit rusty with regexes)
There was an issue with MLH, so Mergeability is stuck. Considered that all CI jobs are successful, review is in, marking this ready to merge. |
Relates: cilium#24750 (review) Signed-off-by: Tam Mach <tam.mach@cilium.io>
Relates: #24750 (review) Signed-off-by: Tam Mach <tam.mach@cilium.io>
This commit is to add the support for the below image tags
Different envoy image tag formats:
Testing was done as per below, kindly note the existing format should be working as usual.
Fixes: #24749