)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"758531b101b0bca7c50e67a9263f140a19c2852b","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-07-16 19:41:12 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Skip tests that require tenant isolation if isolation not allowed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-bug #1253021"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I9e76681172bf55e081870cae8a4d806e8c876de4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"baada198_68a78150","line":8,"updated":"2014-07-21 17:00:04.000000000","message":"can you also add a comment here about removing the force_tenant_isolation from the quotas test cause it wasn\u0027t used?","commit_id":"a4ba43dd603e7eaae9d5014382f0a91c1706d051"}],"tempest/api/compute/base.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"69d2e37d1e0d2d3024e7e552312a77ec720ebc5e","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    \"\"\"Base test case class for all Compute API tests.\"\"\""},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    _api_version \u003d 3"},{"line_number":34,"context_line":"    force_tenant_isolation \u003d False"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @classmethod"},{"line_number":37,"context_line":"    def setUpClass(cls):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_9b285287","side":"PARENT","line":34,"updated":"2014-07-22 15:40:21.000000000","message":"I don\u0027t think you want to remove this, it will raise an exception cause the missing attribute on L 386 of the new version","commit_id":"2b8655221bf17d83cdae25e1eb09133cdcb949e9"}],"tempest/api/network/admin/test_floating_ips_admin_actions.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"758531b101b0bca7c50e67a9263f140a19c2852b","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class FloatingIPAdminTestJSON(base.BaseAdminNetworkTest):"},{"line_number":25,"context_line":"    _interface \u003d \u0027json\u0027"},{"line_number":26,"context_line":"    force_tenant_isolation \u003d True"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    @classmethod"},{"line_number":29,"context_line":"    def setUpClass(cls):"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_ebdd52de","side":"PARENT","line":26,"updated":"2014-07-21 17:00:04.000000000","message":"I guess this shouldn \u0027t be removed, there is logic to deal with it in BaseAdminNetworkTest here https://github.com/openstack/tempest/blob/master/tempest/api/network/base.py#L373","commit_id":"2b8655221bf17d83cdae25e1eb09133cdcb949e9"}],"tempest/api/network/base.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"758531b101b0bca7c50e67a9263f140a19c2852b","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    Finally, it is assumed that the following option is defined in the"},{"line_number":47,"context_line":"    [service_available] section of etc/tempest.conf"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        neutron as True"},{"line_number":50,"context_line":"    \"\"\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    force_tenant_isolation \u003d False"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_48855dab","side":"PARENT","line":49,"updated":"2014-07-21 17:00:04.000000000","message":"why did you remov this piece? ^","commit_id":"2b8655221bf17d83cdae25e1eb09133cdcb949e9"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"6fb96dc0252318ecf0efe47c3b369e6f067defe4","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        Finally, it is assumed that the following option is defined in the"},{"line_number":47,"context_line":"        [service_available] section of etc/tempest.conf"},{"line_number":48,"context_line":"        "},{"line_number":49,"context_line":"            neutron as True"},{"line_number":50,"context_line":"    \"\"\""},{"line_number":51,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_eca55eb5","line":48,"updated":"2014-07-21 19:12:08.000000000","message":"forgot those whitespaces and added some spaces to the paragraph","commit_id":"88267f97ae436fe488531f6b98450cbccac1839b"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"69d2e37d1e0d2d3024e7e552312a77ec720ebc5e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        neutron as True"},{"line_number":50,"context_line":"    \"\"\""},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    force_tenant_isolation \u003d False"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    # Default to ipv4."},{"line_number":55,"context_line":"    _ip_version \u003d 4"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_db7e4a6a","side":"PARENT","line":52,"updated":"2014-07-22 15:40:21.000000000","message":"same as compute/base.py\nYou will face a attribute missing in line 375... the tests passed only because we don\u0027t test with tenant_isolation off right now","commit_id":"2b8655221bf17d83cdae25e1eb09133cdcb949e9"}],"tempest/api/volume/admin/test_volume_quotas.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"3e95c9dd35654fe1a4a89510b983e65d0b4af78b","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"class VolumeQuotasAdminTestJSON(base.BaseVolumeV1AdminTest):"},{"line_number":26,"context_line":"    _interface \u003d \"json\""},{"line_number":27,"context_line":"    force_tenant_isolation \u003d True"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    @classmethod"},{"line_number":30,"context_line":"    def setUpClass(cls):"}],"source_content_type":"text/x-python","patch_set":10,"id":"fa98f980_86d10f25","side":"PARENT","line":27,"updated":"2014-09-04 19:27:51.000000000","message":"While this works in the gate because tenant isolation is enabled, aren\u0027t quotas in cinder also tenant specific? So if we are using the config specified tenant here instead of creating a new one aren\u0027t we at risk for the same polution issue we see in the other quota test cases?","commit_id":"2b8655221bf17d83cdae25e1eb09133cdcb949e9"}]}