Adding Kustomize structure in deployment README #562
Conversation
k8s-ci-robot
added
release-note-none
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: verult The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
|
/cc @misterikkit |
k8s-ci-robot
added
the
size/S
verult
changed the title
deploy/kubernetes/README.md
Outdated
| * Windows: includes node yaml file and related setting that is only applicable for Windows. | ||
| * `images`: it has a list of images for different versions. | ||
| * `stable`: currently only has image list for Linux stable version. | ||
| * `alpha`: besides image list in stable, it also includes Windows image updates. |
There was a problem hiding this comment.
This will become out of date once Windows is out of alpha. Should we make this more generic and say it's images used for alpha features? ditto throughout
There was a problem hiding this comment.
I think Linux and Windows support should be mentioned for each bullet because it's part of the Kustomization complexity, but I'll update it to mention it's more generally used for features in development.
| * Windows: includes node yaml file and related setting that is only applicable for Windows. | ||
| * `images`: it has a list of images for different versions. | ||
| * `stable`: image list of a stable driver release. Currently only has image list for Linux stable version. | ||
| * `alpha`: image list containing features in development, in addition to images in stable. It also includes Windows images. |
There was a problem hiding this comment.
maybe mention the reason windows images are only included here because the support for windows is still alpha?
|
/lgtm |
k8s-ci-robot
added
the
lgtm
|
/retest |
1 similar comment
|
/retest |
Based on #520 (comment)
What type of PR is this?
/kind documentation
Does this PR introduce a user-facing change?:
/assign @jingxu97 @msau42