[VVP] test_environment_file_parameters violation fix
[vvp/validation-scripts.git] / ice_validator / tests / test_environment_file_parameters.py
1 # -*- coding: utf8 -*-
2 # ============LICENSE_START====================================================
3 # org.onap.vvp/validation-scripts
4 # ===================================================================
5 # Copyright © 2019 AT&T Intellectual Property. All rights reserved.
6 # ===================================================================
7 #
8 # Unless otherwise specified, all software contained herein is licensed
9 # under the Apache License, Version 2.0 (the "License");
10 # you may not use this software except in compliance with the License.
11 # You may obtain a copy of the License at
12 #
13 #             http://www.apache.org/licenses/LICENSE-2.0
14 #
15 # Unless required by applicable law or agreed to in writing, software
16 # distributed under the License is distributed on an "AS IS" BASIS,
17 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
18 # See the License for the specific language governing permissions and
19 # limitations under the License.
20 #
21 #
22 #
23 # Unless otherwise specified, all documentation contained herein is licensed
24 # under the Creative Commons License, Attribution 4.0 Intl. (the "License");
25 # you may not use this documentation except in compliance with the License.
26 # You may obtain a copy of the License at
27 #
28 #             https://creativecommons.org/licenses/by/4.0/
29 #
30 # Unless required by applicable law or agreed to in writing, documentation
31 # distributed under the License is distributed on an "AS IS" BASIS,
32 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
33 # See the License for the specific language governing permissions and
34 # limitations under the License.
35 #
36 # ============LICENSE_END============================================
37 #
38 #
39 """ environment file structure
40 """
41 import os
42
43 import re
44 import pytest
45 from tests.helpers import (
46     prop_iterator,
47     get_param,
48     get_environment_pair,
49     validates,
50     find_environment_file,
51     categories,
52 )
53 from tests.structures import Heat
54 from tests.utils.nested_files import file_is_a_nested_template
55
56
57 # Whats persistent mean? It means it goes in env.
58 # When adding an additional case, note the ","
59 # at the end of a property to make it a tuple.
60 ENV_PARAMETER_SPEC = {
61     "PLATFORM PROVIDED": [
62         {"property": ("vnf_id",), "persistent": False, "kwargs": {}},
63         {"property": ("vnf_name",), "persistent": False, "kwargs": {}},
64         {"property": ("vf_module_id",), "persistent": False, "kwargs": {}},
65         {"property": ("vf_module_index",), "persistent": False, "kwargs": {}},
66         {"property": ("vf_module_name",), "persistent": False, "kwargs": {}},
67         {"property": ("workload_context",), "persistent": False, "kwargs": {}},
68         {"property": ("environment_context",), "persistent": False, "kwargs": {}},
69         {"property": (r"^(.+?)_net_fqdn$",), "persistent": False, "kwargs": {}},
70     ],
71     "ALL": [{"property": ("name",), "persistent": False, "kwargs": {}}],
72     "OS::Nova::Server": [
73         {"property": ("image",), "persistent": True, "kwargs": {}},
74         {"property": ("flavor",), "persistent": True, "kwargs": {}},
75         {"property": ("availability_zone",), "persistent": False, "kwargs": {}},
76     ],
77     "OS::Neutron::Port": [
78         {"property": ("network",), "persistent": False, "kwargs": {}},
79         {
80             "property": ("fixed_ips", "ip_address"),
81             "persistent": False,
82             "network_type": "external",
83             "kwargs": {"exclude_parameter": re.compile(r"^(.+?)_int_(.+?)$")},
84         },
85         {
86             "property": ("fixed_ips", "ip_address"),
87             "persistent": True,
88             "network_type": "internal",
89             "kwargs": {"exclude_parameter": re.compile(r"^((?!_int_).)*$")},
90         },
91         {"property": ("fixed_ips", "subnet"), "persistent": False, "kwargs": {}},
92         {
93             "property": ("fixed_ips", "allowed_address_pairs"),
94             "persistent": False,
95             "network_type": "external",
96             "kwargs": {"exclude_parameter": re.compile(r"^(.+?)_int_(.+?)$")},
97         },
98         {
99             "property": ("fixed_ips", "allowed_address_pairs"),
100             "persistent": True,
101             "network_type": "internal",
102             "kwargs": {"exclude_parameter": re.compile(r"^((?!_int_).)*$")},
103         },
104     ],
105     "OS::ContrailV2::InterfaceRouteTable": [
106         {
107             "property": (
108                 "interface_route_table_routes",
109                 "interface_route_table_routes_route",
110             ),
111             "persistent": False,
112             "kwargs": {},
113         }
114     ],
115     "OS::Heat::ResourceGroup": [
116         {
117             "property": ("count",),
118             "persistent": True,
119             "kwargs": {
120                 "exclude_resource": re.compile(
121                     r"^(.+?)_subint_(.+?)_port_(.+?)_subinterfaces$"
122                 )
123             },
124         }
125     ],
126     "OS::ContrailV2::InstanceIp": [
127         {
128             "property": ("instance_ip_address",),
129             "persistent": False,
130             "network_type": "external",
131             "kwargs": {"exclude_resource": re.compile(r"^.*_int_.*$")},
132         },
133         {
134             "property": ("instance_ip_address",),
135             "persistent": True,
136             "network_type": "internal",
137             "kwargs": {"exclude_resource": re.compile(r"(?!.*_int_.*)")},
138         },
139         {
140             "property": ("subnet_uuid",),
141             "persistent": False,
142             "network_type": "internal",
143             "kwargs": {"exclude_resource": re.compile(r"(?!.*_int_.*)")},
144         },
145     ],
146     "OS::ContrailV2::VirtualMachineInterface": [
147         {
148             "property": (
149                 "virtual_machine_interface_allowed_address_pairs",
150                 "virtual_machine_interface_allowed_address_pairs_allowed_address_pair",
151                 "virtual_machine_interface_allowed_address_pairs_allowed_address_pair_ip",
152                 "virtual_machine_interface_allowed_address_pairs_allowed_address_pair_ip_ip_prefix",
153             ),
154             "persistent": False,
155             "network_type": "external",
156             "kwargs": {"exclude_resource": re.compile(r"(?!.*_int_.*)")},
157         }
158     ],
159 }
160
161
162 def run_test_parameter(yaml_file, resource_type, *prop, **kwargs):
163     template_parameters = []
164     invalid_parameters = []
165     param_spec = {}
166     parameter_spec = ENV_PARAMETER_SPEC.get(
167         resource_type
168     )  # matching spec dict on resource type
169     for spec in parameter_spec:
170         # iterating through spec dict and trying to match on property
171         if spec.get("property") == prop:
172             yep = True
173             for (
174                 k,
175                 v,
176             ) in (
177                 kwargs.items()
178             ):  # now matching on additional kwargs passed in from test (i.e. network_type)
179                 if not spec.get(k) or spec.get(k) != v:
180                     yep = False
181             if yep:
182                 param_spec = spec
183                 if resource_type == "PLATFORM PROVIDED":
184                     if file_is_a_nested_template(yaml_file):
185                         pytest.skip(
186                             "Not checking nested files for PLATFORM PROVIDED params"
187                         )
188                     template_parameters.append(
189                         {"resource": "", "param": param_spec.get("property")[0]}
190                     )
191                 else:
192                     all_resources = False
193                     if resource_type == "ALL":
194                         all_resources = True
195                     template_parameters = get_template_parameters(
196                         yaml_file,
197                         resource_type,
198                         param_spec,
199                         all_resources=all_resources,
200                     )  # found the correct spec, proceeding w/ test
201                 break
202
203     for parameter in template_parameters:
204         param = parameter.get("param")
205         persistence = param_spec.get("persistent")
206
207         if env_violation(yaml_file, param, spec.get("persistent")):
208             human_text = "must" if persistence else "must not"
209             human_text2 = "was not" if persistence else "was"
210
211             invalid_parameters.append(
212                 "{} parameter {} {} be enumerated in an environment file, but "
213                 "parameter {} for {} {} found.".format(
214                     resource_type, prop, human_text, param, yaml_file, human_text2
215                 )
216             )
217
218     assert not invalid_parameters, "\n".join(invalid_parameters)
219
220
221 def get_preload_excluded_parameters(yaml_file):
222     """
223     Returns set of all parameters that should not be included in the preload's
224     vnf parameters/tag-values section.
225     """
226     results = []
227     for resource_type, specs in ENV_PARAMETER_SPEC.items():
228         # apply to all resources if not in the format of an OpenStack resource
229         all_resources = "::" not in resource_type
230         for spec in specs:
231             results.extend(get_template_parameters(yaml_file, resource_type,
232                                                    spec, all_resources))
233     return {item["param"] for item in results}
234
235
236 def get_template_parameters(yaml_file, resource_type, spec, all_resources=False):
237     parameters = []
238
239     heat = Heat(yaml_file)
240     if all_resources:
241         resources = heat.resources
242     else:
243         resources = heat.get_resource_by_type(resource_type)
244
245     for rid, resource_props in resources.items():
246         for param in prop_iterator(resource_props, *spec.get("property")):
247             if param and get_param(param) and param_helper(spec, get_param(param), rid):
248                 # this is first getting the param
249                 # then checking if its actually using get_param
250                 # then checking a custom helper function (mostly for internal vs external networks)
251                 parameters.append({"resource": rid, "param": get_param(param)})
252
253     return parameters
254
255
256 def env_violation(yaml_file, parameter, persistent):
257     # Returns True IF there's a violation, False if everything looks good.
258
259     filepath, filename = os.path.split(yaml_file)
260     environment_pair = get_environment_pair(yaml_file)
261     if not environment_pair:  # this is a nested file perhaps?
262         environment_pair = find_environment_file(
263             yaml_file
264         )  # we want to check parent env
265         if not environment_pair:
266             pytest.skip("unable to determine environment file for nested yaml file")
267
268     env_yaml = environment_pair.get("eyml")
269     parameters = env_yaml.get("parameters", {})
270     in_env = False
271     if parameters:  # env file can be just parameters:
272         for param, value in parameters.items():
273             if re.match(parameter, param):
274                 in_env = True
275                 break
276
277     # confusing return. This function is looking for a violation.
278     return not persistent == in_env
279
280
281 def param_helper(spec, param, rid):
282     # helper function that has some predefined additional
283     # checkers, mainly to figure out if internal/external network
284     keeper = True
285     for k, v in spec.get("kwargs").items():
286         if k == "exclude_resource" and re.match(v, rid):
287             keeper = False
288             break
289         elif k == "exclude_parameter" and re.match(v, param):
290             keeper = False
291             break
292
293     return keeper
294
295
296 @validates("R-91125")
297 def test_nova_server_image_parameter_exists_in_environment_file(yaml_file):
298     run_test_parameter(yaml_file, "OS::Nova::Server", "image")
299
300
301 @validates("R-69431")
302 def test_nova_server_flavor_parameter_exists_in_environment_file(yaml_file):
303     run_test_parameter(yaml_file, "OS::Nova::Server", "flavor")
304
305
306 @categories("environment_file")
307 @validates("R-22838", "R-99812")
308 def test_nova_server_name_parameter_doesnt_exist_in_environment_file(yaml_file):
309     run_test_parameter(yaml_file, "ALL", "name")
310
311
312 @categories("environment_file")
313 @validates("R-59568")
314 def test_nova_server_az_parameter_doesnt_exist_in_environment_file(yaml_file):
315     run_test_parameter(yaml_file, "OS::Nova::Server", "availability_zone")
316
317
318 @categories("environment_file")
319 @validates("R-20856")
320 def test_nova_server_vnf_id_parameter_doesnt_exist_in_environment_file(yaml_file):
321     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "vnf_id")
322
323
324 @categories("environment_file")
325 @validates("R-72871")
326 def test_nova_server_vf_module_id_parameter_doesnt_exist_in_environment_file(yaml_file):
327     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "vf_module_id")
328
329
330 @categories("environment_file")
331 @validates("R-37039")
332 def test_nova_server_vf_module_index_parameter_doesnt_exist_in_environment_file(
333     yaml_file
334 ):
335     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "vf_module_index")
336
337
338 @categories("environment_file")
339 @validates("R-36542")
340 def test_nova_server_vnf_name_parameter_doesnt_exist_in_environment_file(yaml_file):
341     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "vnf_name")
342
343
344 @categories("environment_file")
345 @validates("R-80374")
346 def test_nova_server_vf_module_name_parameter_doesnt_exist_in_environment_file(
347     yaml_file
348 ):
349     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "vf_module_name")
350
351
352 @categories("environment_file")
353 @validates("R-02691")
354 def test_nova_server_workload_context_parameter_doesnt_exist_in_environment_file(
355     yaml_file
356 ):
357     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "workload_context")
358
359
360 @categories("environment_file")
361 @validates("R-13194")
362 def test_nova_server_environment_context_parameter_doesnt_exist_in_environment_file(
363     yaml_file
364 ):
365     run_test_parameter(yaml_file, "PLATFORM PROVIDED", "environment_context")
366
367
368 @categories("environment_file")
369 @validates("R-29872")
370 def test_neutron_port_network_parameter_doesnt_exist_in_environment_file(yaml_file):
371     run_test_parameter(yaml_file, "OS::Neutron::Port", "network")
372
373
374 @categories("environment_file")
375 @validates("R-39841", "R-87123", "R-62590", "R-98905", "R-93030", "R-62590")
376 def test_neutron_port_external_fixedips_ipaddress_parameter_doesnt_exist_in_environment_file(
377     yaml_file
378 ):
379     run_test_parameter(
380         yaml_file,
381         "OS::Neutron::Port",
382         "fixed_ips",
383         "ip_address",
384         network_type="external",
385     )
386
387
388 @validates("R-28795", "R-97201", "R-93496", "R-90206", "R-98569", "R-93496")
389 def test_neutron_port_internal_fixedips_ipaddress_parameter_exists_in_environment_file(
390     yaml_file
391 ):
392     run_test_parameter(
393         yaml_file,
394         "OS::Neutron::Port",
395         "fixed_ips",
396         "ip_address",
397         network_type="internal",
398     )
399
400
401 @categories("environment_file")
402 @validates("R-83677", "R-80829", "R-69634", "R-22288")
403 def test_neutron_port_fixedips_subnet_parameter_doesnt_exist_in_environment_file(
404     yaml_file
405 ):
406     run_test_parameter(
407         yaml_file, "OS::Neutron::Port", "fixed_ips", "subnet", network_type="internal"
408     )
409
410
411 @categories("environment_file")
412 @validates("R-83412", "R-83418")
413 def test_neutron_port_external_aap_ip_parameter_doesnt_exist_in_environment_file(
414     yaml_file
415 ):
416     run_test_parameter(
417         yaml_file,
418         "OS::Neutron::Port",
419         "allowed_address_pairs",
420         "subnet",
421         network_type="external",
422     )
423
424
425 @categories("environment_file")
426 @validates("R-92193")
427 def test_network_fqdn_parameter_doesnt_exist_in_environment_file(yaml_file):
428     run_test_parameter(yaml_file, "PLATFORM PROVIDED", r"^(.+?)_net_fqdn$")
429
430
431 @categories("environment_file")
432 @validates("R-76682")
433 def test_contrail_route_prefixes_parameter_doesnt_exist_in_environment_file(yaml_file):
434     run_test_parameter(
435         yaml_file,
436         "OS::ContrailV2::InterfaceRouteTable",
437         "interface_route_table_routes",
438         "interface_route_table_routes_route",
439     )
440
441
442 @validates("R-50011")
443 def test_heat_rg_count_parameter_exists_in_environment_file(yaml_file):
444     run_test_parameter(yaml_file, "OS::Heat::ResourceGroup", "count")
445
446
447 @categories("environment_file")
448 @validates("R-100020", "R-100040", "R-100060", "R-100080", "R-100170")
449 def test_contrail_external_instance_ip_does_not_exist_in_environment_file(yaml_file):
450     run_test_parameter(
451         yaml_file,
452         "OS::ContrailV2::InstanceIp",
453         "instance_ip_address",
454         network_type="external",
455     )
456
457
458 @validates("R-100100", "R-100120", "R-100140", "R-100160", "R-100180")
459 def test_contrail_internal_instance_ip_does_exist_in_environment_file(yaml_file):
460     run_test_parameter(
461         yaml_file,
462         "OS::ContrailV2::InstanceIp",
463         "instance_ip_address",
464         network_type="internal",
465     )
466
467
468 @categories("environment_file")
469 @validates("R-100210", "R-100230", "R-100250", "R-100270")
470 def test_contrail_subnet_uuid_does_not_exist_in_environment_file(yaml_file):
471     run_test_parameter(yaml_file, "OS::ContrailV2::InstanceIp", "subnet_uuid")
472
473
474 @categories("environment_file")
475 @validates("R-100320", "R-100340")
476 def test_contrail_vmi_aap_does_not_exist_in_environment_file(yaml_file):
477     run_test_parameter(
478         yaml_file,
479         "OS::ContrailV2::VirtualMachineInterface",
480         "virtual_machine_interface_allowed_address_pairs",
481         "virtual_machine_interface_allowed_address_pairs_allowed_address_pair",
482         "virtual_machine_interface_allowed_address_pairs_allowed_address_pair_ip",
483         "virtual_machine_interface_allowed_address_pairs_allowed_address_pair_ip_ip_prefix",
484     )