Reviewing for approvers and reviewers

SIG Docs Reviewers and Approvers do a few extra things when reviewing a change.

Every week a specific docs approver volunteers to triage and review pull requests. This person is the “PR Wrangler” for the week. See the PR Wrangler scheduler for more information. To become a PR Wrangler, attend the weekly SIG Docs meeting and volunteer. Even if you are not on the schedule for the current week, you can still review pull requests (PRs) that are not already under active review.

In addition to the rotation, a bot assigns reviewers and approvers for the PR based on the owners for the affected files.

Reviewing a PR

Kubernetes documentation follows the Kubernetes code review process.

Everything described in Reviewing a pull request applies, but Reviewers and Approvers should also do the following:

  • Using the /assign Prow command to assign a specific reviewer to a PR as needed. This is extra important when it comes to requesting technical review from code contributors.

    Note: Look at the reviewers field in the front-matter at the top of a Markdown file to see who can provide technical review.

  • Making sure the PR follows the Content and Style guides; link the author to the relevant part of the guide(s) if it doesn’t.

  • Using the GitHub Request Changes option when applicable to suggest changes to the PR author.

  • Changing your review status in GitHub using the /approve or /lgtm Prow commands, if your suggestions are implemented.

Commit into another person’s PR

Leaving PR comments is helpful, but there might be times when you need to commit into another person’s PR instead.

Do not “take over” for another person unless they explicitly ask you to, or you want to resurrect a long-abandoned PR. While it may be faster in the short term, it deprives the person of the chance to contribute.

The process you use depends on whether you need to edit a file that is already in the scope of the PR, or a file that the PR has not yet touched.

You can’t commit into someone else’s PR if either of the following things is true:

  • If the PR author pushed their branch directly to the https://github.com/kubernetes/website/ repository. Only a reviewer with push access can commit to another user’s PR.

    Note: Encourage the author to push their branch to their fork before opening the PR next time.

  • The PR author explicitly disallows edits from approvers.

Prow commands for reviewing

Prow is the Kubernetes-based CI/CD system that runs jobs against pull requests (PRs). Prow enables chatbot-style commands to handle GitHub actions across the Kubernetes organization, like adding and removing labels, closing issues, and assigning an approver. Enter Prow commands as GitHub comments using the /<command-name> format.

The most common prow commands reviewers and approvers use are:

Prow commands for reviewing
Prow CommandRole RestrictionsDescription
/lgtmAnyone, but triggers automation if a Reviewer or Approver uses itSignals that you’ve finished reviewing a PR and are satisfied with the changes.
/approveApproversApproves a PR for merging.
/assignReviewers or ApproversAssigns a person to review or approve a PR
/closeReviewers or ApproversCloses an issue or PR.
/holdAnyoneAdds the do-not-merge/hold label, indicating the PR cannot be automatically merged.
/hold cancelAnyoneRemoves the do-not-merge/hold label.

See the Prow command reference to see the full list of commands you can use in a PR.

Triage and categorize issues

In general, SIG Docs follows the Kubernetes issue triage process and uses the same labels.

This GitHub Issue filter finds issues that might need triage.

Triaging an issue

  1. Validate the issue
  • Make sure the issue is about website documentation. Some issues can be closed quickly by answering a question or pointing the reporter to a resource. See the Support requests or code bug reports section for details.
  • Assess whether the issue has merit.
  • Add the triage/needs-information label if the issue doesn’t have enough detail to be actionable or the template is not filled out adequately.
  • Close the issue if it has both the lifecycle/stale and triage/needs-information labels.
  1. Add a priority label (the Issue Triage Guidelines define priority labels in detail)
Issue labels
LabelDescription
priority/critical-urgentDo this right now.
priority/important-soonDo this within 3 months.
priority/important-longtermDo this within 6 months.
priority/backlogDeferrable indefinitely. Do when resources are available.
priority/awaiting-more-evidencePlaceholder for a potentially good issue so it doesn’t get lost.
help or good first issueSuitable for someone with very little Kubernetes or SIG Docs experience. See Help Wanted and Good First Issue Labels for more information.

At your discretion, take ownership of an issue and submit a PR for it (especially if it’s quick or relates to work you’re already doing).

If you have questions about triaging an issue, ask in #sig-docs on Slack or the kubernetes-sig-docs mailing list.

Adding and removing issue labels

To add a label, leave a comment in one of the following formats:

  • /<label-to-add> (for example, /good-first-issue)
  • /<label-category> <label-to-add> (for example, /triage needs-information or /language ja)

To remove a label, leave a comment in one of the following formats:

  • /remove-<label-to-remove> (for example, /remove-help)
  • /remove-<label-category> <label-to-remove> (for example, /remove-triage needs-information)`

In both cases, the label must already exist. If you try to add a label that does not exist, the command is silently ignored.

For a list of all labels, see the website repository’s Labels section. Not all labels are used by SIG Docs.

Issue lifecycle labels

Issues are generally opened and closed quickly. However, sometimes an issue is inactive after its opened. Other times, an issue may need to remain open for longer than 90 days.

Issue lifecycle labels
LabelDescription
lifecycle/staleAfter 90 days with no activity, an issue is automatically labeled as stale. The issue will be automatically closed if the lifecycle is not manually reverted using the /remove-lifecycle stale command.
lifecycle/frozenAn issue with this label will not become stale after 90 days of inactivity. A user manually adds this label to issues that need to remain open for much longer than 90 days, such as those with a priority/important-longterm label.

Handling special issue types

SIG Docs encounters the following types of issues often enough to document how to handle them.

Duplicate issues

If a single problem has one or more issues open for it, combine them into a single issue. You should decide which issue to keep open (or open a new issue), then move over all relevant information and link related issues. Finally, label all other issues that describe the same problem with triage/duplicate and close them. Only having a single issue to work on reduces confusion and avoids duplicate work on the same problem.

If the dead link issue is in the API or kubectl documentation, assign them /priority critical-urgent until the problem is fully understood. Assign all other dead link issues /priority important-longterm, as they must be manually fixed.

Blog issues

We expect Kubernetes Blog entries to become outdated over time. Therefore, we only maintain blog entries less than a year old. If an issue is related to a blog entry that is more than one year old, close the issue without fixing.

Support requests or code bug reports

Some docs issues are actually issues with the underlying code, or requests for assistance when something, for example a tutorial, doesn’t work. For issues unrelated to docs, close the issue with the kind/support label and a comment directing the requester to support venues (Slack, Stack Overflow) and, if relevant, the repository to file an issue for bugs with features (kubernetes/kubernetes is a great place to start).

Sample response to a request for support:

  1. This issue sounds more like a request for support and less
  2. like an issue specifically for docs. I encourage you to bring
  3. your question to the `#kubernetes-users` channel in
  4. [Kubernetes slack](https://slack.k8s.io/). You can also search
  5. resources like
  6. [Stack Overflow](https://stackoverflow.com/questions/tagged/kubernetes)
  7. for answers to similar questions.
  8. You can also open issues for Kubernetes functionality in
  9. https://github.com/kubernetes/kubernetes.
  10. If this is a documentation issue, please re-open this issue.

Sample code bug report response:

  1. This sounds more like an issue with the code than an issue with
  2. the documentation. Please open an issue at
  3. https://github.com/kubernetes/kubernetes/issues.
  4. If this is a documentation issue, please re-open this issue.