-
Notifications
You must be signed in to change notification settings - Fork 89
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
enabling virtualservice status breaks update tests #585
Comments
I created an upstream issue: istio/istio#31931 |
This issue is stale because it has been open for 90 days with no |
/lifecycle frozen |
* remove script * Disable enable-virtualservice-status on istio-head test Istio head test keeps failing as https://testgrid.k8s.io/r/knative-own-testgrid/serving#istio-head-no-mesh This is most probably due to `enable-virtualservice-status`. This patch changes: - to disables enable-virtualservice-status until #585 was fixed. - to update Istio version to Stable 1.11.4 and Latest 1.12.0. * Drop extra template * Drop config-istio.yaml from istio-head
/good-first-issue We should remove this functionality from net-istio - and the flag |
@dprotaso: Please ensure the request meets the requirements listed here. If this request no longer meets these requirements, the label can be removed In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
I've observed the virtual service
status.observedGeneration
doesn't change so the net-istio reconciler thinks it's stale and doesn't update. I'm going to disable this flag for the latests tests for nowSo because of this we never really ran e2e tests with the virtual status enabled - thus I think this is a legit bug.
Originally posted by @dprotaso in #583 (comment)
The text was updated successfully, but these errors were encountered: