Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

allow cgroups to be disabled in node.conf #1436

Closed
wants to merge 1 commit into from
Closed

allow cgroups to be disabled in node.conf #1436

wants to merge 1 commit into from

Conversation

markllama
Copy link
Contributor

Allow and respond to ENABLE_CGROUPS=0 in /etc/openshift/node.conf

This allows openshift to run on systems where cgroups do not exist, or where cgroup constraints on gears are not needed.

@markllama
Copy link
Contributor Author

This request must come after li pull request 916 so that tests will pass.

https://github.com/openshift/li/pull/916

@kraman
Copy link
Contributor

kraman commented Feb 28, 2013

[test]

@openshift-bot
Copy link

Origin Test Results: FAILURE (https://originci-openshift.rhcloud.com/job/test_pull_requests/294/)

@openshift-bot
Copy link

@markllama
Copy link
Contributor Author

This update cannot pass until https://github.com/openshift/origin-dev-tools/issues/36 and https://github.com/openshift/li/issues/916 have been accepted.

These two requests update the tests to accept oo-accept-node PASS output with warnings

@openshift-bot
Copy link

Evaluated for origin up to f52a752ca1aeaa07ffbf7e7da8e71edc287f7693

@openshift-bot
Copy link

Linked pull request #1436 is not mergeable

These changes allow the service owner to decide if they wish to use cgroup
containment and resource management.
@openshift-bot
Copy link

Evaluated for online up to ef7835a

@openshift-bot
Copy link

Linked pull request 916 in repo 'li' is not mergeable

@openshift-bot
Copy link

User 'markllama' is not permitted to test linked pull request 36 in 'origin-dev-tools'. If the test was requested by another user with permission to test in 'origin-dev-tools', the linked pull request must be in a comment before the test comment (not including the pull request description) or be added by a user with permission to test in 'origin-dev-tools'.

@markllama
Copy link
Contributor Author

I'm cancelling this request until the dependent test changes can be merged.

@markllama markllama closed this Mar 15, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants