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

retryRemoteLocalities uses xds v2 API, broken on recent versions #33737

Closed
howardjohn opened this issue Jun 30, 2021 · 2 comments
Closed

retryRemoteLocalities uses xds v2 API, broken on recent versions #33737

howardjohn opened this issue Jun 30, 2021 · 2 comments

Comments

@howardjohn
Copy link
Member

apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: ratings-route
spec:
  hosts:
  - echo.default.svc.cluster.local
  http:
  - route:
    - destination:
        host: echo.default.svc.cluster.local
    retries:
      attempts: 3
      perTryTimeout: 2s
      retryOn: gateway-error,connect-failure,refused-stream
      retryRemoteLocalities: true
2021-06-30T15:23:13.908540Z     warning envoy config    gRPC config for type.googleapis.com/envoy.config.route.v3.RouteConfiguration rejected: The v2 xDS major version is deprecated and disabled by default. Support for v2 will be removed from Envoy at the start of Q1 2021. You may make use of v2 in Q4 2020 by following the advice in https://www.envoyproxy.io/docs/envoy/latest/faq/api/transition. (envoy.config.retry.previous_priorities.PreviousPrioritiesConfig)

The fix is easy but this indicates a clear testing gap

@pavanponugoti
Copy link

We have encountered this issue during Istio upgrade from 1.9.5 to 1.10.2. After removing retryRemoteLocalities from virtual services, ILBs started working.
Also, this link doesn't seem to exist: https://www.envoyproxy.io/docs/envoy/latest/faq/api/transition
Internal:The v2 xDS major version is deprecated and disabled by default. Support for v2 will be removed from Envoy at the start of Q1 2021. You may make use of v2 in Q4 2020 by following the advice in https://www.envoyproxy.io/docs/envoy/latest/faq/api/transition. (envoy.config.retry.previous_priorities.PreviousPrioritiesConfig)

@ramaraochavali
Copy link
Contributor

Closing this as it is fixed

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

No branches or pull requests

3 participants