Eric Van Norman

Results 210 comments of Eric Van Norman

Also, note that you can click on the `Details` to the right of the deploy-preview job to get at the docs corresponding to this change. https://deploy-preview-13779--preliminary-istio.netlify.app/latest/docs/tasks/observability/distributed-tracing/lightstep/ would be the new...

Lint error is Running linkinator... [404] http://localhost:5080/docs/tasks/observability/distributed-tracing/lightstep/ ERROR: Detected 1 broken links. Scanned 5787 links in 39.311 seconds. You removed this file so makes sense that the link isn't found....

Also added the cherry-pick label since we have cut the new branch.

@robswhitmore Daniel is correct. There are two ways to fix the issue. One is to update the link which is pointing to the file you removed in `news/releases/1.1.x/announcing-1.1/change-notes/` to point...

Does https://github.com/istio/istio.io/pull/13996 replace this?

> Looks like the eng team decided to focus on Tracing, so that new PR is separate from this one, and is taking precedence. Sorry for all the confusion! I...

Would you point to where this is discussed. Is this maybe related to this https://github.com/istio/istio/issues/43785? I believe there was an issue upgrading to 1.17.0 and 1.17.1 but it was fixed...

I don't think we test a lot of external links (disabled long long ago), and this could be one of them. Would need to investigate more. edit: I don't see...

This happened a long time ago and I believe the issue at the time was flaky links causing the test to fail and that holding out PRs. There may have...