)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6088d81414c34c2bca5bea1f811e0d74b1e53185","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ea66304f_bdfdcbe2","updated":"2022-05-04 13:18:30.000000000","message":"https://review.opendev.org/c/openstack/tripleo-quickstart/+/839724 and https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839694 fixes the issue.","commit_id":"e19179859ccd75fc478a190a35d36af95c7c35d5"}],"defaults/main.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"38fb68628c1dc79a868da92cb18ae769c447efc3","unresolved":true,"context_lines":[{"line_number":369,"context_line":"  create: true"},{"line_number":370,"context_line":"  os-cloud: \"{{ tempest_cloud_name }}\""},{"line_number":371,"context_line":"  out: \"{{ tempest_workspace }}/etc/tempest.conf\""},{"line_number":372,"context_line":"  network-id: \"{{ tempest_neutron_public_network_id | default(\u0027\u0027) }}\""},{"line_number":373,"context_line":"  overrides: \"{{ tempest_tempestconf_profile_overrides }}\""},{"line_number":374,"context_line":"tempest_tempestconf_profile_extras: {}"},{"line_number":375,"context_line":"tempest_tempestconf_profile_overrides: \"{{ tempest_tempest_conf_overrides }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"47772f97_9083c3d5","line":372,"updated":"2022-04-29 09:57:08.000000000","message":"I think i would question here why network-id needs to be a top level member of the profile, where everything else is normally defined via tempest_tempestconf_profile_extras and tempest_tempestconf_profile_overrides?\n\nIs that really the root cause here?","commit_id":"e19179859ccd75fc478a190a35d36af95c7c35d5"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"97ce8edbfdf91dd194728dad18f94af6cda8e28d","unresolved":true,"context_lines":[{"line_number":369,"context_line":"  create: true"},{"line_number":370,"context_line":"  os-cloud: \"{{ tempest_cloud_name }}\""},{"line_number":371,"context_line":"  out: \"{{ tempest_workspace }}/etc/tempest.conf\""},{"line_number":372,"context_line":"  network-id: \"{{ tempest_neutron_public_network_id | default(\u0027\u0027) }}\""},{"line_number":373,"context_line":"  overrides: \"{{ tempest_tempestconf_profile_overrides }}\""},{"line_number":374,"context_line":"tempest_tempestconf_profile_extras: {}"},{"line_number":375,"context_line":"tempest_tempestconf_profile_overrides: \"{{ tempest_tempest_conf_overrides }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7c44226e_ad528dca","line":372,"in_reply_to":"47772f97_9083c3d5","updated":"2022-04-29 10:19:30.000000000","message":"Earier https://opendev.org/openstack/openstack-ansible-os_tempest/src/branch/master/tasks/tempest_resources.yml#L217\n\n```\n    - name: Store neutron public network id\n      set_fact:\n        tempest_neutron_public_network_id: \"{{ tempest_service_available_neutron | ternary(tempest_public_network.id, \u0027\u0027) }}\"\n```\n\nwas there without when clause.\nSo tempest_neutron_public_network_id is setting to \u0027\u0027. That\u0027s why we have never seen this issue.\n\nBut after these two conditions got added, I need to enable two two conditions\n      - tempest_service_available_neutron | bool\n        - tempest_public_net_create | bool\n\nvia this patch https://review.opendev.org/c/openstack/tripleo-quickstart/+/839724 and then the issue gone.\n\nThen I thought let\u0027s add a default value there to avoid the issue of undefined variable.","commit_id":"e19179859ccd75fc478a190a35d36af95c7c35d5"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"4b5f6afd9513045ddf354fe1d4f6261cb0a229e9","unresolved":true,"context_lines":[{"line_number":369,"context_line":"  create: true"},{"line_number":370,"context_line":"  os-cloud: \"{{ tempest_cloud_name }}\""},{"line_number":371,"context_line":"  out: \"{{ tempest_workspace }}/etc/tempest.conf\""},{"line_number":372,"context_line":"  network-id: \"{{ tempest_neutron_public_network_id | default(\u0027\u0027) }}\""},{"line_number":373,"context_line":"  overrides: \"{{ tempest_tempestconf_profile_overrides }}\""},{"line_number":374,"context_line":"tempest_tempestconf_profile_extras: {}"},{"line_number":375,"context_line":"tempest_tempestconf_profile_overrides: \"{{ tempest_tempest_conf_overrides }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ba064c9e_a9688e68","line":372,"in_reply_to":"7c44226e_ad528dca","updated":"2022-04-29 10:22:41.000000000","message":"EDIT: I think tempest_neutron_public_network_id value got set via setfact, So we added it under tempest_tempestconf_profile.","commit_id":"e19179859ccd75fc478a190a35d36af95c7c35d5"}]}
