)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e71301db52e573861849567e288bc91a44b64cd8","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-18 15:30:37 +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 becausecells doesn\u0027t support security groups.  The block is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_a6f99aca","line":7,"updated":"2015-09-21 20:46:44.000000000","message":"The rc file mention here is confusing.  Just say:\n\n\"Configure tempest to not use security groups for cells\"","commit_id":"ac5064b2177e7cd6c3d81eafc674e47431326ecf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e71301db52e573861849567e288bc91a44b64cd8","unresolved":false,"context_lines":[{"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 becausecells doesn\u0027t support security groups.  The block is"},{"line_number":11,"context_line":"now done using a nova whitelist:"},{"line_number":12,"context_line":"http://git.openstack.org/cgit/openstack/nova/tree/devstack/tempest-dsvm-cells-rc."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_462f5e3e","line":9,"updated":"2015-09-21 20:46:44.000000000","message":"See comments in other changes about the wording here, it\u0027s confusing.","commit_id":"ac5064b2177e7cd6c3d81eafc674e47431326ecf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e71301db52e573861849567e288bc91a44b64cd8","unresolved":false,"context_lines":[{"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 whitelist:"},{"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."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_06296637","line":11,"updated":"2015-09-21 20:46:44.000000000","message":"blacklist, not whitelist","commit_id":"ac5064b2177e7cd6c3d81eafc674e47431326ecf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e71301db52e573861849567e288bc91a44b64cd8","unresolved":false,"context_lines":[{"line_number":21,"context_line":"option to False if cells are enabled."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I4f74b80024a9bcc374d374d86052dceb40598d32"},{"line_number":24,"context_line":"Partial-Bug: 1495653"},{"line_number":25,"context_line":"Depends-On: Icd62f42809380de6d638b343d3322b102b0d1d1f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_26058ac1","line":24,"updated":"2015-09-21 20:46:44.000000000","message":"This should be Closes-Bug","commit_id":"ac5064b2177e7cd6c3d81eafc674e47431326ecf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2a1aac2b3aa1250b1adb809e84c56f18a590b45d","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Another change set is dependent on this change:"},{"line_number":21,"context_line":"Nova change to remove the security group related tests"},{"line_number":22,"context_line":"from the blacklist."},{"line_number":23,"context_line":"https://review.openstack.org/#/c/226043/"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I4f74b80024a9bcc374d374d86052dceb40598d32"},{"line_number":26,"context_line":"Closes-Bug: 1495653"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a1a9d01_3d111c53","line":23,"updated":"2015-09-25 19:01:59.000000000","message":"Use the nova changeid rather than a URL, so use:\n\nIcb6d9934dfbe4a4f03e38a47fb9d29a0e72d1942","commit_id":"6605c14f744dfaff6b85fbfcee5c730c33bc4ae6"},{"author":{"_account_id":17139,"name":"Chuck Carmack","email":"carmack@us.ibm.com","username":"ccarmack"},"change_message_id":"827b8538a129402ef2d93c708854191aa5db91f2","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Another change set is dependent on this change:"},{"line_number":21,"context_line":"Nova change to remove the security group related tests"},{"line_number":22,"context_line":"from the blacklist."},{"line_number":23,"context_line":"https://review.openstack.org/#/c/226043/"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I4f74b80024a9bcc374d374d86052dceb40598d32"},{"line_number":26,"context_line":"Closes-Bug: 1495653"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a1a9d01_d37d43e8","line":23,"in_reply_to":"9a1a9d01_3d111c53","updated":"2015-09-28 14:15:50.000000000","message":"Done","commit_id":"6605c14f744dfaff6b85fbfcee5c730c33bc4ae6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"72af3cba845c0b1a45a6c264dd2ec786a9f1b52f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Another change set is dependent on this change:"},{"line_number":21,"context_line":"Nova change to remove the security group related tests"},{"line_number":22,"context_line":"from the blacklist."},{"line_number":23,"context_line":"https://review.openstack.org/#/c/226043/"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I4f74b80024a9bcc374d374d86052dceb40598d32"},{"line_number":26,"context_line":"Closes-Bug: 1495653"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"1a26ad4f_bfd8e5f0","line":23,"updated":"2015-10-27 20:32:09.000000000","message":"I\u0027d prefer that this were a change ID:\n\nIcb6d9934dfbe4a4f03e38a47fb9d29a0e72d1942\n\nBut not a deal breaker.","commit_id":"e4310dadf2f0031d3278144fbdf72ba0fc78f7d7"}],"lib/tempest":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e71301db52e573861849567e288bc91a44b64cd8","unresolved":false,"context_lines":[{"line_number":402,"context_line":"        # Cells doesn\u0027t support shelving/unshelving"},{"line_number":403,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled shelve False"},{"line_number":404,"context_line":"        # Cells doesn\u0027t support security groups."},{"line_number":405,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled security_groups False"},{"line_number":406,"context_line":"    fi"},{"line_number":407,"context_line":""},{"line_number":408,"context_line":"    # Network"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"ba15a1d1_e343046b","line":405,"updated":"2015-09-21 20:46:44.000000000","message":"The config option is actually \u0027security-groups\u0027 with a dash rather than underscore:\n\nhttps://review.openstack.org/#/c/225199/3/tempest/config.py\n\nLooks like it should be changed to an underscore though in tempest/config.py to be like the other options.","commit_id":"ac5064b2177e7cd6c3d81eafc674e47431326ecf"}]}