ddbcf1b07d
We broke nodepool configuration with I3795fee1530045363e3f629f0793cbe6e95c23ca by not having the labels defined in the OpenStack provider in the top-level label list. The added check here would have found such a case. The validate() function is reworked slightly; previously it would return various exceptions from the tools it was calling (YAML, voluptuous, etc.). Now we have more testing (and I'd imagine we could do even more, similar vaildations too) we'd have to keep adding exception types. Just make the function return a value; this also makes sure the regular exit paths are taken from the caller in nodepoolcmd.py, rather than dying with an exception at whatever point. A unit test is added. Co-Authored-By: Mohammed Naser <mnaser@vexxhost.com> Change-Id: I5455f5d7eb07abea34c11a3026d630dee62f2185 |
||
---|---|---|
.. | ||
__init__.py | ||
builder.py | ||
config_validator.py | ||
launcher.py | ||
nodepoolcmd.py |