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

envoyproxy instance svc update error after gateway restart #1340

Closed
spwangxp opened this issue Apr 23, 2023 · 2 comments · Fixed by #1342
Closed

envoyproxy instance svc update error after gateway restart #1340

spwangxp opened this issue Apr 23, 2023 · 2 comments · Fixed by #1342
Labels
kind/bug Something isn't working

Comments

@spwangxp
Copy link
Contributor

Description:

What issue is being seen?
after gateway restart, gateway will update the exist envoy proxy instance, so there will encounter an error, seem like https://github.com/envoyproxy/gateway/blob/main/internal/infrastructure/kubernetes/infra.go#L74 miss add the original info to new svc.

Repro steps:

apply an gateway cr and envoyproxy cr, check an envoy proxy instance whether created in envoy-gateway-system, then restart the gateway.

Environment:

the main branch code;
k8s server: v1.19.8

Logs:

2023-04-23T11:00:59.473+0800 ERROR runner/runner.go:68 failed to create new infra {"runner": "infrastructure", "error": "failed to update service envoy-gateway-system/envoy-default-eg-64656661: Service "envoy-default-eg-64656661" is invalid: spec.clusterIP: Invalid value: "": field is immutable"}
github.com/envoyproxy/gateway/internal/infrastructure/runner.(*Runner).subscribeToProxyInfraIR.func1
gateway/internal/infrastructure/runner/runner.go:68
gateway/internal/message.HandleSubscription[...]
gateway/internal/message/watchutil.go:28
github.com/envoyproxy/gateway/internal/infrastructure/runner.(*Runner).subscribeToProxyInfraIR
gateway/internal/infrastructure/runner/runner.go:57

@spwangxp spwangxp added the kind/bug Something isn't working label Apr 23, 2023
@spwangxp
Copy link
Contributor Author

spwangxp commented Apr 23, 2023

this if always be true due to svc.spec.ClusterIP/svc.spec.CheckNodePort/svc.spec.*.nodePort always different.

@zirain
Copy link
Contributor

zirain commented Apr 23, 2023

cc @arkodg , this seems a issue from the begining.

spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 23, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 23, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 23, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 23, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 24, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 24, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 26, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 27, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue Apr 28, 2023
spwangxp pushed a commit to spwangxp/envoy-gateway that referenced this issue May 4, 2023
arkodg added a commit that referenced this issue May 5, 2023
* bugfix #1340: not compare all svc.spec for user modified scene

Signed-off-by: spwangxp <[email protected]>

* lint code(internal/infrastructure/kubernetes)

Signed-off-by: spwangxp <[email protected]>

* bugfix #1340: not compare all svc.spec for user modified scene

Signed-off-by: spwangxp <[email protected]>

* bugfix #1340: not compare all svc.spec for user modified scene

Signed-off-by: spwangxp <[email protected]>

* update comment for CompareSvc

Signed-off-by: spwangxp <[email protected]>

---------

Signed-off-by: spwangxp <[email protected]>
Co-authored-by: spwangxp <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
Co-authored-by: Arko Dasgupta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants