Skip to content

Conversation

@steveperry-53
Copy link
Contributor

@steveperry-53 steveperry-53 commented Feb 12, 2018

This PR moves some YAML configuration files out of the old user-guide directory and into the newer concepts/cluster-administration directory.

This PR also updates the topic that uses these YAML files: Managing Resources.

The Managing Resources topic also needs and edit pass for style, but that is not a goal of this PR.


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 12, 2018
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 12, 2018
@steveperry-53 steveperry-53 added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 12, 2018
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Feb 12, 2018

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

Built with commit 8b97240

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

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 13, 2018
@steveperry-53
Copy link
Contributor Author

/assign @heckj

@steveperry-53
Copy link
Contributor Author

/retest

@heckj
Copy link
Contributor

heckj commented Feb 14, 2018

@steveperry-53 this one also appears to be failing on the go-based validation tests - guessing related to the update you were making at #7405 ...

@steveperry-53
Copy link
Contributor Author

Eventually, in examples_test.go, we need to add apps.Deployment.

@heckj
Copy link
Contributor

heckj commented Feb 15, 2018

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 15, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: heckj, steveperry-53

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:
  • OWNERS [heckj,steveperry-53]

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 merged commit e2118a7 into kubernetes:master Feb 15, 2018
@steveperry-53 steveperry-53 deleted the old-dirs branch February 15, 2018 20:15
tehut pushed a commit to tehut/website that referenced this pull request Feb 20, 2018
bsalamat pushed a commit to bsalamat/kubernetes.github.io that referenced this pull request Feb 23, 2018
tehut pushed a commit to tehut/website that referenced this pull request Mar 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants