)]}'
{"etc/tempest.conf.sample":[{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"001891d3713af7cec6c638d45eaf29da8a1728bf","unresolved":false,"context_lines":[{"line_number":298,"context_line":"# container to container synchronization"},{"line_number":299,"context_line":"container_sync_interval \u003d 5"},{"line_number":300,"context_line":"# Set to True if the Account Quota middleware is enabled"},{"line_number":301,"context_line":"accounts_quotas_available \u003d True"},{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FRlg%3D","line":301,"updated":"2013-11-11 21:45:26.000000000","message":"This is already define in the object-feature-enabled section.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"cbcd410c719d1084d47d5ca6cd670cf3cd0c1398","unresolved":false,"context_lines":[{"line_number":298,"context_line":"# container to container synchronization"},{"line_number":299,"context_line":"container_sync_interval \u003d 5"},{"line_number":300,"context_line":"# Set to True if the Account Quota middleware is enabled"},{"line_number":301,"context_line":"accounts_quotas_available \u003d True"},{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FKTQ%3D","line":301,"in_reply_to":"AAAATn%2F%2FRlg%3D","updated":"2013-11-13 15:19:51.000000000","message":"Done. FWIW, I believe it\u0027s `object-storage-feature-enabled`.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"001891d3713af7cec6c638d45eaf29da8a1728bf","unresolved":false,"context_lines":[{"line_number":300,"context_line":"# Set to True if the Account Quota middleware is enabled"},{"line_number":301,"context_line":"accounts_quotas_available \u003d True"},{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"},{"line_number":305,"context_line":"crossdomain_available \u003d True"},{"line_number":306,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FRlc%3D","line":303,"updated":"2013-11-11 21:45:26.000000000","message":"This is already define in the object-feature-enabled section.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"cbcd410c719d1084d47d5ca6cd670cf3cd0c1398","unresolved":false,"context_lines":[{"line_number":300,"context_line":"# Set to True if the Account Quota middleware is enabled"},{"line_number":301,"context_line":"accounts_quotas_available \u003d True"},{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"},{"line_number":305,"context_line":"crossdomain_available \u003d True"},{"line_number":306,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FKTM%3D","line":303,"in_reply_to":"AAAATn%2F%2FRlc%3D","updated":"2013-11-13 15:19:51.000000000","message":"Done","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"001891d3713af7cec6c638d45eaf29da8a1728bf","unresolved":false,"context_lines":[{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"},{"line_number":305,"context_line":"crossdomain_available \u003d True"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"# Set operator role for tests that require creating a container"},{"line_number":308,"context_line":"operator_role \u003d Member"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FRlY%3D","line":305,"updated":"2013-11-11 21:45:26.000000000","message":"This should be defined in the object-feature-enabled section.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"cbcd410c719d1084d47d5ca6cd670cf3cd0c1398","unresolved":false,"context_lines":[{"line_number":302,"context_line":"# Set to True if the Container Quota middleware is enabled"},{"line_number":303,"context_line":"container_quotas_available \u003d True"},{"line_number":304,"context_line":"# Set to True if the Crossdomain middleware is enabled"},{"line_number":305,"context_line":"crossdomain_available \u003d True"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"# Set operator role for tests that require creating a container"},{"line_number":308,"context_line":"operator_role \u003d Member"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"AAAATn%2F%2FKTI%3D","line":305,"in_reply_to":"AAAATn%2F%2FRlY%3D","updated":"2013-11-13 15:19:51.000000000","message":"Done","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"}],"tempest/api/object_storage/test_crossdomain.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"53e2f7282cf8e4459741a46ffe785f520a0559ea","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        cls.data.teardown_all()"},{"line_number":52,"context_line":"        super(CrossdomainTest, cls).tearDownClass()"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def set_account_client_url_to_endpoint(self):"},{"line_number":55,"context_line":"        \"\"\"By default, the account client will point to a URL consisting of the"},{"line_number":56,"context_line":"        endpoint + \"v1/$(tenant_id)\"."},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2Bq0%3D","line":54,"updated":"2013-10-23 11:50:55.000000000","message":"This method is also defined in https://review.openstack.org/#/c/51795/ , shouldn\u0027t it be part of some \u0027utils\u0027 module ?","commit_id":"2a12f8fd87a4ffce984388c1792d1440a02b196e"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"811a6b066708b35caaeac432de4e05aeb9b2d441","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        super(CrossdomainTest, self).tearDown()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    @testtools.skipIf(not crossdomain_available,"},{"line_number":72,"context_line":"                      \"Crossdomain middleware not available\")"},{"line_number":73,"context_line":"    @attr(\u0027gate\u0027)"},{"line_number":74,"context_line":"    def test_get_crossdomain_policy(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2FPuQ%3D","line":71,"updated":"2013-11-12 10:28:21.000000000","message":"I think this class is for the cross-domain test. So I think this skip logic should be moved to setUpClass() like api/compute/volumes/test_attach_volume.py.\nWhat do you think?","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"cbcd410c719d1084d47d5ca6cd670cf3cd0c1398","unresolved":false,"context_lines":[{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        super(CrossdomainTest, self).tearDown()"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    @testtools.skipIf(not crossdomain_available,"},{"line_number":72,"context_line":"                      \"Crossdomain middleware not available\")"},{"line_number":73,"context_line":"    @attr(\u0027gate\u0027)"},{"line_number":74,"context_line":"    def test_get_crossdomain_policy(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2FKTc%3D","line":71,"in_reply_to":"AAAATn%2F%2FPuQ%3D","updated":"2013-11-13 15:19:51.000000000","message":"I like this, thanks for pointing it out.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"}],"tempest/common/rest_client.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2b0adaddce5afc21e3b47d46a6be1d00b586cee0","unresolved":false,"context_lines":[{"line_number":412,"context_line":""},{"line_number":413,"context_line":"        return resp, resp_body"},{"line_number":414,"context_line":""},{"line_number":415,"context_line":"    def _request_base(self, method, url,"},{"line_number":416,"context_line":"                      headers\u003dNone, body\u003dNone):"},{"line_number":417,"context_line":"        \"\"\"This method is similar to self._request, but it targets"},{"line_number":418,"context_line":"        the root url of the cluster."}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fw7w%3D","line":415,"updated":"2013-10-10 06:18:58.000000000","message":"This method is almost the same as _request().\nSo how about adding a parameter to _request()? For example:\n\ndef _request(self, method, url,\n                   headers\u003dNone, body\u003dNone, is_base\u003dFalse):\n    req_url \u003d \"%s/%s\" % (self.base_url, url)\n    if is_base:\n        req_url \u003d \"%s/%s\" % (\"/\".join(self.base_url.split(\"/\")[:-2]), url)\n    :","commit_id":"a18ad6b5c411ac557b79a667c4a9fe6092a8335a"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2b0adaddce5afc21e3b47d46a6be1d00b586cee0","unresolved":false,"context_lines":[{"line_number":463,"context_line":"                            resp, resp_body)"},{"line_number":464,"context_line":"        return resp, resp_body"},{"line_number":465,"context_line":""},{"line_number":466,"context_line":"    def request_base(self, method, url,"},{"line_number":467,"context_line":"                     headers\u003dNone, body\u003dNone):"},{"line_number":468,"context_line":"        retry \u003d 0"},{"line_number":469,"context_line":"        if (self.token is None) or (self.base_url is None):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fw6k%3D","line":466,"updated":"2013-10-10 06:18:58.000000000","message":"I would prefer that you don\u0027t copy\u0026paste here.","commit_id":"a18ad6b5c411ac557b79a667c4a9fe6092a8335a"}],"tempest/config.py":[{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"001891d3713af7cec6c638d45eaf29da8a1728bf","unresolved":false,"context_lines":[{"line_number":380,"context_line":"               default\u003d5,"},{"line_number":381,"context_line":"               help\u003d\"Number of seconds to wait while looping to check the\""},{"line_number":382,"context_line":"                    \"status of a container to container synchronization\"),"},{"line_number":383,"context_line":"    cfg.BoolOpt(\u0027accounts_quotas_available\u0027,"},{"line_number":384,"context_line":"                default\u003dTrue,"},{"line_number":385,"context_line":"                help\u003d\"Set to True if the Account Quota middleware is enabled\"),"},{"line_number":386,"context_line":"    cfg.BoolOpt(\u0027crossdomain_available\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2FRlM%3D","line":383,"updated":"2013-11-11 21:45:26.000000000","message":"Same comments as in the tempest.conf.sample about these being in the feature-enabled section.","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"cbcd410c719d1084d47d5ca6cd670cf3cd0c1398","unresolved":false,"context_lines":[{"line_number":380,"context_line":"               default\u003d5,"},{"line_number":381,"context_line":"               help\u003d\"Number of seconds to wait while looping to check the\""},{"line_number":382,"context_line":"                    \"status of a container to container synchronization\"),"},{"line_number":383,"context_line":"    cfg.BoolOpt(\u0027accounts_quotas_available\u0027,"},{"line_number":384,"context_line":"                default\u003dTrue,"},{"line_number":385,"context_line":"                help\u003d\"Set to True if the Account Quota middleware is enabled\"),"},{"line_number":386,"context_line":"    cfg.BoolOpt(\u0027crossdomain_available\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F%2FKTE%3D","line":383,"in_reply_to":"AAAATn%2F%2FRlM%3D","updated":"2013-11-13 15:19:51.000000000","message":"Done","commit_id":"c8d0c9617bd5ce9d145408cd6fd7da8373db3da6"}]}