Skip to content

Commit

Permalink
Fix and document environment variable KUBE_MASTERS (#7127)
Browse files Browse the repository at this point in the history
This variable was added as KUBE_MASTERS_MASTERS. That's probably a typo.
Remove the redundant `_MASTERS` suffix. Also, document the variable in the
help message.
  • Loading branch information
0mp authored Jan 11, 2021
1 parent 5dca522 commit 5517e62
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion contrib/inventory_builder/inventory.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ def get_var_as_bool(name, default):


CONFIG_FILE = os.environ.get("CONFIG_FILE", "./inventory/sample/hosts.yaml")
KUBE_MASTERS = int(os.environ.get("KUBE_MASTERS_MASTERS", 2))
KUBE_MASTERS = int(os.environ.get("KUBE_MASTERS", 2))
# Reconfigures cluster distribution at scale
SCALE_THRESHOLD = int(os.environ.get("SCALE_THRESHOLD", 50))
MASSIVE_SCALE_THRESHOLD = int(os.environ.get("MASSIVE_SCALE_THRESHOLD", 200))
Expand Down Expand Up @@ -402,6 +402,7 @@ def show_help(self):
DEBUG Enable debug printing. Default: True
CONFIG_FILE File to write config to Default: ./inventory/sample/hosts.yaml
HOST_PREFIX Host prefix for generated hosts. Default: node
KUBE_MASTERS Set the number of kube-masters. Default: 2
SCALE_THRESHOLD Separate ETCD role if # of nodes >= 50
MASSIVE_SCALE_THRESHOLD Separate K8s master and ETCD if # of nodes >= 200
''' # noqa
Expand Down

0 comments on commit 5517e62

Please sign in to comment.