)]}'
{"defaults/main.yml":[{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"733fc3a5f8b70631ec39e69bd2ddb21a6d0cc5bd","unresolved":false,"context_lines":[{"line_number":40,"context_line":"# This needs to coincide with tempest_network_tenant_network_cidr and"},{"line_number":41,"context_line":"# tempest_network_tenant_network_mask_bits below"},{"line_number":42,"context_line":"tempest_private_subnet_cidr: \"192.168.74.0/28\""},{"line_number":43,"context_line":"tempest_private_net_provider_type: \"vlan\""},{"line_number":44,"context_line":"tempest_private_net_sed_id: \"\""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# Public network configuration"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dab17558_0cf7a28b","line":43,"updated":"2016-05-11 10:53:17.000000000","message":"I believe this defaults to vxlan right now, right?  We should retain the default if at all possible.","commit_id":"6295444cbfacd0b2f5de8d42086cd364d753b5e9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"733fc3a5f8b70631ec39e69bd2ddb21a6d0cc5bd","unresolved":false,"context_lines":[{"line_number":41,"context_line":"# tempest_network_tenant_network_mask_bits below"},{"line_number":42,"context_line":"tempest_private_subnet_cidr: \"192.168.74.0/28\""},{"line_number":43,"context_line":"tempest_private_net_provider_type: \"vlan\""},{"line_number":44,"context_line":"tempest_private_net_sed_id: \"\""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# Public network configuration"},{"line_number":47,"context_line":"tempest_public_subnet_cidr: \"10.1.13.0/24\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dab17558_6cdbce1b","line":44,"updated":"2016-05-11 10:53:17.000000000","message":"Could this be `tempest_private_net_seg_id` instead?  The sed bit is confusing me.  :)","commit_id":"6295444cbfacd0b2f5de8d42086cd364d753b5e9"},{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"733fc3a5f8b70631ec39e69bd2ddb21a6d0cc5bd","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# Public network configuration"},{"line_number":47,"context_line":"tempest_public_subnet_cidr: \"10.1.13.0/24\""},{"line_number":48,"context_line":"tempest_public_net_provider_type: \"flat\""},{"line_number":49,"context_line":"tempest_public_met_physical_type: \"flat\""},{"line_number":50,"context_line":"tempest_public_net_seg_id: \"\""},{"line_number":51,"context_line":"tempest_public_router_external: \"true\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dab17558_ece7bea9","line":48,"updated":"2016-05-11 10:53:17.000000000","message":"Would be good to document the accepted types since the task file only actions flat or vlan.  Same for the private bit above.","commit_id":"6295444cbfacd0b2f5de8d42086cd364d753b5e9"}],"releasenotes/notes/neutron-network-variables-ff6d2c7f8c7c3ccd.yaml":[{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"8ef13669448963eb50caf456c0128a78bfdeccc3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    This release adds the ability to override the"},{"line_number":4,"context_line":"    neutron network provider, physical network, seg id"},{"line_number":5,"context_line":"    and router external variables to allow the user to"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dab17558_05aaab84","line":2,"updated":"2016-05-12 02:57:18.000000000","message":"We generally avoid putting anything in the \u0027prelude\u0027 section unless it\u0027s extremely critical in the release. You can probably move this into the \u0027features\u0027 section. (See next comment)","commit_id":"27440b0804b4f0b427650421e47c3bb72e088fc9"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"8ef13669448963eb50caf456c0128a78bfdeccc3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    creates and uses for tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"features:"},{"line_number":10,"context_line":"  - neutron network variable override"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"other:"},{"line_number":13,"context_line":"  - type of provider networks are flat and vlan"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dab17558_e5acc7a0","line":10,"updated":"2016-05-12 02:57:18.000000000","message":"Could you write a sentence that explains the new feature and how someone could use it?  Here\u0027s an example you can follow:\n\nhttps://github.com/openstack/openstack-ansible/blob/master/releasenotes/notes/neutron-fwaas-5c7c6508f2cc05c3.yaml\n\nYou could say something to the effect of: \"Deployers can now choose to run tempest tests with public and private networks by changing VARIABLE_NAME to VALUE.\"","commit_id":"27440b0804b4f0b427650421e47c3bb72e088fc9"},{"author":{"_account_id":538,"name":"Major Hayden","email":"major@mhtx.net","username":"rackerhacker"},"change_message_id":"8ef13669448963eb50caf456c0128a78bfdeccc3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"features:"},{"line_number":10,"context_line":"  - neutron network variable override"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"other:"},{"line_number":13,"context_line":"  - type of provider networks are flat and vlan"},{"line_number":14,"context_line":"  - type of tenant networks are vlan and vxlan"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dab17558_45983378","line":12,"updated":"2016-05-12 02:57:18.000000000","message":"All of this could probably go into the \u0027features\u0027 section. (see previous comment)","commit_id":"27440b0804b4f0b427650421e47c3bb72e088fc9"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d2361cc82eea31f7b88e484b080b8e8f148e2c27","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"other:"},{"line_number":13,"context_line":"  - type of provider networks are flat and vlan"},{"line_number":14,"context_line":"  - type of tenant networks are vlan and vxlan"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dab17558_ffeba276","line":14,"updated":"2016-05-12 11:23:36.000000000","message":"Each list item must make sense to read without the context of the patch. To see why take a look at http://docs.openstack.org/releasenotes/openstack-ansible/mitaka.html and see how the releasenotes come out as a result.\n\nThe prelude section should only be used for major feature announcements and is typically only put together by the PTL, so please don\u0027t use that. For most use cases the sections you need to care about are \u0027features\u0027 for any new feature/functionality and \u0027upgrade\u0027 for any default behaviour changes.","commit_id":"27440b0804b4f0b427650421e47c3bb72e088fc9"},{"author":{"_account_id":18784,"name":"Michael Carden","email":"mike.carden@gmail.com","username":"mcarden"},"change_message_id":"b9d61f7610a0715a828e1a4f47ace399a9df8379","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - Deployers can now configure tempest public and private networks by setting"},{"line_number":4,"context_line":"    the following variables, tempest_private_net_provider_type to either vxlan"},{"line_number":5,"context_line":"    or vlan and tempest_public_net_provider_type to flat or vlan. Depending on"},{"line_number":6,"context_line":"    what the deployer sets these variables to the may also need to update other"},{"line_number":7,"context_line":"    variables accordingly, this mainly involves tempest_public_net_physical_type"},{"line_number":8,"context_line":"    and tempest_public_net_seg_id. Please refer to"},{"line_number":9,"context_line":"    http://docs.openstack.org/mitaka/networking-guide/intro-basic-networking.html"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"dab17558_7a031677","line":6,"updated":"2016-05-13 04:55:24.000000000","message":"\u0027they\u0027","commit_id":"79bae6da3b1ea63d395a5d23def5e0589478b4e7"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"76abe19446d744efa4c7342ac8a57bb74d3f930f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - Deployers can now configure tempest public and private networks by setting"},{"line_number":4,"context_line":"    the following variables, tempest_private_net_provider_type to either vxlan"},{"line_number":5,"context_line":"    or vlan and tempest_public_net_provider_type to flat or vlan. Depending on"},{"line_number":6,"context_line":"    what the deployer sets these variables to the may also need to update other"},{"line_number":7,"context_line":"    variables accordingly, this mainly involves tempest_public_net_physical_type"},{"line_number":8,"context_line":"    and tempest_public_net_seg_id. Please refer to"},{"line_number":9,"context_line":"    http://docs.openstack.org/mitaka/networking-guide/intro-basic-networking.html"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"dab17558_32eb697a","line":6,"in_reply_to":"dab17558_7a031677","updated":"2016-05-13 10:38:33.000000000","message":"Done","commit_id":"79bae6da3b1ea63d395a5d23def5e0589478b4e7"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"a7f45225c5847f0c26563ae682af96b672a053f1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Deployers can now configure tempest public and private networks by setting"},{"line_number":4,"context_line":"    the following variables, tempest_private_net_provider_type to either vxlan"},{"line_number":5,"context_line":"    or vlan and tempest_public_net_provider_type to flat or vlan. Depending on"},{"line_number":6,"context_line":"    what the deployer sets these variables to, they may also need to update other"},{"line_number":7,"context_line":"    variables accordingly, this mainly involves tempest_public_net_physical_type"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dab17558_282bdf95","line":4,"range":{"start_line":4,"start_character":29,"end_line":4,"end_character":62},"updated":"2016-05-13 18:32:31.000000000","message":"surround with ``","commit_id":"a9932853d75b228dbdb64404ec87fd2e8edcacfe"},{"author":{"_account_id":19814,"name":"Travis Truman (automagically)","email":"travis_truman@cable.comcast.com","username":"trumant"},"change_message_id":"a7f45225c5847f0c26563ae682af96b672a053f1","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Deployers can now configure tempest public and private networks by setting"},{"line_number":4,"context_line":"    the following variables, tempest_private_net_provider_type to either vxlan"},{"line_number":5,"context_line":"    or vlan and tempest_public_net_provider_type to flat or vlan. Depending on"},{"line_number":6,"context_line":"    what the deployer sets these variables to, they may also need to update other"},{"line_number":7,"context_line":"    variables accordingly, this mainly involves tempest_public_net_physical_type"},{"line_number":8,"context_line":"    and tempest_public_net_seg_id. Please refer to"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dab17558_c8121362","line":5,"range":{"start_line":5,"start_character":16,"end_line":5,"end_character":48},"updated":"2016-05-13 18:32:31.000000000","message":"same as above","commit_id":"a9932853d75b228dbdb64404ec87fd2e8edcacfe"}],"tasks/tempest_resources.yml":[{"author":{"_account_id":7307,"name":"Matt Thompson","email":"mattt@defunct.ca","username":"mattt416"},"change_message_id":"733fc3a5f8b70631ec39e69bd2ddb21a6d0cc5bd","unresolved":false,"context_lines":[{"line_number":222,"context_line":"    openrc_path: /root/openrc"},{"line_number":223,"context_line":"    net_name: private"},{"line_number":224,"context_line":"    provider_network_type: \"{{ tempest_private_net_provider_type }}\""},{"line_number":225,"context_line":"    provider_segmentation_id: \"{{ tempest_private_net_sed_id }}\""},{"line_number":226,"context_line":"    tenant_id: \"{{ keystone_demo_tenant_id }}\""},{"line_number":227,"context_line":"    insecure: \"{{ keystone_service_internaluri_insecure }}\""},{"line_number":228,"context_line":"  when: (tempest_service_available_neutron | bool  and tempest_private_net_provider_type \u003d\u003d \"vxlan\")"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dab17558_0c8842c4","line":225,"updated":"2016-05-11 10:53:17.000000000","message":"What happens if you specify provider_segmentation_id for type vlan?  I\u0027m just wondering if we can get away with only having one task irrespective of provider_type.  :)","commit_id":"6295444cbfacd0b2f5de8d42086cd364d753b5e9"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"f1411c40e3a4f8d72d15ac082a2c4b695be012d2","unresolved":false,"context_lines":[{"line_number":234,"context_line":"    provider_physical_network: \"{{ tempest_public_net_physical_type }}\""},{"line_number":235,"context_line":"    router_external: \"{{ tempest_public_router_external }}\""},{"line_number":236,"context_line":"    insecure: \"{{ keystone_service_internaluri_insecure }}\""},{"line_number":237,"context_line":"  when: (tempest_service_available_neutron | bool and tempest_public_net_provider_type \u003d\u003d \"flat\")"},{"line_number":238,"context_line":"  tags:"},{"line_number":239,"context_line":"    - tempest-setup"},{"line_number":240,"context_line":"    - tempest-config"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"bab6814e_46daeb79","line":237,"range":{"start_line":237,"start_character":0,"end_line":237,"end_character":97},"updated":"2016-05-18 08:48:00.000000000","message":"Can you make this vertically formatted for easier readability, eg:\n\n  when:\n    - tempest_service_available_neutron | bool\n    - tempest_public_net_provider_type \u003d\u003d \"flat\"","commit_id":"c21ea2b0feeaddcb8bae9224b5cb120812d0b361"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"f1411c40e3a4f8d72d15ac082a2c4b695be012d2","unresolved":false,"context_lines":[{"line_number":250,"context_line":"    router_external: \"{{ tempest_public_router_external }}\""},{"line_number":251,"context_line":"    tenant_id: \"{{ keystone_demo_tenant_id }}\""},{"line_number":252,"context_line":"    insecure: \"{{ keystone_service_internaluri_insecure }}\""},{"line_number":253,"context_line":"  when: (tempest_service_available_neutron | bool and tempest_public_net_provider_type \u003d\u003d \"vlan\")"},{"line_number":254,"context_line":"  tags:"},{"line_number":255,"context_line":"    - tempest-setup"},{"line_number":256,"context_line":"    - tempest-config"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"bab6814e_46c14b80","line":253,"range":{"start_line":253,"start_character":2,"end_line":253,"end_character":97},"updated":"2016-05-18 08:48:00.000000000","message":"Can you make this vertically formatted for easier readability, eg:\n\n  when:\n    - tempest_service_available_neutron | bool\n    - tempest_public_net_provider_type \u003d\u003d \"vlan\"","commit_id":"c21ea2b0feeaddcb8bae9224b5cb120812d0b361"}]}