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

Jump to bottom

Add an example demonstrating runtime constraints #14446

Merged
merged 1 commit into from Oct 7, 2015
Merged

Add an example demonstrating runtime constraints #14446

merged 1 commit into from Oct 7, 2015

Conversation

derekwaynecarr
Copy link
Member

This is an example document demonstrating runtime constraint enforcement on the node.

@k8s-bot
Copy link

k8s-bot commented Sep 23, 2015

GCE e2e build/test failed for commit 38382282a48a61b5d7011a4615f0b82ce6dcb256.

@k8s-github-robot k8s-github-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 23, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/L

@k8s-bot
Copy link

k8s-bot commented Sep 23, 2015

GCE e2e build/test passed for commit 91b5f49a33f02c3989f23c54a6d76503ca3e517c.

@bgrant0607 bgrant0607 assigned davidopp and vishh and unassigned brendandburns and davidopp Sep 24, 2015
@derekwaynecarr derekwaynecarr changed the title WIP: Add an example demonstrating runtime constraints Add an example demonstrating runtime constraints Sep 29, 2015
@k8s-bot
Copy link

k8s-bot commented Sep 29, 2015

Unit, integration and GCE e2e test build/test passed for commit a23b1be.

@vishh
Copy link
Contributor

vishh commented Oct 5, 2015

@derekwaynecarr: This PR LGTM. We might have to update the last section once #14943 gets merged. Let me know if you'd like this to be merged right away instead of waiting for #14943.

@derekwaynecarr
Copy link
Member Author

Let's merge as is, and I will then update as required.

On Mon, Oct 5, 2015 at 1:57 PM, Vish Kannan notifications@github.com
wrote:

@derekwaynecarr https://github.com/derekwaynecarr: This PR LGTM. We
might have to update the last section once #14943
#14943 gets merged. Let me
know if you'd like this to be merged right away instead of waiting for
#14943 #14943.

—
Reply to this email directly or view it on GitHub
#14446 (comment)
.

@vishh vishh added lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-merge labels Oct 6, 2015
@vishh
Copy link
Contributor

vishh commented Oct 6, 2015

Acknowledged.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

mikedanese added a commit that referenced this pull request Oct 7, 2015
@mikedanese
Merge pull request #14446 from derekwaynecarr/runtime_constraints_exa…
be0c619 
…mple

Add an example demonstrating runtime constraints
@mikedanese mikedanese merged commit be0c619 into kubernetes:master Oct 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@vishh vishh

Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@derekwaynecarr @k8s-bot @k8s-github-robot @vishh @mikedanese @googlebot @brendandburns @erictune @davidopp