search

openshift / origin-server

OpenShift 2 (deprecated)
889 stars 516 forks source link

oo-accept-node: update to check cgroup tasks #6347

Closed thrasher-redhat closed 8 years ago

thrasher-redhat commented 8 years ago

Bug 1067107 BZ https://bugzilla.redhat.com/show_bug.cgi?id=1067107

Updates oo-accept-node to check thread ids against cgroup tasks instead of checking process ids against cgroup procs.

thrasher-redhat commented 8 years ago

[test]

openshift-bot commented 8 years ago

Evaluated for online test up to 5c6699e31aee4139d976bb9a254d252eebdb719c

openshift-bot commented 8 years ago

Online Test Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests/9147/)

thrasher-redhat commented 8 years ago

@a13m Wanted to run this by you before merging to make sure that this is what you had in mind. Don't want to introduce a major change into a commonly used script like oo-accept-node without lots of input.

Miciah commented 8 years ago

Looks spiffin'. Please [merge]!

openshift-bot commented 8 years ago

Online Merge Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/merge_pull_requests/6697/) (Image: devenv_5759)

openshift-bot commented 8 years ago

Evaluated for online merge up to 5c6699e31aee4139d976bb9a254d252eebdb719c

Miciah commented 8 years ago

Yay! Thanks!