)]}'
{"specs/neutron-provider-networking.rst":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"9cbafcce75acc79efac65a444b128326935bdb8d","unresolved":false,"context_lines":[{"line_number":20,"context_line":"agent and do not include any tests for Provider Networking API extension."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"This BP would add the necessary Provider Networking Scenario tests for IPv4"},{"line_number":23,"context_line":"and/or IPv6 subnets."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_d77bd3bb","line":23,"updated":"2014-07-07 01:27:09.000000000","message":"Are IPv6 subnets mandatory for Provider Networking scenario tests?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":20,"context_line":"agent and do not include any tests for Provider Networking API extension."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"This BP would add the necessary Provider Networking Scenario tests for IPv4"},{"line_number":23,"context_line":"and/or IPv6 subnets."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed change"},{"line_number":26,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_2fef241f","line":23,"in_reply_to":"baada198_d77bd3bb","updated":"2014-07-08 10:08:09.000000000","message":"@Masayuki, thanks for the review. \n\nIts not mandatory to have IPv6 subnets in the provider network. We can create the provider network using IPv4 subnet alone.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"da0507daf25a58b5e3629f1537dd48cfc5fbd26f","unresolved":false,"context_lines":[{"line_number":51,"context_line":"- Devstack configures the provider network id in tempest.conf file"},{"line_number":52,"context_line":"  (i.e., CONF.network.provider_network_id)"},{"line_number":53,"context_line":"  Note: provider network tests would be run only when provider_network_id is"},{"line_number":54,"context_line":"  set and would be skipped otherwise."},{"line_number":55,"context_line":"- It is also assumed that provider network is reachable from tempest host."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_5605fa11","line":54,"updated":"2014-07-07 10:44:09.000000000","message":"Can you add a not here about the behaviour with icehouse branch?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":51,"context_line":"- Devstack configures the provider network id in tempest.conf file"},{"line_number":52,"context_line":"  (i.e., CONF.network.provider_network_id)"},{"line_number":53,"context_line":"  Note: provider network tests would be run only when provider_network_id is"},{"line_number":54,"context_line":"  set and would be skipped otherwise."},{"line_number":55,"context_line":"- It is also assumed that provider network is reachable from tempest host."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_2fcd6431","line":54,"in_reply_to":"baada198_5605fa11","updated":"2014-07-08 10:08:09.000000000","message":"@Andrea, thanks for the review.\n\nAs you know, devstack has branches and while running on stable/icehouse branch it uses the icehouse branch of devstack. The changes in devstack mentioned as part of this BP would be done only in the latest devstack code, so for the icehouse builds these tests would not be run.\n\nI\u0027ll update the patch with this note.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"da0507daf25a58b5e3629f1537dd48cfc5fbd26f","unresolved":false,"context_lines":[{"line_number":44,"context_line":"  default."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Assumptions/Expectations from devstack:"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"- It is expected that devstack creates the necessary Provider Network (either a"},{"line_number":49,"context_line":"  flat or VLAN based), IPv4 and/or IPv6 subnets and associates the subnets to the"},{"line_number":50,"context_line":"  provider network."},{"line_number":51,"context_line":"- Devstack configures the provider network id in tempest.conf file"},{"line_number":52,"context_line":"  (i.e., CONF.network.provider_network_id)"},{"line_number":53,"context_line":"  Note: provider network tests would be run only when provider_network_id is"},{"line_number":54,"context_line":"  set and would be skipped otherwise."},{"line_number":55,"context_line":"- It is also assumed that provider network is reachable from tempest host."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_b666564f","line":55,"range":{"start_line":47,"start_character":0,"end_line":55,"end_character":74},"updated":"2014-07-07 10:44:09.000000000","message":"It is possible to setup neutron to provide the usual L3 agent and in the same deployment also support a provider network?\nI guess my question really is, does this require a dedicate dsvm job, or would it run as part of the existing neutron one?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":52,"context_line":"  (i.e., CONF.network.provider_network_id)"},{"line_number":53,"context_line":"  Note: provider network tests would be run only when provider_network_id is"},{"line_number":54,"context_line":"  set and would be skipped otherwise."},{"line_number":55,"context_line":"- It is also assumed that provider network is reachable from tempest host."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_efde0cd9","line":55,"in_reply_to":"baada198_b666564f","updated":"2014-07-08 10:08:09.000000000","message":"AFAIU it runs in the existing neutron one and does not require any dedicate dvsm job.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"9cbafcce75acc79efac65a444b128326935bdb8d","unresolved":false,"context_lines":[{"line_number":75,"context_line":"----------"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Target Milestone for completion:"},{"line_number":78,"context_line":"  Juno release"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Work Items"},{"line_number":81,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_b7e49fac","line":78,"updated":"2014-07-07 01:27:09.000000000","message":"I just want to clarify here. Do you mean the target is around October 16?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":75,"context_line":"----------"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Target Milestone for completion:"},{"line_number":78,"context_line":"  Juno release"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"Work Items"},{"line_number":81,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_8fb2f091","line":78,"in_reply_to":"baada198_b7e49fac","updated":"2014-07-08 10:08:09.000000000","message":"Yes, as we have some dependencies the idea is to get this work incorporated in Juno release. If the dependencies are addressed quickly, we can target some earlier date.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"da0507daf25a58b5e3629f1537dd48cfc5fbd26f","unresolved":false,"context_lines":[{"line_number":80,"context_line":"Work Items"},{"line_number":81,"context_line":"----------"},{"line_number":82,"context_line":"The work items are implemented as part of the following patch"},{"line_number":83,"context_line":"https://review.openstack.org/#/c/100143"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"- Implement the required scenario tests as part of TestProviderNetworkBasicOps"},{"line_number":86,"context_line":"  class."}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_19abc789","line":83,"updated":"2014-07-07 10:44:09.000000000","message":"I would not include a reference to a specific change in the work items. I may find out that you need more than one change for instance.\nIf you really want to include a gerrit link here, you could use a gerrit search for all changes with the topic pointing to this bp.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":80,"context_line":"Work Items"},{"line_number":81,"context_line":"----------"},{"line_number":82,"context_line":"The work items are implemented as part of the following patch"},{"line_number":83,"context_line":"https://review.openstack.org/#/c/100143"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"- Implement the required scenario tests as part of TestProviderNetworkBasicOps"},{"line_number":86,"context_line":"  class."}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_af7414cf","line":83,"in_reply_to":"baada198_19abc789","updated":"2014-07-08 10:08:09.000000000","message":"Good suggestion, thanks.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"9cbafcce75acc79efac65a444b128326935bdb8d","unresolved":false,"context_lines":[{"line_number":100,"context_line":"  devstack does not include ping6 utility. A bug needs to be raised aganist"},{"line_number":101,"context_line":"  Cirros to include ping6 utility."},{"line_number":102,"context_line":"  Note: Provider Networking scenario tests would check for the presence of"},{"line_number":103,"context_line":"  the utility and would execute the ping6 test only when present."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- Python Neutron Client: Create new IPv6 attributes for Subnets by client"},{"line_number":106,"context_line":"  https://review.openstack.org/#/c/75871/"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_97879bb4","line":103,"updated":"2014-07-07 01:27:09.000000000","message":"Does this mean we can\u0027t run this provider networking test in the gate, right?\nIf so, do you have any plan or idea to install ping6 utility into cirros?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":100,"context_line":"  devstack does not include ping6 utility. A bug needs to be raised aganist"},{"line_number":101,"context_line":"  Cirros to include ping6 utility."},{"line_number":102,"context_line":"  Note: Provider Networking scenario tests would check for the presence of"},{"line_number":103,"context_line":"  the utility and would execute the ping6 test only when present."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- Python Neutron Client: Create new IPv6 attributes for Subnets by client"},{"line_number":106,"context_line":"  https://review.openstack.org/#/c/75871/"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_4a560607","line":103,"in_reply_to":"baada198_562edaa5","updated":"2014-07-08 10:08:09.000000000","message":"We will not be hard-coding the name/assuming anything in the code. We would check for the presence of the utility and run the tests if the utility is present, skipping otherwise. \n\nPlease have a look at the code Line 252.\nhttps://review.openstack.org/#/c/100143/4/tempest/scenario/test_provider_net_ops.py","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":100,"context_line":"  devstack does not include ping6 utility. A bug needs to be raised aganist"},{"line_number":101,"context_line":"  Cirros to include ping6 utility."},{"line_number":102,"context_line":"  Note: Provider Networking scenario tests would check for the presence of"},{"line_number":103,"context_line":"  the utility and would execute the ping6 test only when present."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- Python Neutron Client: Create new IPv6 attributes for Subnets by client"},{"line_number":106,"context_line":"  https://review.openstack.org/#/c/75871/"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_ef28cc98","line":103,"in_reply_to":"baada198_97879bb4","updated":"2014-07-08 10:08:09.000000000","message":"The provider network that is created in Devstack can have IPv4 subnet alone or an IPv6 subnet alone or a combination of IPv4 and IPv6 subnets. \n\nWhen an IPv4 subnet is present along with IPv6 subnet, IPv4 tests would still be run. Only IPv6 ping6 test would be skipped in such cases.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"da0507daf25a58b5e3629f1537dd48cfc5fbd26f","unresolved":false,"context_lines":[{"line_number":100,"context_line":"  devstack does not include ping6 utility. A bug needs to be raised aganist"},{"line_number":101,"context_line":"  Cirros to include ping6 utility."},{"line_number":102,"context_line":"  Note: Provider Networking scenario tests would check for the presence of"},{"line_number":103,"context_line":"  the utility and would execute the ping6 test only when present."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"- Python Neutron Client: Create new IPv6 attributes for Subnets by client"},{"line_number":106,"context_line":"  https://review.openstack.org/#/c/75871/"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_562edaa5","line":103,"in_reply_to":"baada198_97879bb4","updated":"2014-07-07 10:44:09.000000000","message":"Will you check the version of cirros? What if the test image is not cirros? Will you check for availability of ping6 in the image and else skip?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"da0507daf25a58b5e3629f1537dd48cfc5fbd26f","unresolved":false,"context_lines":[{"line_number":114,"context_line":"- Oslo: Utility API to generate EUI-64 IPv6 address. Implement an API in"},{"line_number":115,"context_line":"  Oslo which calculates the EUI-64 address based on IPv6 prefix and interface"},{"line_number":116,"context_line":"  MAC address. The following code in Neutron (neutron/common/ipv6_utils.py)"},{"line_number":117,"context_line":"  can be moved to oslo and synced in Tempest and Neutron."}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_76df5ed6","line":117,"updated":"2014-07-07 10:44:09.000000000","message":"Is this bp blocked by this, or can we carry a local function in tempest until it\u0027s available from oslo?","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"df5ce20b3e3f5c694caf1dee91674ab3ee70c5ce","unresolved":false,"context_lines":[{"line_number":114,"context_line":"- Oslo: Utility API to generate EUI-64 IPv6 address. Implement an API in"},{"line_number":115,"context_line":"  Oslo which calculates the EUI-64 address based on IPv6 prefix and interface"},{"line_number":116,"context_line":"  MAC address. The following code in Neutron (neutron/common/ipv6_utils.py)"},{"line_number":117,"context_line":"  can be moved to oslo and synced in Tempest and Neutron."}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_2aa1d2f7","line":117,"in_reply_to":"baada198_76df5ed6","updated":"2014-07-08 10:08:09.000000000","message":"In the current patch, I\u0027ve included the api as part of the manager code. \nOnce the changes in oslo are available, the api in the patch can be removed.","commit_id":"eceff89be6e804fc4cc50e45657b9a245d0b52e7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"534e462cbbb186e79b342b681eeae9a20c124c54","unresolved":false,"context_lines":[{"line_number":79,"context_line":"----------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Target Milestone for completion:"},{"line_number":82,"context_line":"  Juno release"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Work Items"},{"line_number":85,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1a930d6b_cb118c53","line":82,"updated":"2015-01-26 13:09:28.000000000","message":"Might want to update this if you have to re-spin","commit_id":"8ac713d788d1ef71c9148bc9865caacbfccc9ee7"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"9d7cca5d11ef5699881dce26d4007707f903387c","unresolved":false,"context_lines":[{"line_number":79,"context_line":"----------"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"Target Milestone for completion:"},{"line_number":82,"context_line":"  Juno release"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Work Items"},{"line_number":85,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"1a930d6b_7e3e94db","line":82,"in_reply_to":"1a930d6b_cb118c53","updated":"2015-01-26 13:51:46.000000000","message":"Sorry Andrea. I\u0027m currently looking into another requirement and will not be having the bandwidth to work on this BP. \n\nI\u0027m fine if someone takes over this work.","commit_id":"8ac713d788d1ef71c9148bc9865caacbfccc9ee7"}]}