diff options
author | Lénaïc Huard <lhuard@amadeus.com> | 2016-08-19 14:33:18 +0200 |
---|---|---|
committer | Lénaïc Huard <lhuard@amadeus.com> | 2016-08-19 14:53:03 +0200 |
commit | bce3235761a16fd8048760ce8ee63e0a64d672a7 (patch) | |
tree | 07ac1c972980bb67f67c19aa6e809c86ed07c76e /roles/openshift_cloud_provider | |
parent | a96a9b713bb15146fa15f2535731c4ac83cc7bd5 (diff) | |
download | openshift-bce3235761a16fd8048760ce8ee63e0a64d672a7.tar.gz openshift-bce3235761a16fd8048760ce8ee63e0a64d672a7.tar.bz2 openshift-bce3235761a16fd8048760ce8ee63e0a64d672a7.tar.xz openshift-bce3235761a16fd8048760ce8ee63e0a64d672a7.zip |
Fix the “node on master” feature
What we want to do is to add the master as a node if:
* `g_nodeonmaster` is set to true, and
* we are not in the case where we want to add new nodes.
The second test was done by only checking whether `g_new_node_hosts` was defined.
This was wrong because, in all cloud-provider setups, this variable was set
with the default value of “empty list” (`[]`).
The test has been changed to use the `bool` filter so that it correctly evaluates
to false (and hence, effectively add the master as a node) when `g_new_node_hosts`
is the empty list.
Diffstat (limited to 'roles/openshift_cloud_provider')
0 files changed, 0 insertions, 0 deletions