docs: remove outdated config-status references#3662
docs: remove outdated config-status references#3662nagendrareddy10 wants to merge 1 commit intoistio:masterfrom
Conversation
|
😊 Welcome @nagendrareddy10! This is either your first contribution to the Istio api repo, or it's been You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines Thanks for contributing! Courtesy of your friendly welcome wagon. |
istio-testing
added
size/XS
|
Hi @nagendrareddy10. Thanks for your PR. I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with Tip We noticed you've done this a few times! Consider joining the org to skip this step and gain Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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-sigs/prow repository. |
|
/ok-to-test |
istio-testing
added
ok-to-test
|
|
||
| message IstioStatus { | ||
| // Current service state of the resource. | ||
| // More info: https://istio.io/docs/reference/config/config-status/ |
There was a problem hiding this comment.
you are trying to get rid of the lint failure on Istio.io alone? is this section overall deprecated and removed? cc @ramaraochavali @keithmattix
There was a problem hiding this comment.
@kfaseela No, it's not just to get rid of the lint failure. The config_distribution_tracking feature was completely removed from the Istio core in istio/istio#51866.
Because of that, the related documentation page is being deleted in istio/istio.io#17161, and I am removing the dead links here so the istio.io site lint validation passes.
There was a problem hiding this comment.
@nagendrareddy10 : My question is whether IstioStatus is completely removed as part of config_distribution_tracking , if so - why are you removing only the dead link ?
There was a problem hiding this comment.
@kfaseela IstioStatus is not completely removed because it's still actively used by other core features like Configuration Analysis. I am only removing the dead links to the deleted config-status page so the istio.io site lint validation passes.
nagendrareddy10
added
the
release-notes-none
|
/test release-notes |
4 participants
What this PR does:
Removes the outdated "Configuration Status Field" documentation references (
/docs/reference/config/config-status/) as part of the documentation cleanup in istio/istio.io.This PR complements istio/istio.io#17161 to allow
linkinatorchecks to pass.