Skip to content
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

dead links in k8s.io/docs/reference/setup-tools/kubeadm/kubeadm-join/ #16998

Closed
smirnov-mi opened this issue Oct 17, 2019 · 11 comments
Closed

dead links in k8s.io/docs/reference/setup-tools/kubeadm/kubeadm-join/ #16998

smirnov-mi opened this issue Oct 17, 2019 · 11 comments
Assignees
Labels
kind/bug Categorizes issue or PR as related to a bug. sig/docs Categorizes an issue or PR as relevant to SIG Docs.

Comments

@smirnov-mi
Copy link

This is a Bug Report

Problem:
Dead links on the "kubeadm join" page:

kubeadm - kubeadm: easily bootstrap a secure Kubernetes cluster
kubeadm join phase - Use this command to invoke single phase of the join workflow

Proposed Solution:
fix them

@neolit123
Copy link
Member

/help
/sig cluster-lifecycle
/kind bug

thanks for logging the issue

@k8s-ci-robot k8s-ci-robot added sig/cluster-lifecycle Categorizes an issue or PR as relevant to SIG Cluster Lifecycle. kind/bug Categorizes issue or PR as related to a bug. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. labels Oct 17, 2019
@neolit123
Copy link
Member

/assign

@neolit123
Copy link
Member

/remove-help

@k8s-ci-robot k8s-ci-robot removed the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Oct 17, 2019
@neolit123
Copy link
Member

neolit123 commented Oct 17, 2019

@k8s-ci-robot k8s-ci-robot added sig/docs Categorizes an issue or PR as relevant to SIG Docs. and removed sig/cluster-lifecycle Categorizes an issue or PR as relevant to SIG Cluster Lifecycle. labels Oct 17, 2019
@k8s-ci-robot
Copy link
Contributor

@neolit123: Those labels are not set on the issue: sig/cluster-lifecycle

In response to this:

/unassign
/remove-sig cluster-lifecycle
/sig docs

/assign @tengqm @jimangel

it seems this generated file contains invalid links at the bottom under "see also":
https://github.com/kubernetes/website/blob/master/content/en/docs/reference/setup-tools/kubeadm/generated/kubeadm_join.md

preview:
https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-join/#options-inherited-from-parent-commands

EDIT: looks like the same problem happens for other generated files such as:
https://github.com/kubernetes/website/blob/master/content/en/docs/reference/setup-tools/kubeadm/generated/kubeadm_init.md

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.

@aimeeu
Copy link
Contributor

aimeeu commented Oct 17, 2019

The links in the markdown files point to other markdown files in github.

In https://github.com/kubernetes/website/blob/master/content/en/docs/reference/setup-tools/kubeadm/generated/kubeadm_join.md, L224-225:

* [kubeadm](kubeadm.md)	 - kubeadm: easily bootstrap a secure Kubernetes cluster
* [kubeadm join phase](kubeadm_join_phase.md)	 - Use this command to invoke single phase of the join workflow

The links should point to GitHub files.

But in https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-join/#options-inherited-from-parent-commands:

<p>SEE ALSO</p>
<ul>
<li><a href=kubeadm.md target=_blank>kubeadm</a> - kubeadm: easily bootstrap a secure Kubernetes cluster</li>
<li><a href=kubeadm_join_phase.md target=_blank>kubeadm join phase</a> - Use this command to invoke single phase of the join workflow</li>
</ul>

So Hugo is translating the link to be relative to kubernetes.io. Is the fix for this to change [kubeadm](kubeadm.md) to [kubeadm](https://github.com/kubernetes/website/blob/master/content/en/docs/reference/setup-tools/kubeadm/generated/kubeadm.md) in the upstream kubeadm docs?

@aimeeu
Copy link
Contributor

aimeeu commented Oct 21, 2019

@tengqm, @jimangel any thoughts on how to fix this issue?

@kbhawkey
Copy link
Contributor

@neolit123 , Since there are static links in the What's next section on each kubeadm command page, is the generated See Also section needed?

@neolit123
Copy link
Member

neolit123 commented Oct 21, 2019 via email

@kbhawkey
Copy link
Contributor

kbhawkey commented Oct 21, 2019

I don't think it's needed. Ideally we should remove it.

OK. Then the link errors would be resolved, correct?
If no one is working on this, feel free to assign to me.

@tengqm tengqm assigned kbhawkey and unassigned tengqm Oct 22, 2019
@tengqm
Copy link
Contributor

tengqm commented Oct 22, 2019

I'm trapped by some company projects recently. Would try allocate some time for code review though.

kbhawkey added a commit to kbhawkey/website that referenced this issue Oct 22, 2019
- For further information, see issue kubernetes#16998.
- Regenerated files using,
  kubernetes/website/update-imported-docs/update-imported-docs
k8s-ci-robot pushed a commit that referenced this issue Oct 24, 2019
- For further information, see issue #16998.
- Regenerated files using,
  kubernetes/website/update-imported-docs/update-imported-docs
@jimangel jimangel closed this as completed Nov 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. sig/docs Categorizes an issue or PR as relevant to SIG Docs.
Projects
None yet
Development

No branches or pull requests

7 participants