-
Notifications
You must be signed in to change notification settings - Fork 2.8k
feat(source/istio): support version 1.25+ #5611
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
feat(source/istio): support version 1.25+ #5611
Conversation
Signed-off-by: ivan katliarchuk <[email protected]>
Signed-off-by: ivan katliarchuk <[email protected]>
Signed-off-by: ivan katliarchuk <[email protected]>
Signed-off-by: ivan katliarchuk <[email protected]>
Signed-off-by: ivan katliarchuk <[email protected]>
Hi @mthemis-provenir, @simonostendorf and @stevehipwell can you confirm that this PR fixes your issue ? If needed, see how to test a PR ?. This will require golang to be installed. |
Signed-off-by: ivan katliarchuk <[email protected]>
Signed-off-by: ivan katliarchuk <[email protected]>
Would it not make more sense to target support of Istio 1.26, given that only 1.25 and 1.26 are supported versions? Resources are stored as v1 in 1.26, so I doubt v1beta1 support will change anything. |
Everything is stored in I will try You could try |
Strange, you are correct - gateways and virtualservices are stored as |
Co-authored-by: mthemis-provenir <[email protected]>
/lgtm |
Any chance of an approval / merge here, guys? Would be good to get external-dns supporting Istio releases that aren't EOL. |
I have no issues merging this; the change is approved. However, please be aware that the next release is likely to occur in September due to summer holiday period |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ivankatliarchuk The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
docker pull gcr.io/k8s-staging-external-dns/external-dns:v20250707-v0.18.0-25-g252a5e01 |
* feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.22 Signed-off-by: ivan katliarchuk <[email protected]> * feat(source/istio): support version 1.25+ Co-authored-by: mthemis-provenir <[email protected]> --------- Signed-off-by: ivan katliarchuk <[email protected]> Co-authored-by: mthemis-provenir <[email protected]>
What does it do ?
istio.md
with pre-commit-hookCurrently supported version is 1.25, as 1.22 end of life is Jan 22, 2025
Stored versions
It is safe to use
v1
, but I think we should supportv1beta
whilev1beta
is current storage versionMotivation
Fixes: #4473, #2798
Follow-up:
istio-virtualservice
in some occasions incorrectly identify RecordTypecname
instead ofa
recordMore
Smoke test for
kind: Gateway
Arguments
go run main.go \ --provider=aws \ --registry=txt \ --source=service \ --log-level=info \ --source=istio-gateway \ --policy=sync \ --fqdn-template="{{ .Kind | toLower }}-{{ .APIVersion | replace \"networking.istio.io/\" \"\"}}-{{.Name}}.local.tld" \ --combine-fqdn-annotation \ --domain-filter=local.tld
Create

Delete
Virtual Service
Manifest files
Create

Delete