)]}'
{".zuul.d/base.yaml":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"066993042f8cd56965d35c6b6b4f60935205712d","unresolved":true,"context_lines":[{"line_number":44,"context_line":"        ENABLE_TLS: true"},{"line_number":45,"context_line":"        ETCD_USE_RAMDISK: true"},{"line_number":46,"context_line":"        KURYR_ENABLED_HANDLERS: vif,endpoints,service,namespace,pod_label,policy,kuryrnetworkpolicy,kuryrnetwork,kuryrport,kuryrloadbalancer"},{"line_number":47,"context_line":"        KURYR_SG_DRIVER: policy"},{"line_number":48,"context_line":"        KURYR_SUBNET_DRIVER: namespace"},{"line_number":49,"context_line":"      devstack_services:"},{"line_number":50,"context_line":"        c-api: false"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"b3fa641e_77eba774","line":47,"range":{"start_line":47,"start_character":8,"end_line":47,"end_character":31},"updated":"2021-09-07 20:53:54.000000000","message":"If we set the sg_driver by default to policy, the actual default of this setting won\u0027t be tested. Perhaps we should create a separate job for np? Same applies to KURYR_SUBNET_DRIVER","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"bb2f9c817a319bb6457a0deab1118f15dc9b7b9b","unresolved":true,"context_lines":[{"line_number":44,"context_line":"        ENABLE_TLS: true"},{"line_number":45,"context_line":"        ETCD_USE_RAMDISK: true"},{"line_number":46,"context_line":"        KURYR_ENABLED_HANDLERS: vif,endpoints,service,namespace,pod_label,policy,kuryrnetworkpolicy,kuryrnetwork,kuryrport,kuryrloadbalancer"},{"line_number":47,"context_line":"        KURYR_SG_DRIVER: policy"},{"line_number":48,"context_line":"        KURYR_SUBNET_DRIVER: namespace"},{"line_number":49,"context_line":"      devstack_services:"},{"line_number":50,"context_line":"        c-api: false"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"4aff8346_c705219c","line":47,"range":{"start_line":47,"start_character":8,"end_line":47,"end_character":31},"in_reply_to":"b3fa641e_77eba774","updated":"2021-09-09 10:11:54.000000000","message":"ok, that\u0027s a valid point. I can create one setup with default values.","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"52742e6743a52ce1447bd776f9ca865edcde44c1","unresolved":true,"context_lines":[{"line_number":97,"context_line":"        devstack_plugins:"},{"line_number":98,"context_line":"          neutron: https://opendev.org/openstack/neutron"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      # Default swap size got shrinked to 1 GB, it\u0027s way too small for us."},{"line_number":101,"context_line":"      network_api_extensions_common:"},{"line_number":102,"context_line":"        - tag-ports-during-bulk-creation"},{"line_number":103,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"190f14c0_c6f6defb","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":74},"updated":"2021-09-14 16:13:08.000000000","message":"This is not related to here, right?","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"06f61a57388f8c0c3c43ed15f33116852fabf71b","unresolved":true,"context_lines":[{"line_number":97,"context_line":"        devstack_plugins:"},{"line_number":98,"context_line":"          neutron: https://opendev.org/openstack/neutron"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      # Default swap size got shrinked to 1 GB, it\u0027s way too small for us."},{"line_number":101,"context_line":"      network_api_extensions_common:"},{"line_number":102,"context_line":"        - tag-ports-during-bulk-creation"},{"line_number":103,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"0f99dc95_3be56d3b","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":74},"in_reply_to":"190f14c0_c6f6defb","updated":"2021-09-15 08:29:02.000000000","message":"yeah. I\u0027ll move the comment to the right place.","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"838b3c51d22e9e3b75689f03b94002fb475abe9c","unresolved":true,"context_lines":[{"line_number":97,"context_line":"        devstack_plugins:"},{"line_number":98,"context_line":"          neutron: https://opendev.org/openstack/neutron"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      # Default swap size got shrinked to 1 GB, it\u0027s way too small for us."},{"line_number":101,"context_line":"      network_api_extensions_common:"},{"line_number":102,"context_line":"        - tag-ports-during-bulk-creation"},{"line_number":103,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":76,"id":"ac4fd7ad_65ad0fb6","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":74},"updated":"2021-09-15 14:01:36.000000000","message":"This is still irrelevant.","commit_id":"3d647fb16ac05a129d7f238572cb85fd87bfbf21"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"e7620a269285a9391fa7cfce17475c6ec19788de","unresolved":true,"context_lines":[{"line_number":97,"context_line":"        devstack_plugins:"},{"line_number":98,"context_line":"          neutron: https://opendev.org/openstack/neutron"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      # Default swap size got shrinked to 1 GB, it\u0027s way too small for us."},{"line_number":101,"context_line":"      network_api_extensions_common:"},{"line_number":102,"context_line":"        - tag-ports-during-bulk-creation"},{"line_number":103,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":76,"id":"805fe06d_f4b257eb","line":100,"range":{"start_line":100,"start_character":0,"end_line":100,"end_character":74},"in_reply_to":"ac4fd7ad_65ad0fb6","updated":"2021-09-15 14:13:43.000000000","message":"ouch.","commit_id":"3d647fb16ac05a129d7f238572cb85fd87bfbf21"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"1c42cee498ad5a30cddece05950d1e7f1cf78013","unresolved":true,"context_lines":[{"line_number":89,"context_line":"    parent: kuryr-kubernetes-base"},{"line_number":90,"context_line":"    description: Base kuryr-kubernetes-job with OVN"},{"line_number":91,"context_line":"    required-projects:"},{"line_number":92,"context_line":"      - openstack/neutron"},{"line_number":93,"context_line":"    timeout: 10800"},{"line_number":94,"context_line":"    post-run: playbooks/copy-k8s-logs.yaml"},{"line_number":95,"context_line":"    host-vars:"}],"source_content_type":"text/x-yaml","patch_set":77,"id":"24c5aa37_af66e783","line":92,"updated":"2021-09-15 17:29:13.000000000","message":"We probably need this here as well:\n\n      - openstack/octavia-tempest-plugin","commit_id":"5cfe79f6c8c2916ea07e228e5d41a0fa3b095c47"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"55dbab507be55c67939e003850ea09855b487f96","unresolved":true,"context_lines":[{"line_number":89,"context_line":"    parent: kuryr-kubernetes-base"},{"line_number":90,"context_line":"    description: Base kuryr-kubernetes-job with OVN"},{"line_number":91,"context_line":"    required-projects:"},{"line_number":92,"context_line":"      - openstack/neutron"},{"line_number":93,"context_line":"    timeout: 10800"},{"line_number":94,"context_line":"    post-run: playbooks/copy-k8s-logs.yaml"},{"line_number":95,"context_line":"    host-vars:"}],"source_content_type":"text/x-yaml","patch_set":77,"id":"2de6487e_ad9dc3a7","line":92,"in_reply_to":"24c5aa37_af66e783","updated":"2021-09-16 05:13:21.000000000","message":"Actually - no. It is needed only when we pull octavia bits, which takes a place in kuryr-kubernetes-octavia-base job - which is ovn based, and add octavia load balancer support - ovn-provider by default for Neutron OVN, and amphora by default for Neutron OVS.\n\nSo the idea here was to differentiate between kuryr being based on Neutron in two variants: OVN and OVS, while two other jobs with octavia in the name are adding loadbalancer to those two variants: ovn-provider and amphora respectively.\n\nThe reason behind it was to support Neutron OVN + ovn-provider or amphora, which is pretty simple, as adding Octavia dependencies to kuryr-kubernetes-base-ovn (see the hierarchy on the commit message). \n\nI guess, that one of the base job can be removed (merge kuryr-kubernetes-base-ovs + kuryr-kubernetes-octavia-base-ovs), since there is no further use for kuryr-kubernetes-base-ovs. As for OVN - I guess having jobs for neutron and octavia separately makes relatively easy to define jobs which have either ovn-provider or amphora as an dependency.\n\nThoughts?","commit_id":"5cfe79f6c8c2916ea07e228e5d41a0fa3b095c47"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"e81fc4251114f0656ea5bcc4b48ae60a3ec5d36a","unresolved":true,"context_lines":[{"line_number":89,"context_line":"    parent: kuryr-kubernetes-base"},{"line_number":90,"context_line":"    description: Base kuryr-kubernetes-job with OVN"},{"line_number":91,"context_line":"    required-projects:"},{"line_number":92,"context_line":"      - openstack/neutron"},{"line_number":93,"context_line":"    timeout: 10800"},{"line_number":94,"context_line":"    post-run: playbooks/copy-k8s-logs.yaml"},{"line_number":95,"context_line":"    host-vars:"}],"source_content_type":"text/x-yaml","patch_set":77,"id":"53a2d3de_32e27ab2","line":92,"in_reply_to":"2de6487e_ad9dc3a7","updated":"2021-09-16 08:31:22.000000000","message":"Alright, so it\u0027s probably needed at kuryr-kubernetes-tempest-amphora to prevent it from failing.","commit_id":"5cfe79f6c8c2916ea07e228e5d41a0fa3b095c47"}],".zuul.d/project.yaml":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"52742e6743a52ce1447bd776f9ca865edcde44c1","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        - kuryr-kubernetes-e2e-np"},{"line_number":30,"context_line":"    gate:"},{"line_number":31,"context_line":"      jobs:"},{"line_number":32,"context_line":"        - kuryr-kubernetes-tempest-systemd"},{"line_number":33,"context_line":"        - kuryr-kubernetes-tempest"},{"line_number":34,"context_line":"    experimental:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - kuryr-kubernetes-tempest-l2"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"3743a0dd_93207f1c","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":34},"updated":"2021-09-14 16:13:08.000000000","message":"Okay, I\u0027m sure we want one of the Amphora jobs here. I\u0027d vote for the OVN one.","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"acee5694cb2fb54e0602b44234893a90a0a20685","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        - kuryr-kubernetes-e2e-np"},{"line_number":30,"context_line":"    gate:"},{"line_number":31,"context_line":"      jobs:"},{"line_number":32,"context_line":"        - kuryr-kubernetes-tempest-systemd"},{"line_number":33,"context_line":"        - kuryr-kubernetes-tempest"},{"line_number":34,"context_line":"    experimental:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - kuryr-kubernetes-tempest-l2"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"913b9db1_1d6cec3d","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":34},"in_reply_to":"27b9804a_46110261","updated":"2021-09-15 09:54:08.000000000","message":"Ack","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"06f61a57388f8c0c3c43ed15f33116852fabf71b","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        - kuryr-kubernetes-e2e-np"},{"line_number":30,"context_line":"    gate:"},{"line_number":31,"context_line":"      jobs:"},{"line_number":32,"context_line":"        - kuryr-kubernetes-tempest-systemd"},{"line_number":33,"context_line":"        - kuryr-kubernetes-tempest"},{"line_number":34,"context_line":"    experimental:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - kuryr-kubernetes-tempest-l2"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"a956eebf_1576ae53","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":34},"in_reply_to":"3743a0dd_93207f1c","updated":"2021-09-15 08:29:02.000000000","message":"Note, that kk-tempest and -tempest-systemd both run on OVN and OVN provider. The only jobs, which are run with OVS have \"ovs\" in their names. With this patch there would be only 3 of them - ipv6, multinode and plain ovs.\n\nPerhaps I should omit \"ovn\" part in amphora test, to have it clear (or add \"ovn\" for every job name).","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"fa8b57a63eb9a14a2e263d66655ec36827fc818b","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        - kuryr-kubernetes-e2e-np"},{"line_number":30,"context_line":"    gate:"},{"line_number":31,"context_line":"      jobs:"},{"line_number":32,"context_line":"        - kuryr-kubernetes-tempest-systemd"},{"line_number":33,"context_line":"        - kuryr-kubernetes-tempest"},{"line_number":34,"context_line":"    experimental:"},{"line_number":35,"context_line":"      jobs:"},{"line_number":36,"context_line":"        - kuryr-kubernetes-tempest-l2"}],"source_content_type":"text/x-yaml","patch_set":74,"id":"27b9804a_46110261","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":34},"in_reply_to":"a956eebf_1576ae53","updated":"2021-09-15 08:45:21.000000000","message":"You\u0027re right, no need to say \"ovn\" on amphora job. Anyway having an amphora job on the gate pipeline is beneficial, this is where CI would protect from a combination of two independent patches breaking support for Amphora. And Amphora has totally different code paths, so it\u0027s important to exercise them as well.","commit_id":"46c0053bc24b0d50cd0cf3f3538ed708df27dbb4"}],".zuul.d/tempest-jobs.yaml":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"066993042f8cd56965d35c6b6b4f60935205712d","unresolved":true,"context_lines":[{"line_number":184,"context_line":"        o-cw: true"},{"line_number":185,"context_line":"        o-hk: true"},{"line_number":186,"context_line":"        o-hm: true"},{"line_number":187,"context_line":"    voting: false"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"- job:"},{"line_number":190,"context_line":"    name: kuryr-kubernetes-tempest-ovs-amphora"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"f452c090_500c6ce0","line":187,"range":{"start_line":187,"start_character":4,"end_line":187,"end_character":17},"updated":"2021-09-07 20:53:54.000000000","message":"should it be set to voting true as ovn is the default?","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"bb2f9c817a319bb6457a0deab1118f15dc9b7b9b","unresolved":true,"context_lines":[{"line_number":184,"context_line":"        o-cw: true"},{"line_number":185,"context_line":"        o-hk: true"},{"line_number":186,"context_line":"        o-hm: true"},{"line_number":187,"context_line":"    voting: false"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"- job:"},{"line_number":190,"context_line":"    name: kuryr-kubernetes-tempest-ovs-amphora"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"69e377d3_ac83c5e6","line":187,"range":{"start_line":187,"start_character":4,"end_line":187,"end_character":17},"in_reply_to":"f452c090_500c6ce0","updated":"2021-09-09 10:11:54.000000000","message":"I\u0027d leave it as is, as we have octavia-ovn-provider the default as well.","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"066993042f8cd56965d35c6b6b4f60935205712d","unresolved":true,"context_lines":[{"line_number":187,"context_line":"    voting: false"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"- job:"},{"line_number":190,"context_line":"    name: kuryr-kubernetes-tempest-ovs-amphora"},{"line_number":191,"context_line":"    parent: kuryr-kubernetes-octavia-base-ovs"},{"line_number":192,"context_line":"    vars:"},{"line_number":193,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"97ed1823_684d7071","line":190,"range":{"start_line":190,"start_character":27,"end_line":190,"end_character":46},"updated":"2021-09-07 20:53:54.000000000","message":"is tempest-ovn-provider-ovn being missed?","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"bb2f9c817a319bb6457a0deab1118f15dc9b7b9b","unresolved":true,"context_lines":[{"line_number":187,"context_line":"    voting: false"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"- job:"},{"line_number":190,"context_line":"    name: kuryr-kubernetes-tempest-ovs-amphora"},{"line_number":191,"context_line":"    parent: kuryr-kubernetes-octavia-base-ovs"},{"line_number":192,"context_line":"    vars:"},{"line_number":193,"context_line":"      devstack_localrc:"}],"source_content_type":"text/x-yaml","patch_set":70,"id":"81221ba6_37a42c4c","line":190,"range":{"start_line":190,"start_character":27,"end_line":190,"end_character":46},"in_reply_to":"97ed1823_684d7071","updated":"2021-09-09 10:11:54.000000000","message":"OVN and octavia-ovn-provider is the default now, so this setup covers amphora with OVS, while the previous job (on line 155) is for tests OVN with amphora.","commit_id":"8ec8090c11b30556a96a02b8ad8f95dde32a4762"}]}