)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d16c41ace155a0a5e84f9c24785bd459693e3c2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"WIP: Disable security group tests in tempest-dsvm-cells-rc using a config option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked becausecells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc."},{"line_number":13,"context_line":"Change to block the tests based on a new tempest config option -"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_c32120fe","line":10,"updated":"2015-09-21 20:34:31.000000000","message":"s/becausecells/because cells/","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d16c41ace155a0a5e84f9c24785bd459693e3c2","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked becausecells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc."},{"line_number":13,"context_line":"Change to block the tests based on a new tempest config option -"},{"line_number":14,"context_line":"compute-feature-enabled.security_groups."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_a3266c14","line":12,"updated":"2015-09-21 20:34:31.000000000","message":"This path doesn\u0027t exist, the rc file is in nova, not devstack.  Since you\u0027re changing the file here, I don\u0027t see the point in adding the link.  Just point out the rc file.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c9f288a27ef345443a93a0caa10640ced1157177","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked becausecells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc."},{"line_number":13,"context_line":"Change to block the tests based on a new tempest config option -"},{"line_number":14,"context_line":"compute-feature-enabled.security_groups."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_03b95879","line":12,"in_reply_to":"ba15a1d1_a3266c14","updated":"2015-09-21 20:35:37.000000000","message":"Oh nvm, it\u0027s the path in nova, the URL doesn\u0027t work because of the period.  Still, we don\u0027t need the full path here since we\u0027re in nova, just mention the rc file.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d16c41ace155a0a5e84f9c24785bd459693e3c2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"are blocked becausecells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc."},{"line_number":13,"context_line":"Change to block the tests based on a new tempest config option -"},{"line_number":14,"context_line":"compute-feature-enabled.security_groups."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Also, some of the tempest scenario tests attempt to create a server with a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_c3684024","line":13,"updated":"2015-09-21 20:34:31.000000000","message":"This doesn\u0027t actually happen here, it happens in the devstack change.  You could point out that happens in the dependent devstack change though.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d16c41ace155a0a5e84f9c24785bd459693e3c2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Change to block the tests based on a new tempest config option -"},{"line_number":14,"context_line":"compute-feature-enabled.security_groups."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Also, some of the tempest scenario tests attempt to create a server with a"},{"line_number":17,"context_line":"security group.  These will not work with cells, so use the new config option"},{"line_number":18,"context_line":"to block those tests."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_23eedc8b","line":16,"updated":"2015-09-21 20:34:31.000000000","message":"Again, not really relevant to the nova change.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c9f288a27ef345443a93a0caa10640ced1157177","unresolved":false,"context_lines":[{"line_number":21,"context_line":"that are now blocked by the config option."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: Icb6d9934dfbe4a4f03e38a47fb9d29a0e72d1942"},{"line_number":24,"context_line":"Partial-Bug: 1495653"},{"line_number":25,"context_line":"Depends-On: I4f74b80024a9bcc374d374d86052dceb40598d32"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_c3ab601d","line":24,"updated":"2015-09-21 20:35:37.000000000","message":"This should be Closes-Bug since it closes this part of the bug for the nova project.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"547efeb0225dd97cee6d0b010e10bc6e977c4be7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Chuck Carmack \u003ccarmack@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-09-25 18:54:36 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Disable security group tests in tempest-dsvm-cells-rc using a config option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked because cells doesn\u0027t support security groups.  The block is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9a1a9d01_623b75ec","line":7,"updated":"2015-09-25 19:00:19.000000000","message":"The wording here is a bit confusing.  This makes sense for the Tempest change, but in Nova we\u0027re really just removing the tests from the cells rc blacklist file which we were only excluding because they used security groups.  So I\u0027d maybe word this as:\n\n\"Remove security group tests from tempest-dsvm-cells-rc\"","commit_id":"dc51106b3888d1396a8f7716378a0b36795bf695"},{"author":{"_account_id":17139,"name":"Chuck Carmack","email":"carmack@us.ibm.com","username":"ccarmack"},"change_message_id":"6fb45a608df7cedad6dcdd77addbcdab2dc3f505","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Chuck Carmack \u003ccarmack@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-09-25 18:54:36 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Disable security group tests in tempest-dsvm-cells-rc using a config option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked because cells doesn\u0027t support security groups.  The block is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9a1a9d01_33b27f95","line":7,"in_reply_to":"9a1a9d01_623b75ec","updated":"2015-09-28 14:19:18.000000000","message":"Done","commit_id":"dc51106b3888d1396a8f7716378a0b36795bf695"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"547efeb0225dd97cee6d0b010e10bc6e977c4be7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked because cells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This commit changes tempest-dsvm-cells-rc to remove blacklisted tests."},{"line_number":15,"context_line":"Inclusion of security group related tests is now controlled"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9a1a9d01_e22f859f","line":12,"updated":"2015-09-25 19:00:19.000000000","message":"I\u0027d remove this whole last sentence since it\u0027s redundant.","commit_id":"dc51106b3888d1396a8f7716378a0b36795bf695"},{"author":{"_account_id":17139,"name":"Chuck Carmack","email":"carmack@us.ibm.com","username":"ccarmack"},"change_message_id":"6fb45a608df7cedad6dcdd77addbcdab2dc3f505","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked because cells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova blacklist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This commit changes tempest-dsvm-cells-rc to remove blacklisted tests."},{"line_number":15,"context_line":"Inclusion of security group related tests is now controlled"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9a1a9d01_53adb32f","line":12,"in_reply_to":"9a1a9d01_e22f859f","updated":"2015-09-28 14:19:18.000000000","message":"Done","commit_id":"dc51106b3888d1396a8f7716378a0b36795bf695"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ee169ffb57a4bfbedefdb2a88b3d5b0cf71c14eb","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove security group tests from tempest-dsvm-cells-rc"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently, the tempest security group tests for tempest-dsvm-cells-rc"},{"line_number":10,"context_line":"are blocked because cells doesn\u0027t support security groups.  "},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit changes tempest-dsvm-cells-rc to remove blacklisted tests."},{"line_number":13,"context_line":"Inclusion of security group related tests is now controlled"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"9a1a9d01_d94a8fac","line":10,"updated":"2015-10-02 01:59:51.000000000","message":"whitespace issue here","commit_id":"df3508ce37a52ce1aee3b60b9ba9126cd6595916"}],"devstack/tempest-dsvm-cells-rc":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d16c41ace155a0a5e84f9c24785bd459693e3c2","unresolved":false,"context_lines":[{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip security group tests"},{"line_number":26,"context_line":"# r\u003d\"$r(?:tempest\\.api\\.compute\\.security_groups.*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip test that requires security groups"},{"line_number":29,"context_line":"# r\u003d\"$r|(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"ba15a1d1_e3e784aa","line":26,"updated":"2015-09-21 20:34:31.000000000","message":"Remove them rather than comment them out.","commit_id":"98aaf9f58089d8aa58a1cd6bbf55aa21646fc24a"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"3e210e5c28ec7b7e3fd4e1ea32d0458a1627fc7a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# to avoid features unsupported by Nova Cells."},{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip test that requires a floating-ip\t\t"},{"line_number":26,"context_line":"r\u003d\"$r(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip aggregates tests"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9a1a9d01_4674659b","line":25,"updated":"2015-09-25 01:35:30.000000000","message":"nit: Here contains unnecessary TABs.\n\nand tempest-dsvm-cells-rc doesn\u0027t support floating-ip also?\nThe following test_compute_with_volumes of Tempest requires security-group and floating-ip.\nbut I\u0027m not sure this patch purpose because the commit message doesn\u0027t mention about floating-ip.","commit_id":"2fa445348491423c0f48a8c1d16606e2a9633321"},{"author":{"_account_id":17139,"name":"Chuck Carmack","email":"carmack@us.ibm.com","username":"ccarmack"},"change_message_id":"6b8bdc546a02a163e4e27f5d75cbf096805ef68d","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# to avoid features unsupported by Nova Cells."},{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip test that requires a floating-ip\t\t"},{"line_number":26,"context_line":"r\u003d\"$r(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip aggregates tests"}],"source_content_type":"application/octet-stream","patch_set":7,"id":"9a1a9d01_3114ed42","line":25,"in_reply_to":"9a1a9d01_4674659b","updated":"2015-09-25 17:19:16.000000000","message":"Thanks for the comment.  While I was trying to get some of the tests to work, I found out the cells doesn\u0027t work with floating IP.  I\u0027ll work on floating IP in a different bug.","commit_id":"2fa445348491423c0f48a8c1d16606e2a9633321"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"547efeb0225dd97cee6d0b010e10bc6e977c4be7","unresolved":false,"context_lines":[{"line_number":22,"context_line":"# to avoid features unsupported by Nova Cells."},{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip test that requires a floating-ip"},{"line_number":26,"context_line":"r\u003d\"$r(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip aggregates tests"}],"source_content_type":"application/octet-stream","patch_set":9,"id":"9a1a9d01_827249e4","line":25,"updated":"2015-09-25 19:00:19.000000000","message":"confirmed:\n\nhttps://github.com/openstack/tempest/blob/master/tempest/thirdparty/boto/test_ec2_instance_run.py#L274","commit_id":"dc51106b3888d1396a8f7716378a0b36795bf695"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ee169ffb57a4bfbedefdb2a88b3d5b0cf71c14eb","unresolved":false,"context_lines":[{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip security group tests"},{"line_number":26,"context_line":"r\u003d\"$r(?:tempest\\.api\\.compute\\.security_groups.*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip test that requires security groups"},{"line_number":29,"context_line":"r\u003d\"$r|(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""}],"source_content_type":"application/octet-stream","patch_set":10,"id":"9a1a9d01_d9e12f9c","side":"PARENT","line":26,"updated":"2015-10-02 01:59:51.000000000","message":"Hmm, so why are these running?\n\nhttp://logs.openstack.org/43/226043/10/check/gate-tempest-dsvm-cells/b28ebdb/console.html#_2015-09-29_18_44_27_266\n\nI thought it should skip them.  I thought the tempest change would not actually run the tempest.api.compute.security_groups.* tests if compute-feature-enabled.security_groups was False, but that\u0027s not checked in the tempest change:\n\nhttps://review.openstack.org/#/c/225199/\n\nI\u0027m surprised the tests are actually passing though...that\u0027s weird.","commit_id":"3f26a620073d75123675186eefff6ac3a745b38e"},{"author":{"_account_id":17139,"name":"Chuck Carmack","email":"carmack@us.ibm.com","username":"ccarmack"},"change_message_id":"29c6ee9934d2bc19ff0edbe646efb8dbc40a8690","unresolved":false,"context_lines":[{"line_number":23,"context_line":"r\u003d\"^(?!.*\""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# skip security group tests"},{"line_number":26,"context_line":"r\u003d\"$r(?:tempest\\.api\\.compute\\.security_groups.*)\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# skip test that requires security groups"},{"line_number":29,"context_line":"r\u003d\"$r|(?:tempest\\.thirdparty\\.boto\\.test_ec2_instance_run\\.InstanceRunTest\\.test_compute_with_volumes*)\""}],"source_content_type":"application/octet-stream","patch_set":10,"id":"9a1a9d01_79e79f40","side":"PARENT","line":26,"in_reply_to":"9a1a9d01_d9e12f9c","updated":"2015-10-02 18:36:19.000000000","message":"I replied in https://review.openstack.org/218468","commit_id":"3f26a620073d75123675186eefff6ac3a745b38e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ee169ffb57a4bfbedefdb2a88b3d5b0cf71c14eb","unresolved":false,"context_lines":[{"line_number":65,"context_line":"r\u003d\"$r|(?:tempest\\.scenario\\.test_encrypted_cinder_volumes\\.TestEncryptedCinderVolumes\\.test_encrypted_cinder_volumes_luks*)\""},{"line_number":66,"context_line":"r\u003d\"$r|(?:tempest\\.scenario\\.test_server_advanced_ops\\.TestServerAdvancedOps\\.test_resize_server_confirm*)\""},{"line_number":67,"context_line":"r\u003d\"$r|(?:tempest\\.thirdparty\\.boto\\.test_ec2_network\\.EC2NetworkTest\\.test_disassociate_not_associated_floating_ip*)\""},{"line_number":68,"context_line":"r\u003d\"$r|(?:tempest\\.scenario\\.test_server_basic_ops\\.TestServerBasicOps\\.test_server_basicops*)\""},{"line_number":69,"context_line":"r\u003d\"$r|(?:tempest\\.scenario\\.test_snapshot_pattern\\.TestSnapshotPattern\\.test_snapshot_pattern*)\""},{"line_number":70,"context_line":"r\u003d\"$r|(?:tempest\\.api\\.compute\\.admin\\.test_hosts\\.HostsAdminTestJSON\\.test_show_host_detail*)\""},{"line_number":71,"context_line":"r\u003d\"$r|(?:tempest\\.api\\.compute\\.test_tenant_networks\\.ComputeTenantNetworksTest\\.test_list_show_tenant_networks*)\""}],"source_content_type":"application/octet-stream","patch_set":10,"id":"9a1a9d01_99f2d76e","side":"PARENT","line":68,"updated":"2015-10-02 01:59:51.000000000","message":"ack:\n\nhttp://logs.openstack.org/43/226043/10/check/gate-tempest-dsvm-cells/b28ebdb/console.html#_2015-09-29_18_50_56_514","commit_id":"3f26a620073d75123675186eefff6ac3a745b38e"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"2baa8fd96d5e3f1d6619483b7eb05da7e1435677","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# NOTE(mriedem): Resize tests are skipped in devstack until custom flavors"},{"line_number":61,"context_line":"# in devstack used in Tempest runs are synced to the cells database."},{"line_number":62,"context_line":"\u003e\u003e\u003e\u003e\u003e\u003e\u003e Disable security group tests in tempest-dsvm-cells-rc using a config option"},{"line_number":63,"context_line":"r\u003d\"$r|(?:tempest\\.api\\.compute\\.admin\\.test_networks\\.NetworksTest\\.test_get_network)\""},{"line_number":64,"context_line":"r\u003d\"$r|(?:tempest\\.api\\.compute\\.admin\\.test_networks\\.NetworksTest\\.test_list_all_networks)\""},{"line_number":65,"context_line":"r\u003d\"$r|(?:tempest\\.api\\.compute\\.servers\\.test_server_rescue\\.ServerRescueTestJSON)\""}],"source_content_type":"application/octet-stream","patch_set":12,"id":"7a740942_ff06071f","line":62,"updated":"2015-12-10 22:27:28.000000000","message":"Accidentally left this line in","commit_id":"e8cba30fcf6e4220377f52e54dc3a5fc8e8eadbd"}]}