search

openshift / origin-server

OpenShift 2 (deprecated)
889 stars 516 forks source link

oo-admin-ctl-team: small fixes #6343

Closed thrasher-redhat closed 8 years ago

thrasher-redhat commented 8 years ago

check to avoid output if no members

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

The oo-admin-ctl-team script was printing "MEMBER|ADD|groupname|" even if there were no members being added. This will now only output if there are members in the members_to_add variable.

Also included is a commit to fix a variable typo which should mean a more accurate select statement.

Miciah commented 8 years ago

Looks good. Thanks!

openshift-bot, please [merge]!

openshift-bot commented 8 years ago

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

openshift-bot commented 8 years ago

Evaluated for online merge up to 37a930d958c2fe08cc267567cb6523e256c58479