)]}'
{"fixtures/scenario001.pp":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"322aa3e7c1e5095e8605a3bf72e43335256a74d7","unresolved":false,"context_lines":[{"line_number":222,"context_line":"  neutron_admin_auth_url \u003d\u003e \u0027http://127.0.0.1:35357/v2.0\u0027,"},{"line_number":223,"context_line":"}"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"# Configure Tempest and the resources"},{"line_number":226,"context_line":"$os_auth_options \u003d \u0027--os-username admin --os-password a_big_secret --os-tenant-name openstack --os-auth-url http://127.0.0.1:5000/v2.0\u0027"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"exec { \u0027manage_m1.nano_nova_flavor\u0027:"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_6e6c5235","line":225,"updated":"2015-08-30 15:17:17.000000000","message":"Not sure, I feel this belongs in puppet-tempest over our local manifest. We can talk about this in MTL, but my understanding is since tempest expects these settings by default, we should just have puppet-tempest do it by default.","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"59ad44bc89c609c1dc9221f213ff4f817974c614","unresolved":false,"context_lines":[{"line_number":222,"context_line":"  neutron_admin_auth_url \u003d\u003e \u0027http://127.0.0.1:35357/v2.0\u0027,"},{"line_number":223,"context_line":"}"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"# Configure Tempest and the resources"},{"line_number":226,"context_line":"$os_auth_options \u003d \u0027--os-username admin --os-password a_big_secret --os-tenant-name openstack --os-auth-url http://127.0.0.1:5000/v2.0\u0027"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"exec { \u0027manage_m1.nano_nova_flavor\u0027:"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_eb435e11","line":225,"in_reply_to":"da20952f_6e6c5235","updated":"2015-08-31 11:04:57.000000000","message":"It makes sense to move flavor management to puppet-tempest, though I did not do it because I wanted this proof of concept first.\n\nRegarding the neutron_network, neutron_subnet and glance_image resources, I think we should not manage them from puppet-tempest but keep using the scenario manifest for now.","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"322aa3e7c1e5095e8605a3bf72e43335256a74d7","unresolved":false,"context_lines":[{"line_number":262,"context_line":"  container_format \u003d\u003e \u0027bare\u0027,"},{"line_number":263,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":264,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":265,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":266,"context_line":"}"},{"line_number":267,"context_line":"glance_image { \u0027cirros_alt\u0027:"},{"line_number":268,"context_line":"  ensure           \u003d\u003e present,"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_6e253207","line":265,"updated":"2015-08-30 15:17:17.000000000","message":"we have this cached locally, please us:\n\n/home/jenkins/cache/files/cirros-0.3.4-x86_64-disk.img","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"59ad44bc89c609c1dc9221f213ff4f817974c614","unresolved":false,"context_lines":[{"line_number":262,"context_line":"  container_format \u003d\u003e \u0027bare\u0027,"},{"line_number":263,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":264,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":265,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":266,"context_line":"}"},{"line_number":267,"context_line":"glance_image { \u0027cirros_alt\u0027:"},{"line_number":268,"context_line":"  ensure           \u003d\u003e present,"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_ab956687","line":265,"in_reply_to":"da20952f_6e253207","updated":"2015-08-31 11:04:57.000000000","message":"Done","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"322aa3e7c1e5095e8605a3bf72e43335256a74d7","unresolved":false,"context_lines":[{"line_number":269,"context_line":"  container_format \u003d\u003e \u0027bare\u0027,"},{"line_number":270,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":271,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":272,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":273,"context_line":"}"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"class { \u0027::tempest\u0027:"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_2e1bbacb","line":272,"updated":"2015-08-30 15:17:17.000000000","message":"same comment","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"59ad44bc89c609c1dc9221f213ff4f817974c614","unresolved":false,"context_lines":[{"line_number":269,"context_line":"  container_format \u003d\u003e \u0027bare\u0027,"},{"line_number":270,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":271,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":272,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":273,"context_line":"}"},{"line_number":274,"context_line":""},{"line_number":275,"context_line":"class { \u0027::tempest\u0027:"}],"source_content_type":"text/x-puppet","patch_set":33,"id":"da20952f_8b98a2c1","line":272,"in_reply_to":"da20952f_2e1bbacb","updated":"2015-08-31 11:04:57.000000000","message":"Done","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"0a45dc31bf11737a7e3d736a59474696b7fc5313","unresolved":false,"context_lines":[{"line_number":262,"context_line":"  container_format \u003d\u003e \u0027bare\u0027,"},{"line_number":263,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":264,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":265,"context_line":"  source           \u003d\u003e \u0027/home/jenkins/cache/files/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":266,"context_line":"}"},{"line_number":267,"context_line":"glance_image { \u0027cirros_alt\u0027:"},{"line_number":268,"context_line":"  ensure           \u003d\u003e present,"}],"source_content_type":"text/x-puppet","patch_set":34,"id":"da20952f_13b5967c","line":265,"updated":"2015-08-31 18:51:16.000000000","message":"probably should parameterize the version of cirros for easier changes later but not necessary now.","commit_id":"b9c50dca95fd563bfde41c3004bc396092ecc8c7"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ecb9b219f4beef6a18c8094d0601b909b49bb8fa","unresolved":false,"context_lines":[{"line_number":139,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":140,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":141,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":142,"context_line":"}"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"# Deploy Neutron"},{"line_number":145,"context_line":"class { \u0027::neutron::db::mysql\u0027:"}],"source_content_type":"text/x-puppet","patch_set":36,"id":"da20952f_1e96493e","side":"PARENT","line":142,"updated":"2015-09-03 21:05:42.000000000","message":"Was there any reason this needed to be moved? seems unrelated","commit_id":"6f673f0d7903ce3f04892dd2f646c58c0a3ec0cd"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"bb1a015cc9e9458af989e1e40fa0f1b8250f8869","unresolved":false,"context_lines":[{"line_number":139,"context_line":"  disk_format      \u003d\u003e \u0027qcow2\u0027,"},{"line_number":140,"context_line":"  is_public        \u003d\u003e \u0027yes\u0027,"},{"line_number":141,"context_line":"  source           \u003d\u003e \u0027http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img\u0027,"},{"line_number":142,"context_line":"}"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"# Deploy Neutron"},{"line_number":145,"context_line":"class { \u0027::neutron::db::mysql\u0027:"}],"source_content_type":"text/x-puppet","patch_set":36,"id":"da20952f_fecfd553","side":"PARENT","line":142,"in_reply_to":"da20952f_1e96493e","updated":"2015-09-03 21:06:54.000000000","message":"yes, I wanted our manifest with a structure where tempest resources could be at the end.","commit_id":"6f673f0d7903ce3f04892dd2f646c58c0a3ec0cd"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"a6c46389241bed1c2d5526a4a22e2eadec1e8993","unresolved":false,"context_lines":[{"line_number":219,"context_line":"  migration_support \u003d\u003e true,"},{"line_number":220,"context_line":"  vncserver_listen  \u003d\u003e \u00270.0.0.0\u0027,"},{"line_number":221,"context_line":"}"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"class { \u0027::nova::scheduler\u0027: }"},{"line_number":224,"context_line":"class { \u0027::nova::vncproxy\u0027: }"},{"line_number":225,"context_line":"class { \u0027::nova::network::neutron\u0027:"}],"source_content_type":"text/x-puppet","patch_set":59,"id":"da20952f_84b18793","line":222,"updated":"2015-09-11 19:53:20.000000000","message":"I can drop the whitespace","commit_id":"0e5f8cc341e5ad1938be9c3d5bd02d7faa09de05"}],"run_tests.sh":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"65950b4bcdfe7c497396ba94ea90a9bfaeb749af","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    exit 1"},{"line_number":59,"context_line":"fi"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"# required by tempest to work"},{"line_number":62,"context_line":"AUTH_OPTIONS\u003d\u0027--os-username admin --os-password a_big_secret --os-tenant-name openstack --os-auth-url http://127.0.0.1:5000/v2.0\u0027"},{"line_number":63,"context_line":"nova $AUTH_OPTIONS flavor-create m1.nano 42 128 0 1"},{"line_number":64,"context_line":"nova $AUTH_OPTIONS flavor-create m1.micro 84 128 0 1"}],"source_content_type":"text/x-sh","patch_set":3,"id":"da20952f_5b2c4eab","line":61,"updated":"2015-08-26 19:37:04.000000000","message":"We should add this into puppet-tempest if possible.  Seems like something it should manage vs our shell scripts.","commit_id":"aa53fab7c321e9b99eb0311e40e9b38a7a028246"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"322aa3e7c1e5095e8605a3bf72e43335256a74d7","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    exit 1"},{"line_number":58,"context_line":"fi"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(emilien) move this code somewhere else..."},{"line_number":61,"context_line":"# it makes floating-ip working in VM with neutron \u0026 OVS"},{"line_number":62,"context_line":"$SUDO ovs-vsctl add-br br-ex # need two runs with puppet TODO: fix it"},{"line_number":63,"context_line":"$SUDO ovs-vsctl set Bridge br-ex other_config:disable-in-band\u003dtrue"}],"source_content_type":"text/x-sh","patch_set":33,"id":"da20952f_ee662250","line":60,"updated":"2015-08-30 15:17:17.000000000","message":"Ya, lets keep this file clean as much as possible.\n\nI\u0027m thinking we might need to create a tools/prepare_tempest.sh or something.","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"59ad44bc89c609c1dc9221f213ff4f817974c614","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    exit 1"},{"line_number":58,"context_line":"fi"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(emilien) move this code somewhere else..."},{"line_number":61,"context_line":"# it makes floating-ip working in VM with neutron \u0026 OVS"},{"line_number":62,"context_line":"$SUDO ovs-vsctl add-br br-ex # need two runs with puppet TODO: fix it"},{"line_number":63,"context_line":"$SUDO ovs-vsctl set Bridge br-ex other_config:disable-in-band\u003dtrue"}],"source_content_type":"text/x-sh","patch_set":33,"id":"da20952f_cb553ac2","line":60,"in_reply_to":"da20952f_ee662250","updated":"2015-08-31 11:04:57.000000000","message":"Yes, again I wanted this proof of concept working before cleaning/moving the code somewhere else.\n\nA patch on top of this would be useful to clean up the code.\n\nI think now, it\u0027s easier to review and understand what we\u0027re doing.","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"322aa3e7c1e5095e8605a3bf72e43335256a74d7","unresolved":false,"context_lines":[{"line_number":65,"context_line":"$SUDO /usr/sbin/ip link set br-ex up"},{"line_number":66,"context_line":"$SUDO /usr/sbin/service neutron-l3-agent restart"},{"line_number":67,"context_line":"# and prepare image files for tempest scenarios"},{"line_number":68,"context_line":"wget http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img -P /tmp/openstack/tempest"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"# run a scenario that validates Keystone, Nova, Glance and Neutron"},{"line_number":71,"context_line":"cd /tmp/openstack/tempest; tox -eall -- smoke"}],"source_content_type":"text/x-sh","patch_set":33,"id":"da20952f_2e72da11","line":68,"updated":"2015-08-30 15:17:17.000000000","message":"This specifically is not needed, look in:\n\n/home/jenkins/cache/files/cirros-0.3.4-x86_64-disk.img","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"59ad44bc89c609c1dc9221f213ff4f817974c614","unresolved":false,"context_lines":[{"line_number":65,"context_line":"$SUDO /usr/sbin/ip link set br-ex up"},{"line_number":66,"context_line":"$SUDO /usr/sbin/service neutron-l3-agent restart"},{"line_number":67,"context_line":"# and prepare image files for tempest scenarios"},{"line_number":68,"context_line":"wget http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img -P /tmp/openstack/tempest"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"# run a scenario that validates Keystone, Nova, Glance and Neutron"},{"line_number":71,"context_line":"cd /tmp/openstack/tempest; tox -eall -- smoke"}],"source_content_type":"text/x-sh","patch_set":33,"id":"da20952f_eb111e05","line":68,"in_reply_to":"da20952f_2e72da11","updated":"2015-08-31 11:04:57.000000000","message":"Done","commit_id":"429cac3c691e53cfceca2fe22642258ccfc1fe4a"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ecb9b219f4beef6a18c8094d0601b909b49bb8fa","unresolved":false,"context_lines":[{"line_number":63,"context_line":"$SUDO ovs-vsctl set Bridge br-ex other_config:disable-in-band\u003dtrue"},{"line_number":64,"context_line":"$SUDO /usr/sbin/ip addr add 200.0.0.1/24 dev br-ex"},{"line_number":65,"context_line":"$SUDO /usr/sbin/ip link set br-ex up"},{"line_number":66,"context_line":"$SUDO /usr/sbin/service neutron-l3-agent restart"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"# TODO(emilien) later, we should use local image if present. That would be a next iteration."},{"line_number":69,"context_line":"wget http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img -P /tmp/openstack/tempest"}],"source_content_type":"text/x-sh","patch_set":36,"id":"da20952f_5eb7e1d7","line":66,"updated":"2015-09-03 21:05:42.000000000","message":"Shouldn\u0027t we be doing this in puppet?","commit_id":"bf49c74e426be56a7dd2bb5d97fabfeb97274a3c"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"f8e9941cc93641363058329c9d01e21097c2c023","unresolved":false,"context_lines":[{"line_number":63,"context_line":"$SUDO ovs-vsctl set Bridge br-ex other_config:disable-in-band\u003dtrue"},{"line_number":64,"context_line":"$SUDO /usr/sbin/ip addr add 200.0.0.1/24 dev br-ex"},{"line_number":65,"context_line":"$SUDO /usr/sbin/ip link set br-ex up"},{"line_number":66,"context_line":"$SUDO /usr/sbin/service neutron-l3-agent restart"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"# TODO(emilien) later, we should use local image if present. That would be a next iteration."},{"line_number":69,"context_line":"wget http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img -P /tmp/openstack/tempest"}],"source_content_type":"text/x-sh","patch_set":36,"id":"da20952f_7e9bc518","line":66,"in_reply_to":"da20952f_1e8089be","updated":"2015-09-03 21:17:02.000000000","message":"So a more context:\n\n* We want to test floating IP but for that we need to set br-ex.\n* We also need to create a public network bridged on br-ex\n\nSince we don\u0027t have a real public network interface, we need to fake it by creating a virtual device for br-ex with a virtual network, with 200.0.0.1/24 as primary IP address, and tell neutron this interface is our external network gateway.\nAnd then, we need to restart Neutron L3 agent because it needs to create virtual network interfaces on the right bridge...\n\nThis code is imported from devstack and I see no way to make it in Puppet right now.","commit_id":"bf49c74e426be56a7dd2bb5d97fabfeb97274a3c"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"e326280ca08fd243d4398b3bf1ebc1df1426cf01","unresolved":false,"context_lines":[{"line_number":63,"context_line":"$SUDO ovs-vsctl set Bridge br-ex other_config:disable-in-band\u003dtrue"},{"line_number":64,"context_line":"$SUDO /usr/sbin/ip addr add 200.0.0.1/24 dev br-ex"},{"line_number":65,"context_line":"$SUDO /usr/sbin/ip link set br-ex up"},{"line_number":66,"context_line":"$SUDO /usr/sbin/service neutron-l3-agent restart"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"# TODO(emilien) later, we should use local image if present. That would be a next iteration."},{"line_number":69,"context_line":"wget http://download.cirros-cloud.net/0.3.4/cirros-0.3.4-x86_64-disk.img -P /tmp/openstack/tempest"}],"source_content_type":"text/x-sh","patch_set":36,"id":"da20952f_1e8089be","line":66,"in_reply_to":"da20952f_5eb7e1d7","updated":"2015-09-03 21:09:38.000000000","message":"no because this is an hack that is executed after puppet... I don\u0027t see how we can Puppetize that code.","commit_id":"bf49c74e426be56a7dd2bb5d97fabfeb97274a3c"}]}