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.

Sign up for GitHub

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

Jump to bottom

[WIP] start serverside-apply documentation #12898

Closed
wants to merge 145 commits into from
Closed

[WIP] start serverside-apply documentation #12898

wants to merge 145 commits into from

Conversation

kwiesmueller
Copy link
Member

Add documentation for the serverside-apply alpha feature

/milestone 1.14

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 28, 2019
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Feb 28, 2019 •
edited
Loading

Deploy preview for kubernetes-io-vnext-staging processing.

Building with commit b7f174b

https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/5c77ef3685378d0008ade902

@k8s-ci-robot k8s-ci-robot requested review from aanm and ahmetb February 28, 2019 12:54
@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. language/en Issues or PRs related to English language language/ko Issues or PRs related to Korean language language/zh Issues or PRs related to Chinese language labels Feb 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: stewart-yu

If they are not already assigned, you can assign the PR to them by writing /assign @stewart-yu 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 cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 28, 2019
@kwiesmueller
Copy link
Member Author

/milestone v1.14

@kwiesmueller
Copy link
Member Author

/cc @apelisse @jennybuckley @lavalamp

@kbhawkey
Copy link
Contributor

kbhawkey commented Feb 28, 2019 •
edited
Loading

@kwiesmueller, you intend to add one commit, right?

@kwiesmueller
Copy link
Member Author

Yeah, I added one to at least have the PR open.
I guess when switching to the dev-1.14 branch i failed to rebase first. Let me check.

Himanshu Pandey and others added 13 commits February 28, 2019 15:23
Updated Recommended docker version (kubernetes#12591)
2331f93 
* Updated Recommended docker version

* minor changes
@coderanger
First pass on a blog post about CVE-2019-5736. (kubernetes#12592)
9df0042 
* First pass on a blog post about CVE-2019-5736.

* Clarify that k8s is not the problem.

* Fix example of pinning to an image hash.

My mistake deftly caught by @zparnold.

Co-Authored-By: coderanger <[email protected]>

* Add links to the rest of the notices or releases.

* Add ways to get more info.

* RHEL link for those that don't selinux.

* Link to Rancher's back ports.
@VineethReddy02
Update krib.md (kubernetes#12580)
7f44d68 
Fixed a broken link to kubespray in the following page: https://kubernetes.io/docs/setup/on-premises-metal/krib/
@php-coder
pod-security-policy.md: fix a broken link. (kubernetes#12590)
65a1ce9
Fixed a minor space issue (kubernetes#12576)
226a5ac
@neomantra
fix typo in install-kubeadm.md (kubernetes#12527)
00231a7 
"Kubernetes control panel" instead of "Kubernetes control plane"
@danielbryantuk
Adding proposed blog post for building-a-kubernetes-edge-control-plane (
a8f86da 
kubernetes#12404)

* Adding proposed blog post for building-a-kubernetes-edge-control-plane-for-envoy-v2

* Update article with improvements and author list

* Update and rename building-a-kubernetes-edge-control-plane-for-envoy-v2.md to 2019-02-11-building-a-kubernetes-edge-control-plane-for-envoy-v2.md

* Add extra coverage about use of Golang in Ambassador 0.50

* Minor typo fixes

* Minor typo fix, and merge fix with date

* Update 2019-02-11-building-a-kubernetes-edge-control-plane-for-envoy-v2.md

* Delete building-a-kubernetes-edge-control-plane-for-envoy-v2.md

* Update 2019-02-11-building-a-kubernetes-edge-control-plane-for-envoy-v2.md
@seokho-son
Fix a misleading word in containers/images.md (kubernetes#12553)
762180a 
* Fix a misleading word in containers/images.md

* Rev Fix a misleading word in containers/images.md
@stacyped
Add ICP to Pick Right Solutions table (kubernetes#12604)
d2641f0
@coderanger
Some corrections identified by @tallclair. (kubernetes#12605)
988641a
@stacyped
Updating ICP page with new AWS quick start and latest links (kubernet…
1b2a795 
…es#12606)
@sftim
Fix capitalization (kubernetes#12613)
09539fd
@goostleek
Add option to install kubectl on Windows via scoop (kubernetes#12359)
6a5ec8b 
* Add option to install kubectl via scoop

* Review suggestions

* Fixed tab rendering

* Fixed list numbering
jygastaud and others added 20 commits February 28, 2019 15:23
@jygastaud
Allow translation of kubeweekly block (kubernetes#12873)
85ccdb3
@DanyC97
Add the meeting calendar to the contrib docs (kubernetes#12855)
df6edd3
@daminisatya
Fix kubernetes#12764 - Update troubleshooting-kubeadm doc (kubernetes…
7f05938 
…#12825)
@sftim
Adjust for Minikube running RBAC by default (kubernetes#12851)
000a8ee
@DanyC97
Fix the borken link in kubelet-integration.md page (kubernetes#12864)
4e4a529
@cody-clark
Fix Ingress callouts, links, nav, and templating (kubernetes#12720)
884d94a 
* Fix Ingress callouts, links, nav, and templating

* Update ingress.md

* Update ingress-minikube.md

* Delete .md that caused 404

* Delete .md that caused 404

* Update ingress-controllers.md

* Update ingress-minikube.md

* spell check pass

* Update ingress.md

* Update ingress-minikube.md
@DanyC97
Update debug-service.md's tasks file to remove $ from kubectl and tur…
b9de8a1 
…n some questions into bullet list (kubernetes#12798)
@lucperkins
Fix sticky footer issue (kubernetes#10681)
f7a7213 
* Fix sticky footer issue

* Fix merge conflict

* Aesthetic changes to 404 page
@sftim
Move Docker credentials import to task documentation (kubernetes#12668)
605a967 
* Move docker credentials import to task documentation

Relevant to kubernetes#12072

* Call out helpful note about per-namespace secrets
@sftim
List supported protocols for Service (kubernetes#12669)
e536a19 
Tidy up protocol support to show that TCP and UDP have support in
stable, whereas SCTP support is in alpha. For HTTP, direct users to
read up on Ingress instead.
@makocchi-git
apply content_template and follow the style guide (kubernetes#12759)
c9d67b0
Small refactor of Custom Resources topic (kubernetes#12502)
6d623a0 
- Adjusts levels
- Removes some obsolete info
- Rewrite of some of the intro material
@palmerabollo
docs: update acs-engine -> aks-engine (kubernetes#12804)
fd4743e 
* docs: update acs-engine -> aks-engine

acs-engine has been deprecated by Microsoft in favor of aks-engine

* chore: PR feedback

* fix missing AKS
@DanyC97
Update the parallel-processing-expansion.md task file and remove $ (k…
386edef 
…ubernetes#12794)
@sftim
Reference glossary definitions in “Connect a Front End to a Back End …
63861f4 
…Using a Service” (kubernetes#12706)

* Reference terms in glossary

* Enable syntax highlighting for shell snippets
@DanyC97
Update the fine-parallel-processing-work-queue.md task file to remove…
72932cc 
… $ and also remove text which is not appropiate for end user (kubernetes#12793)
@justinsb
Fix SSH tunnel links (kubernetes#12585)
4ea01e1 
* Fix SSH tunnel links

* Removed reference to GKE, which seemed to prompt the initial removal.
* Fixed the link to refer back to the correct paragraph now that it exists again.
* Noted deprecation of SSH tunnels but work to create a replacement.

Fix kubernetes#8262

This reverts commit 1da6560.

This reverts commit b8b71b6.

* Update master-node-communication.md
@WanLinghao
Add docs to illustrate how to schedule pod to specific node (kubernet…
f6e80ea 
…es#12871)
@remyleone
Update OWNERS_ALIASES (kubernetes#12708)
5c37466
start serverside-apply documentation
b7f174b
@kwiesmueller kwiesmueller force-pushed the serverside-apply-documentation branch from 5b85425 to b7f174b Compare February 28, 2019 14:23
@kwiesmueller
Copy link
Member Author

Okay that made it even worse.
Sorry, it's my first PR to the website repo and I wasn't which branch to select.
Is dev-1.14 correct for this?

@kbhawkey
Copy link
Contributor

Okay that made it even worse.
Sorry, it's my first PR to the website repo and I wasn't which branch to select.
Is dev-1.14 correct for this?

Yes, dev-1.14 is the correct branch, cc @jimangel

@apelisse apelisse mentioned this pull request Mar 5, 2019
Closed
@jennybuckley jennybuckley mentioned this pull request Mar 5, 2019
Closed
@jimangel
Copy link
Member

@kwiesmueller it looks like you might have combined some docs from other branches. I would recommend fixing them and force pushing back to this branch. Alternately, you can close this PR and open a new one with your changes if it's easier.

Either way, there is a "ready for review" deadline tomorrow (Monday) and it would be great to not have this at risk due to merge conflicts :)

Thanks!

@jimangel jimangel added this to the 1.14 milestone Mar 10, 2019
@kwiesmueller kwiesmueller mentioned this pull request Mar 11, 2019
Merged
@kwiesmueller
Copy link
Member Author

closed in favor of #13077

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanm aanm Awaiting requested review from aanm

@ahmetb ahmetb Awaiting requested review from ahmetb

@apelisse apelisse Awaiting requested review from apelisse

@jennybuckley jennybuckley Awaiting requested review from jennybuckley

@lavalamp lavalamp Awaiting requested review from lavalamp

Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. 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 language/ko Issues or PRs related to Korean language language/zh Issues or PRs related to Chinese language size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
1.14
Development

Successfully merging this pull request may close these issues.