diff options
author | Ben Nemec <bnemec@redhat.com> | 2016-05-31 11:47:10 -0500 |
---|---|---|
committer | Ben Nemec <bnemec@redhat.com> | 2017-06-12 15:02:50 -0500 |
commit | f503d1b0e7fb9fe77e6fd1e71e08ca2d43427578 (patch) | |
tree | 124943345293b00d8225b7078176aa1be2703664 /tools | |
parent | 4e24c8cb6a1f65b1e5c764febfe96c2c465cfed1 (diff) |
Support config dir for env generator input files
We're not going to want to list every single sample environment in
a single file, so let's also take a directory and just read every
yaml file in it. This commit adds support for that as well as
some initial environments to demonstrate its use.
Change-Id: If2c608f2a61fc5e16784ab594d23f1fa335e1d3c
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/yaml-validate.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/tools/yaml-validate.py b/tools/yaml-validate.py index 92d76d23..d7c0bd93 100755 --- a/tools/yaml-validate.py +++ b/tools/yaml-validate.py @@ -20,8 +20,15 @@ import yaml required_params = ['EndpointMap', 'ServiceNetMap', 'DefaultPasswords', 'RoleName', 'RoleParameters'] +# NOTE(bnemec): The duplication in this list is intentional. For the +# transition to generated environments we have two copies of these files, +# so they need to be listed twice. Once the deprecated version can be removed +# the duplicate entries can be as well. envs_containing_endpoint_map = ['tls-endpoints-public-dns.yaml', 'tls-endpoints-public-ip.yaml', + 'tls-everywhere-endpoints-dns.yaml', + 'tls-endpoints-public-dns.yaml', + 'tls-endpoints-public-ip.yaml', 'tls-everywhere-endpoints-dns.yaml'] ENDPOINT_MAP_FILE = 'endpoint_map.yaml' REQUIRED_DOCKER_SECTIONS = ['service_name', 'docker_config', 'puppet_config', @@ -294,9 +301,9 @@ if base_endpoint_map and \ else: print("%s matches base endpoint map" % env_endpoint_map['file']) else: - print("ERROR: Can't validate endpoint maps since a file is missing. " - "If you meant to delete one of these files you should update this " - "tool as well.") + print("ERROR: Did not find expected number of environments containing the " + "EndpointMap parameter. If you meant to add or remove one of these " + "environments then you also need to update this tool.") if not base_endpoint_map: failed_files.append(ENDPOINT_MAP_FILE) if len(env_endpoint_maps) != len(envs_containing_endpoint_map): |