merge-new-params-nic-config-script - StopIteration ERROR - Not userfriendly

[Harald Jensås]

The script fails with a StopIteration error here:

    nic_config_name = next((x.get('deprecated_nic_config_name',
                                  OPTS.role_name.lower() + '.yaml') for x in
                            roles_data if x['name'] == OPTS.role_name))

The exception StopIteration indicates that it iterated trough all the roles in roles data without finding a match. The user specified a --role-name that does not exist in roles_data.

The script should handle this error case more gracefully and indicate the role is missing instead of generating StopIteration.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/631950

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/631950
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=3fa634908969d3f7b24c6cbd284610075952a6be
Submitter: Zuul
Branch: master

commit 3fa634908969d3f7b24c6cbd284610075952a6be
Author: Harald Jensås <email address hidden>
Date: Sun Jan 20 03:57:03 2019 +0100

    UX - Useful error msg if role is not in roles data

    If the user specify a role name that does not exist
    in the provided roles data the scipt exits with a
    StopIteration error. Catch it and raise RuntimeError
    with user friendly error message.

    Closes-Bug: #1812530
    Change-Id: I704316f66c197668a7d8e373efe00889776d2a85

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/tripleo-heat-templates 10.4.0 release.