Skip to content

Conversation

@hanxiaop
Copy link
Member

@hanxiaop hanxiaop commented Nov 1, 2021

In link https://istio.io/latest/docs/tasks/traffic-management/locality-load-balancing/, the topology.istio.io/subzone link does not exist.

And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.

  • Configuration Infrastructure
  • Docs
  • Installation
  • Networking
  • Performance and Scalability
  • Policies and Telemetry
  • Security
  • Test and Release
  • User Experience
  • Developer Infrastructure

@hanxiaop hanxiaop requested a review from a team as a code owner November 1, 2021 07:30
@google-cla google-cla bot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Nov 1, 2021
@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Nov 1, 2021
Copy link
Member

@Arhell Arhell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
cc @craigbox

Copy link
Contributor

@craigbox craigbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ericvn thoughts on if this should be tied to master? Either it goes stale if the file changes, or we use a release-1.12 link and it goes stale as versions increase.

@hanxiaop
Copy link
Member Author

hanxiaop commented Nov 1, 2021

@craigbox I had the same doubt at the beginning. Before using the master version, I was tried to use the permalink. I agree that both of the implementations have their own flaws, and we may discuss which one is more preferred.

@ericvn
Copy link
Contributor

ericvn commented Nov 1, 2021

Can we just use something like github_blob which takes into account the currently built branch? Since this is in api, we could create another shortcode that use the api repo. It doesn't help if the line number changes in the file across releases.

@craigbox
Copy link
Contributor

craigbox commented Nov 1, 2021

I told you @ericvn would know 🙂

@istio-policy-bot istio-policy-bot added the lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. label Jan 21, 2022
@craigbox craigbox added the lifecycle/staleproof Indicates a PR or issue has been deemed to be immune from becoming stale and/or automatically closed label Jan 23, 2022
@craigbox
Copy link
Contributor

@hanxiaop would love to see this moved forward, so have marked it to be open.

@craigbox craigbox reopened this Jan 23, 2022
@hanxiaop hanxiaop requested a review from a team as a code owner January 24, 2022 02:03
@hanxiaop hanxiaop added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Jan 24, 2022
@hanxiaop
Copy link
Member Author

@ericvn How about other links using https://github.com/istio/api/blob? After I added the lint check for github_api_blob, I realized there are lots of places using the api link right now, and some of the links will be invalid as well if I replace them with the github_api_blob form. Should they keep the original permalink form or should we do something?

@ericvn ericvn removed the lifecycle/automatically-closed Indicates a PR or issue that has been closed automatically. label Feb 14, 2022
@istio-testing
Copy link
Contributor

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
lint_istio.io d360ec3 link true /test lint_istio.io
doc.test.profile_default_istio.io d360ec3 link true /test doc.test.profile_default

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 understand the commands that are listed here.

@ericvn
Copy link
Contributor

ericvn commented May 25, 2023

Replaced by #13242.

@ericvn ericvn closed this May 25, 2023
@hanxiaop
Copy link
Member Author

Replaced by #13242.

@ericvn nice work, thanks!

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

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. kind/site infra lifecycle/staleproof Indicates a PR or issue has been deemed to be immune from becoming stale and/or automatically closed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants