Skip to content
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

Regression in service entry merging in 1.19 -> 1.20 #49489

Closed
howardjohn opened this issue Feb 22, 2024 · 4 comments · Fixed by #49507
Closed

Regression in service entry merging in 1.19 -> 1.20 #49489

howardjohn opened this issue Feb 22, 2024 · 4 comments · Fixed by #49507

Comments

@howardjohn
Copy link
Member

apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  name: example-443
spec:
  exportTo:
  - .
  hosts:
  - www.google.co.uk
  location: MESH_EXTERNAL
  ports:
  - name: i443
    number: 443
    protocol: TLS
  resolution: DNS
---
apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  name: example-8443
spec:
  exportTo:
  - .
  hosts:
  - www.google.co.uk
  location: MESH_EXTERNAL
  ports:
  - name: i8443
    number: 8443
    protocol: TLS
  resolution: DNS

In 1.20: STRICT_DNS cluster without endpoints outbound|443||www.google.co.uk found while pushing CDS

cc @stevenctl @ramaraochavali @hzxuzhonghu - I haven't looked into it yet, but all 4 of us have PRs in this area around this time

@hzxuzhonghu
Copy link
Member

Will take a look today

@hzxuzhonghu
Copy link
Member

I suspect it is #45733 that introduced this regression

@hzxuzhonghu
Copy link
Member

The regression is from #46329

@vinay01tech
Copy link

vinay01tech commented Mar 6, 2024

@hzxuzhonghu @howardjohn Do we know the tentative release date ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants