This page assumes that you’ve read and mastered the tasks in the start contributing topic and are ready to learn about more ways to contribute.
Note: Some tasks require you to use the Git command line client and other tools.
Now that you’ve gotten your feet wet and helped out with the Kubernetes docs in the ways outlined in the start contributing topic, you may feel ready to do more. These tasks assume that you have, or are willing to gain, deeper knowledge of the following topic areas:
These tasks are not as sequential as the beginner tasks. There is no expectation that one person does all of them all of the time.
In any given week, a specific docs approver volunteers to do initial triage and review of pull requests and issues. This person is the “PR Wrangler” for the week. The schedule is maintained using the PR Wrangler scheduler. To be added to this list, 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, an automated system comments on each new PR and suggests reviewers and approvers for the PR, based on the list of approvers and reviewers in the affected files. The PR author is expected to follow the guidance of the bot, and this also helps PRs to get reviewed quickly.
We want to get pull requests (PRs) merged and published as quickly as possible. To ensure the docs are accurate and up to date, each PR needs to be reviewed by people who understand the content, as well as people with experience writing great documentation.
Reviewers and approvers need to provide actionable and constructive feedback to keep contributors engaged and help them to improve. Sometimes helping a new contributor get their PR ready to merge takes more time than just rewriting it yourself, but the project is better in the long term when we have a diversity of active participants.
Before you start reviewing PRs, make sure you are familiar with the Documentation Style Guide and the code of conduct
To see all open PRs, go to the Pull Requests tab in the Github repository. A PR is eligible for review when it meets all of the following criteria:
cncf-cla:yes
tagdo-not-merge
master
unless the PR relates to
a feature that has not yet been released)If a PR is not eligible to merge, leave a comment to let the author know about the problem and offer to help them fix it. If they’ve been informed and have not fixed the problem in several weeks or months, eventually their PR will be closed without merging.
If you’re new to reviewing, or you don’t have a lot of bandwidth, look for PRs
with the size/XS
or size/S
tag set. The size is automatically determined by
the number of lines the PR changes.
The Kubernetes website repo operates differently than some of the Kubernetes code repositories when it comes to the roles of reviewers and approvers. For more information about the responsibilities of reviewers and approvers, see Participating. Here’s an overview.
A reviewer reviews pull request content for technical accuracy. A reviewer
indicates that a PR is technically accurate by leaving a /lgtm
comment on
the PR.
Note: Don’t add an/lgtm
unless you are confident in the technical accuracy of the documentation modified or introduced in the PR.
An approver reviews pull request content for docs quality and adherence to
SIG Docs guidelines, such as the
style guide. Only people listed as
approvers in the
OWNERS
file can
approve a PR. To approve a PR, leave an /approved
comment on the PR.
A PR is merged when it has both a /lgtm
comment from anyone in the Kubernetes
organization and an /approved
comment from an approver in the
sig-docs-maintainers
group, as long as it is not on hold and the PR author
has signed the CLA.
Read the PR description and read any attached issues or links, if applicable. “Drive-by reviewing” is sometimes more harmful than helpful, so make sure you have the right knowledge to provide a meaningful review.
If someone else is the best person to review this particular PR, let them
know by adding a comment with /assign @<github-username>
. If you have
asked a non-docs person for technical review but still want to review the PR
from a docs point of view, keep going.
Go to the Files changed tab. Look over all the changed lines. Removed
content has a red background, and those lines also start with a -
symbol.
Added content has a green background, and those lines also start with a +
symbol. Within a line, the actual modified content has a slightly darker
green background than the rest of the line.
deploy/netlify
test, near the bottom of the page. It opens in
the same browser window by default, so open it in a new window so you
don’t lose your partial review. Switch back to the Files changed tab
to resume your review.+
symbol that
appears. Type your comment and click Start a review.nit:
so that the author knows
you consider it trivial. They should still address it.When you’ve reviewed everything, or if you didn’t have any comments, go back to the top of the page and click Review changes. Choose either Comment or Request Changes. Add a summary of your review, and add appropriate Prow commands to separate lines in the Review Summary field. SIG Docs follows the Kubernetes code review process. All of your comments will be sent to the PR author in a single notification.
/approve
to
your summary./lgtm
as well./assign
with the Github username of the person who needs to
provide technical review. Look at the reviewers
field in the
front-matter at the top of a given Markdown file to see who can
provide technical review./hold
. This sets the
label do-not-merge/hold
.lgtm
and approved
label but
no hold
label, it is merged automatically.If a PR has the lgtm
and/or approved
labels and new changes are
detected, these labels are removed automatically.
See the list of all available slash commands that can be used in PRs.
If you previously selected Request changes and the PR author has
addressed your concerns, you can change your review status either in the
Files changed tab or at the bottom of the Conversation tab. Be
sure to add the /approve
tag and assign technical reviewers if necessary,
so that the PR can be merged.
Leaving PR comments is helpful, but there may be times when you need to commit into another person’s PR, rather than just leaving a review.
Resist the urge to “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:
This method uses the Github UI. If you prefer, you can use the command line even if the file you want to change is part of the PR, if you are more comfortable working that way.
Your commit is now pushed to the branch the PR represents (probably on the author’s fork) and now shows up in the PR and your changes are reflected in the Files changed tab. Leave a comment letting the PR author know you changed the PR.
If the author is using the command line rather than the Github UI to work on this PR, they need to fetch their fork’s changes and rebase their local branch on the branch in their fork, before doing additional work on the PR.
If changes need to be made to a file that is not yet included in the PR, you need to use the command line. You can always use this method, if you prefer it to the Github UI.
Get the URL for the author’s fork. You can find it near the bottom of the Conversation tab. Look for the text Add more commits by pushing to. The first link after this phrase is to the branch, and the second link is to the fork. Copy the second link. Note the name of the branch for later.
Add the fork as a remote. In your terminal, go to your clone of the repository. Decide on a name to give the remote (such as the author’s Github username), and add the remote using the following syntax:
git remote add <name> <url-of-fork>
Fetch the remote. This doesn’t change any local files, but updates your clone’s notion of the remote’s objects (such as branches and tags) and their current state.
git remote fetch <name>
Check out the remote branch. This command will fail if you already have a local branch with the same name.
git checkout <branch-from-PR>
Make your changes, use git add
to add them, and commit them.
Push your changes to the author’s remote.
git push <remote-name> <branch-name>
Go back to the Github IU and refresh the PR. Your changes appear. Leave the PR author a comment letting them know you changed the PR.
If the author is using the command line rather than the Github UI to work on this PR, they need to fetch their fork’s changes and rebase their local branch on the branch in their fork, before doing additional work on the PR.
For changes that require multiple files or changes that involve creating new
files or moving files around, working from a local Git clone makes more sense
than relying on the Github UI. These instructions use the git
command and
assume that you have it installed locally. You can adapt them to use a local
graphical Git client instead.
You only need to clone the repository once per physical system where you work on the Kubernetes documentation.
In a terminal window, use git clone
to clone the repository. You do not
need any credentials to clone the repository.
git clone https://github.com/kubernetes/website
The new directory website
is created in your current directory, with
the contents of the Github repository.
Change to the new website
directory. Rename the default origin
remote
to upstream
.
cd website
git remote rename origin upstream
If you have not done so, create a fork of the repository on Github. In your
web browser, go to
https://github.com/kubernetes/website
and click the Fork button. After a few seconds, you are redirected to
the URL for your fork, which is typically something like
https://github.com/<username>/website
unless you already had a repository
called website
. Copy this URL.
Add your fork as a second remote, called origin
:
git remote add origin <FORK-URL>
Before you start a new unit of work on your local repository, you need to figure out which branch to base your work on. The answer depends on what you are doing, but the following guidelines apply:
master
.master
.For more guidance, see Choose which branch to use.
After you decide which branch to start your work (or base it on, in Git terminology), use the following workflow to be sure your work is based on the most up-to-date version of that branch.
Fetch both the upstream
and origin
branches. This updates your local
notion of what those branches contain, but does not change your local
branches at all.
git fetch upstream
git fetch origin
Create a new tracking branch based on the branch you decided is the most
appropriate. This example assumes you are using master
.
git checkout -b <my_new_branch> upstream/master
This new branch is based on upstream/master
, not your local master
.
It tracks upstream/master
.
With your new branch checked out, make your changes using a text editor.
At any time, use the git status
command to see what you’ve changed.
When you are ready to submit a pull request, commit your changes. First
use git status
to see what changes need to be added to the changeset.
There are two important sections: Changes staged for commit
and
Changes not staged for commit
. Any files that show up in the latter
section under modified
or untracked
need to be added if you want them to
be part of this commit. For each file that needs to be added, use git add
.
git add example-file.md
When all your intended changes are included, create a commit, using the
git commit
command:
git commit -m "Your commit message"
Note:Do not reference a Github issue or pull request by ID or URL in the commit message. If you do, it will cause that issue or pull request to get a notification every time the commit shows up in a new Git branch. You can link issues and pull requests together later, in the Github UI.
Optionally, you can test your change by staging the site locally using the
hugo
command. See View your changes locally.
You’ll be able to view your changes after you submit the pull request, as
well.
Before you can create a pull request which includes your local commit, you
need to push the branch to your fork, which is the endpoint for the origin
remote.
git push origin <my_new_branch>
Technically, you can omit the branch name from the push
command, but
the behavior in that case depends upon the version of Git you are using.
The results are more repeatable if you include the branch name.
At this point, if you go to https://github.com/kubernetes/website in your web browser, Github detects that you pushed a new branch to your fork and offers to create a pull request. Fill in the pull request template.
Fixes #12345
if the pull request fixes a Github
issue. This will cause the issue to be closed automatically when the
pull request is merged.Click Create pull request.
Several automated tests will run against the state of the website with your changes applied. If any of the tests fail, click the Details link for more information. If the Netlify test completes successfully, its Details link goes to a staged version of the Kubernetes website with your changes applied. This is how reviewers will check your changes.
If you notice that more changes need to be made, or if reviewers give you feedback, address the feedback locally, then repeat step 4 - 6 again, creating a new commit. The new commit is added to your pull request and the tests run again, including re-staging the Netlify staged site.
If a reviewer adds changes to your pull request, you need to fetch those changes from your fork before you can add more changes. Use the following commands to do this, assuming that your branch is currently checked out.
git fetch origin
git rebase origin/<your-branch-name>
After rebasing, you need to add the -f
flag to force-push new changes to
the branch to your fork.
git push -f origin <your-branch-name>
If someone else’s change is merged into the branch your work is based on, and you have made changes to the same parts of the same files, a conflict might occur. If the pull request shows that there are conflicts to resolve, you can resolve them using the Github UI or you can resolve them locally.
First, do step 10 to be sure that your fork and your local branch are in the same state.
Next, fetch upstream
and rebase your branch on the branch it was
originally based on, like upstream/master
.
git fetch upstream
git rebase upstream/master
If there are conflicts Git can’t automatically resolve, you can see the
conflicted files using the git status
command. For each conflicted file,
edit it and look for the conflict markers >>>
, <<<
, and ===
. Resolve
the conflict and remove the conflict markers. Then add the changes to the
changeset using git add <filename>
and continue the rebase using
git rebase --continue
. When all commits have been applied and there are
no more conflicts, git status
will show that you are not in a rebase and
there are no changes that need to be committed. At that point, force-push
the branch to your fork, and the pull request should no longer show any
conflicts.
If you’re having trouble resolving conflicts or you get stuck with
anything else related to your pull request, ask for help on the #sig-docs
Slack channel or the
kubernetes-sig-docs mailing list.
If you aren’t ready to create a pull request but you want to see what your changes look like, you can build and run a docker image to generate all the documentation and serve it locally.
Build the image locally:
make docker-image
Once the kubernetes-hugo
image has been built locally, you can build and serve the site:
make docker-serve
In your browser’s address bar, enter localhost:1313
. Hugo will watch the
filesystem for changes and rebuild the site as needed.
To stop the local Hugo instance, go back to the terminal and type Ctrl+C
or just close the terminal window.
Alternatively, you can install and use the hugo
command on your development machine:
Install Hugo version or later.
In a terminal, go to the root directory of your clone of the Kubernetes docs, and enter this command:
hugo server
In your browser’s address bar, enter localhost:1313
.
To stop the local Hugo instance, go back to the terminal and type Ctrl+C
or just close the terminal window.
In any given week, a specific docs approver volunteers to do initial triage and review of pull requests and issues. To get on this list, attend the weekly SIG Docs meeting and volunteer. Even if you are not on the schedule for the current week, you can still review PRs.
People in SIG Docs are responsible only for triaging and categorizing
documentation issues. General website issues are also filed in the
kubernetes/website
repository.
When you triage an issue, you:
If you have questions about triaging an issue, ask in #sig-docs
on Slack or
the
kubernetes-sig-docs mailing list.
These guidelines are not set in stone and are subject to change.
sig/
labels exist, such as sig/cli
and
sig/api-machinery
.Actionable
: There seems to be enough information for the issue to be fixed
or acted upon.good first issue
: Someone with limited Kubernetes or SIG Docs experience
might be able to tackle this issue.kind/bug
, kind/feature
, and kind/documentation
: If the person who
filed the issue did not fill out the template correctly, these labels may
not be assigned automatically. A bug is a problem with existing content or
functionality, and a feature is a request for new content or functionality.
The kind/documentation
label is not currently in use.P1
, P2
, or P3
, if set./label <label-to-add>
. The label must
already exist. If you try to add a label that does not exist, the command is
silently ignored.An issue’s priority influences how quickly it is addressed. For documentation, here are the guidelines for setting a priority on an issue:
This is the default for new issues and pull requests.
We’ve encountered the following types of issues often enough to document how to handle them.
If a single problem has one or more issues open for it, the problem should be consolidated into a single issue. You should decide which issue to keep open (or open a new issue), port over all relevant information, link related issues, and close all the other issues that describe the same problem. Only having a single issue to work on will help reduce confusion and avoid duplicating work on the same problem.
Depending on where the dead link is reported, different actions are required to resolve the issue. Dead links in the API and Kubectl docs are automation issues and should be assigned a P1 until the problem can be fully understood. All other dead links are issues that need to be manually fixed and can be assigned a P3.
Kubernetes Blog entries are expected to become outdated over time, so we maintain only blog entries that are less than one year old. If an issue is related to a blog entry that is more than one year old, it should be closed without fixing.
Some issues opened for docs are instead issues with the underlying code, or requests for assistance when something (like a tutorial) didn’t work. For issues unrelated to docs, close the issue with a comment directing the requester to support venues (Slack, Stack Overflow) and, if relevant, where to file an issue for bugs with features (kubernetes/kubernetes is a great place to start).
Sample response to a request for support:
This issue sounds more like a request for support and less
like an issue specifically for docs. I encourage you to bring
your question to the `#kubernetes-users` channel in
[Kubernetes slack](http://slack.k8s.io/). You can also search
resources like
[Stack Overflow](http://stackoverflow.com/questions/tagged/kubernetes)
for answers to similar questions.
You can also open issues for Kubernetes functionality in
https://github.com/kubernetes/kubernetes.
If this is a documentation issue, please re-open this issue.
Sample code bug report response:
This sounds more like an issue with the code than an issue with
the documentation. Please open an issue at
https://github.com/kubernetes/kubernetes/issues.
If this is a documentation issue, please re-open this issue.
Each major Kubernetes release includes new features, and many of them need at least a small amount of documentation to show people how to use them.
Often, the SIG responsible for a feature submits draft documentation for the
feature as a pull request to the appropriate release branch of
kubernetes/website
repository, and someone on the SIG Docs team provides
editorial feedback or edits the draft directly.
To find out about upcoming features, attend the weekly sig-release meeting (see the community page for upcoming meetings) and monitor the release-specific documentation in the kubernetes/sig-release repository. Each release has a sub-directory under the /sig-release/tree/master/releases/ directory. Each sub-directory contains a release schedule, a draft of the release notes, and a document listing each person on the release team.
The release schedule contains links to all other documents, meetings, meeting minutes, and milestones relating to the release. It also contains information about the goals and timeline of the release, and any special processes in place for this release. Near the bottom of the document, several release-related terms are defined.
This document also contains a link to the Feature tracking sheet, which is the official way to find out about all new features scheduled to go into the release.
The release team document lists who is responsible for each release role. If it’s not clear who to talk to about a specific feature or question you have, either attend the release meeting to ask your question, or contact the release lead so that they can redirect you.
The release notes draft is a good place to find out a little more about specific features, changes, deprecations, and more about the release. The content is not finalized until late in the release cycle, so use caution.
The feature tracking sheet for a given Kubernetes release lists each feature that is planned for a release. Each line item includes the name of the feature, a link to the feature’s main Github issue, its stability level (Alpha, Beta, or Stable), the SIG and individual responsible for implementing it, whether it needs docs, a draft release note for the feature, and whether it has been merged. Keep the following in mind:
As stated above, draft content for new features is usually submitted by the SIG responsible for implementing the new feature. This means that your role may be more of a shepherding role for a given feature than developing the documentation from scratch.
After you’ve chosen a feature to document/shepherd, ask about it in the #sig-docs
Slack channel, in a weekly sig-docs meeting, or directly on the PR filed by the
feature SIG. If you’re given the go-ahead, you can edit into the PR using one of
the techniques described in
Commit into another person’s PR.
If you need to write a new topic, the following links are useful:
If you are a member of a SIG developing a new feature for Kubernetes, you need to work with SIG Docs to be sure your feature is documented in time for the release. Check the feature tracking spreadsheet or check in the #sig-release Slack channel to verify scheduling details and deadlines. Some deadlines related to documentation are:
release-X.Y
branch in the kubernetes/website
repository, with a small
commit that you will amend later. Use the Prow command /milestone X.Y
to
assign the PR to the relevant milestone. This alerts the docs person managing
this release that the feature docs are coming. If your feature does not need
any documentation changes, make sure the sig-release team knows this, by
mentioning it in the #sig-release Slack channel. If the feature does need
documentation but the PR is not created, the feature may be removed from the
milestone.release-X.Y
branch by this deadline, work with the
docs person managing the release to get it in. If your feature needs
documentation and the docs are not ready, the feature may be removed from the
milestone.If your feature is an Alpha feature and is behind a feature gate, make sure you add it to Feature gates as part of your pull request. If your feature is moving out of Alpha, make sure to remove it from that file.
The Kubernetes project contains more than 50 individual repositories. Many of these repositories contain code or content that can be considered documentation, such as user-facing help text, error messages, user-facing text in API references, or even code comments.
If you see text and you aren’t sure where it comes from, you can use Github’s search tool at the level of the Kubernetes organization to search through all repositories for that text. This can help you figure out where to submit your issue or PR.
Each repository may have its own processes and procedures. Before you file an
issue or submit a PR, read that repository’s README.md
, CONTRIBUTING.md
, and
code-of-conduct.md
, if they exist.
Most repositories use issue and PR templates. Have a look through some open issues and PRs to get a feel for that team’s processes. Make sure to fill out the templates with as much detail as possible when you file issues or PRs.
The Kubernetes documentation is written in English first, but we want people to
be able to read it in their language of choice. If you are comfortable
writing in another language, especially in the software domain, you can help
localize the Kubernetes documentation or provide feedback on existing localized
content. See Localization and ask on the
kubernetes-sig-docs mailing list
or in #sig-docs
on Slack if you are interested in helping out.
Follow these guidelines for working with localized content:
Each language has its own reviewers and approvers.
If a PR contains changes to source in more than one language, ask the PR contributor to open separate PRs for each language.
When you are comfortable with all of the tasks discussed in this topic and you want to engage with the Kubernetes docs team in even deeper ways, read the advanced docs contributor topic.
Was this page helpful?
Thanks for the feedback. If you have a specific, answerable question about how to use Kubernetes, ask it on Stack Overflow. Open an issue in the GitHub repo if you want to report a problem or suggest an improvement.