)]}'
{".zuul.d/base.yaml":[{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"44c7278fc8789e4f8f2e2c218668b4cb5ca48508","unresolved":true,"context_lines":[{"line_number":30,"context_line":"        devstack_plugins:"},{"line_number":31,"context_line":"          kuryr-kubernetes: https://opendev.org/openstack/kuryr-kubernetes"},{"line_number":32,"context_line":"          devstack-plugin-container: https://opendev.org/openstack/devstack-plugin-container"},{"line_number":33,"context_line":"          kuryr-tempest-plugin: https://opendev.org/openstack/kuryr-tempest-plugin"},{"line_number":34,"context_line":"          neutron: https://opendev.org/openstack/neutron"},{"line_number":35,"context_line":"    vars:"},{"line_number":36,"context_line":"      # Default swap size got shrinked to 1 GB, it\u0027s way too small for us."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"94227e9a_4c807588","side":"PARENT","line":33,"updated":"2021-08-05 14:29:20.000000000","message":"ok, seems like removing this one was a mistake. this change should be reverted.","commit_id":"41fdde7fe1e004fcd0c8242233c26b22a18498b7"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"b0ba401ae3743784308f33f6e0e2ef5c280b2583","unresolved":true,"context_lines":[{"line_number":46,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch,linuxbridge"},{"line_number":47,"context_line":"        ML2_L3_PLUGIN: router"},{"line_number":48,"context_line":"        KURYR_K8S_API_PORT: 6443"},{"line_number":49,"context_line":"        tempest_plugins: "},{"line_number":50,"context_line":"          - kuryr-tempest-plugin"},{"line_number":51,"context_line":"        KURYR_K8S_CLOUD_PROVIDER: false"},{"line_number":52,"context_line":"        ETCD_USE_RAMDISK: true"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5fe68e8f_4909c1f6","line":49,"range":{"start_line":49,"start_character":24,"end_line":49,"end_character":25},"updated":"2021-08-05 18:14:16.000000000","message":"tempest_plugins is a leaf (key, attribute, whatever you call it) of vars, not devstack_localrc, it cannot be here. and also trailing space.","commit_id":"d3f47d6d0a5ad01c5dee09f0c72a8aae09c7efc0"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"f4062932c9a6c6c3b97952b28551c4c71e1e10df","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch,linuxbridge"},{"line_number":49,"context_line":"        ML2_L3_PLUGIN: router"},{"line_number":50,"context_line":"        KURYR_K8S_API_PORT: 6443"},{"line_number":51,"context_line":"        #TEMPEST_PLUGINS: \u0027/opt/stack/kuryr-tempest-plugin\u0027"},{"line_number":52,"context_line":"        KURYR_K8S_CLOUD_PROVIDER: false"},{"line_number":53,"context_line":"        ETCD_USE_RAMDISK: true"},{"line_number":54,"context_line":"        USE_PYTHON3: true"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"6aaf23a6_cf7dd5f5","line":51,"range":{"start_line":51,"start_character":8,"end_line":51,"end_character":59},"updated":"2021-08-10 11:17:57.000000000","message":"this can be removed?","commit_id":"9ab71e9ffaf1adbebc33c754c314f60c1277ed40"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"d649c951d59b0f01767d28e3d1795537bcda11fa","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        Q_ML2_PLUGIN_MECHANISM_DRIVERS: openvswitch,linuxbridge"},{"line_number":49,"context_line":"        ML2_L3_PLUGIN: router"},{"line_number":50,"context_line":"        KURYR_K8S_API_PORT: 6443"},{"line_number":51,"context_line":"        #TEMPEST_PLUGINS: \u0027/opt/stack/kuryr-tempest-plugin\u0027"},{"line_number":52,"context_line":"        KURYR_K8S_CLOUD_PROVIDER: false"},{"line_number":53,"context_line":"        ETCD_USE_RAMDISK: true"},{"line_number":54,"context_line":"        USE_PYTHON3: true"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"3424a30a_458bc47c","line":51,"range":{"start_line":51,"start_character":8,"end_line":51,"end_character":59},"in_reply_to":"6aaf23a6_cf7dd5f5","updated":"2021-08-10 13:07:23.000000000","message":"Done","commit_id":"9ab71e9ffaf1adbebc33c754c314f60c1277ed40"}],".zuul.d/octavia.yaml":[{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"44c7278fc8789e4f8f2e2c218668b4cb5ca48508","unresolved":true,"context_lines":[{"line_number":69,"context_line":"    description: |"},{"line_number":70,"context_line":"        Kuryr-Kubernetes tempest job running kuryr containerized"},{"line_number":71,"context_line":"    vars:"},{"line_number":72,"context_line":"        tempest_plugins:"},{"line_number":73,"context_line":"          - octavia-tempest-plugin"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ca42c066_7b34d8ba","line":72,"updated":"2021-08-05 14:29:20.000000000","message":"supernit: yaml indentation should be 2 spaces.","commit_id":"442f652e6212ddd85b9c3c3f3c62c5ce783482b8"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"44c7278fc8789e4f8f2e2c218668b4cb5ca48508","unresolved":true,"context_lines":[{"line_number":70,"context_line":"        Kuryr-Kubernetes tempest job running kuryr containerized"},{"line_number":71,"context_line":"    vars:"},{"line_number":72,"context_line":"        tempest_plugins:"},{"line_number":73,"context_line":"          - octavia-tempest-plugin"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- job:"},{"line_number":76,"context_line":"    name: kuryr-kubernetes-tempest-ipv6"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"6685b8c8_c1e2ae1e","line":73,"updated":"2021-08-05 14:29:20.000000000","message":"to make it work, we need also this:\n\n  devstack_plugins:\n    octavia-tempest-plugin: https://opendev.org/openstack/octavia-tempest-plugin.git","commit_id":"442f652e6212ddd85b9c3c3f3c62c5ce783482b8"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"b0ba401ae3743784308f33f6e0e2ef5c280b2583","unresolved":true,"context_lines":[{"line_number":71,"context_line":"    vars:"},{"line_number":72,"context_line":"      tempest_plugins:"},{"line_number":73,"context_line":"          - octavia-tempest-plugin"},{"line_number":74,"context_line":"    devstack_plugins:"},{"line_number":75,"context_line":"      octavia-tempest-plugin: https://opendev.org/openstack/octavia-tempest-plugin.git"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"68b7bbb9_cf838e27","line":74,"updated":"2021-08-05 18:14:16.000000000","message":"that should have same indent as tempest_plugin. devstack_plugins are under vars key.","commit_id":"674711464ab2a7f74934534759e3966911195a1e"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"b0ba401ae3743784308f33f6e0e2ef5c280b2583","unresolved":true,"context_lines":[{"line_number":25,"context_line":"      controller:"},{"line_number":26,"context_line":"        devstack_plugins:"},{"line_number":27,"context_line":"          octavia: https://opendev.org/openstack/octavia"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    vars:"},{"line_number":30,"context_line":"      devstack_localrc:"},{"line_number":31,"context_line":"        OCTAVIA_AMP_IMAGE_FILE: \"/tmp/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"2bf96f30_eab34161","line":28,"updated":"2021-08-05 18:14:16.000000000","message":"unrelated change.","commit_id":"d3f47d6d0a5ad01c5dee09f0c72a8aae09c7efc0"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"b0ba401ae3743784308f33f6e0e2ef5c280b2583","unresolved":true,"context_lines":[{"line_number":71,"context_line":"        Kuryr-Kubernetes tempest job running kuryr containerized"},{"line_number":72,"context_line":"    vars:"},{"line_number":73,"context_line":"      tempest_plugins:"},{"line_number":74,"context_line":"          - octavia-tempest-plugin"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"- job:"},{"line_number":77,"context_line":"    name: kuryr-kubernetes-tempest-ipv6"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"158abdce_2b6e8f3e","line":74,"updated":"2021-08-05 18:14:16.000000000","message":"you also need this:\n\n  vars:\n    tempest_plugins:\n      octavia-tempest-plugin: https://opendev.org/openstack/octavia-tempest-plugin.git\n\nbut for me, both template_plugins and tempest_plugins should be moved to kuryr-kubernetes-tempest-octavia-base job.","commit_id":"d3f47d6d0a5ad01c5dee09f0c72a8aae09c7efc0"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"f4062932c9a6c6c3b97952b28551c4c71e1e10df","unresolved":true,"context_lines":[{"line_number":29,"context_line":"          octavia-tempest-plugin: https://opendev.org/openstack/octavia-tempest-plugin.git"},{"line_number":30,"context_line":"    vars:"},{"line_number":31,"context_line":"      tempest_plugins:"},{"line_number":32,"context_line":"        - kuryr-tempest-plugin"},{"line_number":33,"context_line":"        - octavia-tempest-plugin"},{"line_number":34,"context_line":"      devstack_localrc:"},{"line_number":35,"context_line":"        OCTAVIA_AMP_IMAGE_FILE: \"/tmp/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"ed8e08e3_2e54a85b","line":32,"range":{"start_line":32,"start_character":8,"end_line":32,"end_character":30},"updated":"2021-08-10 11:17:57.000000000","message":"Is it needed to enable here as well if the parent job \"kuryr-kubernetes-tempest-base\" is already enabling it?","commit_id":"9ab71e9ffaf1adbebc33c754c314f60c1277ed40"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"eb1be04e8348124a62e0830c5efcb9b6513e47bc","unresolved":true,"context_lines":[{"line_number":29,"context_line":"          octavia-tempest-plugin: https://opendev.org/openstack/octavia-tempest-plugin.git"},{"line_number":30,"context_line":"    vars:"},{"line_number":31,"context_line":"      tempest_plugins:"},{"line_number":32,"context_line":"        - kuryr-tempest-plugin"},{"line_number":33,"context_line":"        - octavia-tempest-plugin"},{"line_number":34,"context_line":"      devstack_localrc:"},{"line_number":35,"context_line":"        OCTAVIA_AMP_IMAGE_FILE: \"/tmp/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"90b31cb6_5f081c2c","line":32,"range":{"start_line":32,"start_character":8,"end_line":32,"end_character":30},"in_reply_to":"ed8e08e3_2e54a85b","updated":"2021-08-10 13:05:09.000000000","message":"Through trial and error, I\u0027ve discovered that if job doesn\u0027t have kuryr-tempest-plugin, it will be not included, even if it is set on parent job. It seems, that only some keys are updated (as expected), while the others (with exceptions!) are overwritten, like this one.","commit_id":"9ab71e9ffaf1adbebc33c754c314f60c1277ed40"}]}
