-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
TestNormalization test flake #39747
Comments
To help debug istio#39747. The test, I thought, would reproduce it -- but it didn't
* Improve tests and logs around meshconfig update To help debug #39747. The test, I thought, would reproduce it -- but it didn't * fix lint
🤔 ❄️ Hey there's been no update for this test flakes for 3 days. Courtesy of your friendly test flake nag. |
It looks like the test correctly writes out the mesh config, but istiod is no longer processing. Deadlock? #39864 doesn't find anything though. There are a ton of mesh config handles, and they are synchronous, so its pretty hard to identify |
figured it out... we are using a file mount for external istiod 🤦 those are slow to update unlike configmap watcher |
This is cherrypicked from release-1.8 branch. Fixes istio#39747
https://prow.istio.io/view/gs/istio-prow/logs/integ-security-istiodremote_istio_postsubmit/1542515642890981376
https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/39688/integ-security-istiodremote_istio/1542679358819274752
issue is around meshconfig merging. We somehow end up not applying the changes. Assuming its related to e927c33, although not sure how yet
The text was updated successfully, but these errors were encountered: