)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5def716977315a30f27c2342a148c72c9f788681","unresolved":true,"context_lines":[{"line_number":95,"context_line":"    name: os-vif-ovs-iptables"},{"line_number":96,"context_line":"    parent: os-vif-ovs-base"},{"line_number":97,"context_line":"    description: |"},{"line_number":98,"context_line":"      os-vif iptables job derived from neutron-ovs-tempest-iptables_hybrid"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      tox_envlist: full"},{"line_number":101,"context_line":"      devstack_local_conf:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"db3f59bd_c80229fa","line":98,"range":{"start_line":98,"start_character":26,"end_line":98,"end_character":33},"updated":"2021-06-23 17:26:13.000000000","message":"derived form does not mean it inherits form it we created our own copy so that change to the neutorn job did not afffect the os-vif job so technially this was correct before.\n\nmaybe we shoul just remove the refernce to the neutorn jobs since its not really avlid anymore","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7f7990489f89f4150fd1a8f4a4eda59719506ae1","unresolved":true,"context_lines":[{"line_number":95,"context_line":"    name: os-vif-ovs-iptables"},{"line_number":96,"context_line":"    parent: os-vif-ovs-base"},{"line_number":97,"context_line":"    description: |"},{"line_number":98,"context_line":"      os-vif iptables job derived from neutron-ovs-tempest-iptables_hybrid"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      tox_envlist: full"},{"line_number":101,"context_line":"      devstack_local_conf:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"83d3dbd0_3bc0b09e","line":98,"range":{"start_line":98,"start_character":26,"end_line":98,"end_character":33},"in_reply_to":"db3f59bd_c80229fa","updated":"2021-06-23 17:44:36.000000000","message":"And that\u0027s ok, I\u0027m not saying that you are inheriting from those jobs. I\u0027m just updating the reference.","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"300c849d829b0a9ef42f8859d50755ea3482fe40","unresolved":true,"context_lines":[{"line_number":95,"context_line":"    name: os-vif-ovs-iptables"},{"line_number":96,"context_line":"    parent: os-vif-ovs-base"},{"line_number":97,"context_line":"    description: |"},{"line_number":98,"context_line":"      os-vif iptables job derived from neutron-ovs-tempest-iptables_hybrid"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      tox_envlist: full"},{"line_number":101,"context_line":"      devstack_local_conf:"},{"line_number":102,"context_line":"        post-config:"},{"line_number":103,"context_line":"          # NOTE(sean-k-mooney): i do not believe that the devstack role"},{"line_number":104,"context_line":"          # will merge the base /$NEUTRON_CORE_PLUGIN_CONF with the parent"},{"line_number":105,"context_line":"          # job so we redefine the entire section"},{"line_number":106,"context_line":"          \"/$NEUTRON_CORE_PLUGIN_CONF\":"},{"line_number":107,"context_line":"            ml2_type_vlan:"},{"line_number":108,"context_line":"              network_vlan_ranges: foo:1:10"},{"line_number":109,"context_line":"            agent:"},{"line_number":110,"context_line":"              enable_distributed_routing: true"},{"line_number":111,"context_line":"              l2_population: true"},{"line_number":112,"context_line":"              tunnel_types: vxlan,gre"},{"line_number":113,"context_line":"            securitygroup:"},{"line_number":114,"context_line":"              firewall_driver: iptables_hybrid"},{"line_number":115,"context_line":"- job:"},{"line_number":116,"context_line":"    name: os-vif-linuxbridge"},{"line_number":117,"context_line":"    parent: os-vif-tempest-base"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f8089669_ce2400c0","line":114,"range":{"start_line":98,"start_character":4,"end_line":114,"end_character":46},"updated":"2021-06-23 17:22:11.000000000","message":"we have not fixed this since the change to ovn by defualt\n\nthis shoudl be using ml2/ovs but it is using ovn.\n\nwe can see that form the lack of neutron agents in \nhttps://zuul.opendev.org/t/openstack/build/380f6bfba86d41818f6a0b6ae5b40c93/logs","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7f7990489f89f4150fd1a8f4a4eda59719506ae1","unresolved":true,"context_lines":[{"line_number":95,"context_line":"    name: os-vif-ovs-iptables"},{"line_number":96,"context_line":"    parent: os-vif-ovs-base"},{"line_number":97,"context_line":"    description: |"},{"line_number":98,"context_line":"      os-vif iptables job derived from neutron-ovs-tempest-iptables_hybrid"},{"line_number":99,"context_line":"    vars:"},{"line_number":100,"context_line":"      tox_envlist: full"},{"line_number":101,"context_line":"      devstack_local_conf:"},{"line_number":102,"context_line":"        post-config:"},{"line_number":103,"context_line":"          # NOTE(sean-k-mooney): i do not believe that the devstack role"},{"line_number":104,"context_line":"          # will merge the base /$NEUTRON_CORE_PLUGIN_CONF with the parent"},{"line_number":105,"context_line":"          # job so we redefine the entire section"},{"line_number":106,"context_line":"          \"/$NEUTRON_CORE_PLUGIN_CONF\":"},{"line_number":107,"context_line":"            ml2_type_vlan:"},{"line_number":108,"context_line":"              network_vlan_ranges: foo:1:10"},{"line_number":109,"context_line":"            agent:"},{"line_number":110,"context_line":"              enable_distributed_routing: true"},{"line_number":111,"context_line":"              l2_population: true"},{"line_number":112,"context_line":"              tunnel_types: vxlan,gre"},{"line_number":113,"context_line":"            securitygroup:"},{"line_number":114,"context_line":"              firewall_driver: iptables_hybrid"},{"line_number":115,"context_line":"- job:"},{"line_number":116,"context_line":"    name: os-vif-linuxbridge"},{"line_number":117,"context_line":"    parent: os-vif-tempest-base"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"008a680b_cc87cf47","line":114,"range":{"start_line":98,"start_character":4,"end_line":114,"end_character":46},"in_reply_to":"f8089669_ce2400c0","updated":"2021-06-23 17:44:36.000000000","message":"out of scope to this patch","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5def716977315a30f27c2342a148c72c9f788681","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    parent: os-vif-tempest-base"},{"line_number":118,"context_line":"    timeout: 10800"},{"line_number":119,"context_line":"    description: |"},{"line_number":120,"context_line":"      os-vif linux bridge job derived from neutron-linuxbridge-tempest"},{"line_number":121,"context_line":"    vars:"},{"line_number":122,"context_line":"      devstack_localrc:"},{"line_number":123,"context_line":"        Q_AGENT: linuxbridge"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"8255e98f_665b6ea3","line":120,"range":{"start_line":120,"start_character":0,"end_line":120,"end_character":15},"updated":"2021-06-23 17:26:13.000000000","message":"same here this is not really correct we do not inhirt form that and the drived form coms for the fact it was used a a refrence but change to the nueton jobs have no effect on the os-vif ones intentionally.","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7f7990489f89f4150fd1a8f4a4eda59719506ae1","unresolved":true,"context_lines":[{"line_number":117,"context_line":"    parent: os-vif-tempest-base"},{"line_number":118,"context_line":"    timeout: 10800"},{"line_number":119,"context_line":"    description: |"},{"line_number":120,"context_line":"      os-vif linux bridge job derived from neutron-linuxbridge-tempest"},{"line_number":121,"context_line":"    vars:"},{"line_number":122,"context_line":"      devstack_localrc:"},{"line_number":123,"context_line":"        Q_AGENT: linuxbridge"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"83f0a8c4_e1a65660","line":120,"range":{"start_line":120,"start_character":0,"end_line":120,"end_character":15},"in_reply_to":"8255e98f_665b6ea3","updated":"2021-06-23 17:44:36.000000000","message":"same reply","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"}],"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"300c849d829b0a9ef42f8859d50755ea3482fe40","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make explicit the network backend used in the CI jobs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"All Neutron CI jobs (except for unit, functional and fullstack jobs),"},{"line_number":10,"context_line":"have explicitly defined the network backend used:"},{"line_number":11,"context_line":"- linuxbridge"},{"line_number":12,"context_line":"- ovs"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"24e26191_42cc9e1f","line":9,"range":{"start_line":9,"start_character":4,"end_line":9,"end_character":15},"updated":"2021-06-23 17:22:11.000000000","message":"os-vif is not part of the neutron ci\nits part of nova so normally os-vif jobs should be dicussed in the nova team meeting on irc.","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7f7990489f89f4150fd1a8f4a4eda59719506ae1","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Make explicit the network backend used in the CI jobs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"All Neutron CI jobs (except for unit, functional and fullstack jobs),"},{"line_number":10,"context_line":"have explicitly defined the network backend used:"},{"line_number":11,"context_line":"- linuxbridge"},{"line_number":12,"context_line":"- ovs"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a709be93_c71fa989","line":9,"range":{"start_line":9,"start_character":4,"end_line":9,"end_character":15},"in_reply_to":"24e26191_42cc9e1f","updated":"2021-06-23 17:44:36.000000000","message":"I\u0027m not saying that, just updating a reference in your doc.","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"300c849d829b0a9ef42f8859d50755ea3482fe40","unresolved":true,"context_lines":[{"line_number":9,"context_line":"All Neutron CI jobs (except for unit, functional and fullstack jobs),"},{"line_number":10,"context_line":"have explicitly defined the network backend used:"},{"line_number":11,"context_line":"- linuxbridge"},{"line_number":12,"context_line":"- ovs"},{"line_number":13,"context_line":"- ovn"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"That was discussed and approved during the Neutron CI meetings [1]."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"95aa9ba8_194e6348","line":12,"range":{"start_line":12,"start_character":2,"end_line":12,"end_character":5},"updated":"2021-06-23 17:22:11.000000000","message":"the iptable job which is ment to be ml2/ovs is still ovn since we have not fixed that\ncan you update it in this patch. -1 is for this","commit_id":"2489173c4a907e7014ffc69c8a03dfbdd8626577"}]}