Skip to content

Remove old-docs from blocked path#3175

Merged
0xmichalis merged 1 commit intokubernetes:masterfrom
cblecker:old-docs
Aug 1, 2017
Merged

Remove old-docs from blocked path#3175
0xmichalis merged 1 commit intokubernetes:masterfrom
cblecker:old-docs

Conversation

@cblecker
Copy link
Copy Markdown
Member

@cblecker cblecker commented Jun 23, 2017

Per kubernetes/kubernetes#46813, we're removing a bunch of the static content from docs/ in k/k. This adjusts the paths of the old-docs munger to limit the amount of things that aren't allowed to be auto merged.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 23, 2017
@cblecker
Copy link
Copy Markdown
Member Author

@thockin Mind reviewing when you have a moment?

@thockin
Copy link
Copy Markdown
Member

thockin commented Aug 1, 2017

lgtm, needs rebase.

@0xmichalis
Copy link
Copy Markdown
Contributor

Can we move this into a configmap and not mount it directly in the dockerfile?

@cblecker
Copy link
Copy Markdown
Member Author

cblecker commented Aug 1, 2017

@Kargakis I'm totally okay with that.. except that I would want to detach the policy change (what this PR does) from the functional change (have this be a configmap instead of a file added to the docker image)

@cblecker
Copy link
Copy Markdown
Member Author

cblecker commented Aug 1, 2017

@thockin rebased and tests pass

@cblecker
Copy link
Copy Markdown
Member Author

cblecker commented Aug 1, 2017

@kubernetes/test-infra-maintainers This should be good to merge now :)

@0xmichalis
Copy link
Copy Markdown
Contributor

Sure.

@0xmichalis 0xmichalis merged commit e42bb0c into kubernetes:master Aug 1, 2017
@cblecker cblecker deleted the old-docs branch August 1, 2017 19:42
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. lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants