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

WIP: Windows doc set for v1.13 stable #10875

Closed
wants to merge 8 commits into from

Conversation

PatrickLang
Copy link
Contributor

Windows Server support is proposed as a stable feature (kubernetes/enhancements#116) for v1.13.

This is a documentation overhaul that will include typical steps and use-case based examples for both application developers and cluster operators. The old content was moved into another file, and most of that will be removed to link to existing cluster setup steps instead.

/milestone 1.13
/hold

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 2, 2018
@k8sio-netlify-preview-bot
Copy link
Collaborator

Deploy preview for kubernetes-io-master-staging ready!

Built with commit cc29ef9

https://deploy-preview-10875--kubernetes-io-master-staging.netlify.com

@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Nov 2, 2018

Deploy preview for kubernetes-io-master-staging failed.

Built with commit cddd543

https://app.netlify.com/sites/kubernetes-io-master-staging/deploys/5bdcd2ac4ed62f495ba48fbe

@PatrickLang
Copy link
Contributor Author

I just rebased my WIP branch off dev-1.13. This should avoid merge conflicts.

@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
  • If you have done the above and are still having issues with the CLA being reported as unsigned, please email the CNCF helpdesk: [email protected]

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. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 7, 2018
@tfogo tfogo removed this from the 1.13 milestone Nov 19, 2018
@tfogo
Copy link
Contributor

tfogo commented Nov 19, 2018

Pushed to 1.14
/assign @jimangel

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Nov 19, 2018
@tfogo tfogo changed the base branch from dev-1.13 to master November 19, 2018 22:04
@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 19, 2018
@tfogo
Copy link
Contributor

tfogo commented Nov 19, 2018

Changed base to master so it doesn't clutter my base:dev-1.13 searched

@bgrant0607
Copy link
Member

It looks like maybe other changes were accidentally pulled into this.

@PatrickLang Do you plan to update this PR or just create a new one?

@PatrickLang
Copy link
Contributor Author

When this was moved out of v1.13, someone changed the merge target to master which broke a lot of stuff.

@PatrickLang
Copy link
Contributor Author

@bgrant0607 @spiffxp - how does the release team track doc completion? We should follow whatever that process is for v1.14

Patrick Lang and others added 8 commits January 28, 2019 15:32
Based on https://kubernetes.io/docs/tasks/inject-data-application/distribute-credentials-secure/#create-a-pod-that-has-access-to-the-secret-data-through-environment-variables

TODO: figure out how to make the link https://k8s.io/docs/getting-started-guides/windows/secret-pod.yaml work

- Used same secrets as standard instructions
- Used environment variables to inject secrets
- Updated windows/secret-pod.yaml to match new instructions
- Because pod needs to keep running, add a ping -t localhost so that the
pod is never "Completed". Linux uses nginx image, which has a default
command.

Signed-off-by: Ben Moss <[email protected]>
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jan 28, 2019
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: jimangel

If they are not already assigned, you can assign the PR to them by writing /assign @jimangel in a comment when ready.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the language/en Issues or PRs related to English language label Jan 28, 2019
@PatrickLang
Copy link
Contributor Author

@craiglpeters @michmike - I rebased this so you can read my old proposed outline.

We're not sure if we're going to keep this PR or have someone else coordinate the docs. We'll discuss in SIG-Windows meeting tomorrow

@bgrant0607
Copy link
Member

@PatrickLang I'm not sure what the current process is, but believe it's documented here:
https://github.com/kubernetes/sig-release/blob/master/release-team/role-handbooks/docs/README.md

I suggest asking on the SIG Docs slack channel or mailing list.

@jimangel
Copy link
Member

@bgrant0607 @spiffxp - how does the release team track doc completion? We should follow whatever that process is for v1.14

@PatrickLang I have updated docs deadlines in k/sig-release. Essentially you will need to rebase this against dev-1.14 unless you want to scrap it in favor of another PR. Let me know if I can help! Thanks!

@zparnold zparnold changed the base branch from master to dev-1.14 February 18, 2019 02:20
@PatrickLang
Copy link
Contributor Author

/close
Craig has an updated outline and is working with the SIG to get the content finished. #12426 is the new tracking issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. language/en Issues or PRs related to English language size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants