)]}'
{"tempest/api/object_storage/test_healthcheck.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"c30be8e5c559d7a95be81e8ccab37d2aac8bfac5","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        http://192.168.33.10:8080"},{"line_number":40,"context_line":"        \"\"\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        client \u003d self.account_client"},{"line_number":43,"context_line":"        client._set_auth()"},{"line_number":44,"context_line":"        client.base_url \u003d \"/\".join(client.base_url.split(\"/\")[:-2])"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FQGg%3D","line":42,"updated":"2013-10-19 07:16:16.000000000","message":"Please Use addCleanUp for resorting the base url even on failure, \nor explicitly create an account_client instance for this test case.","commit_id":"e9d7868885363c9cb767e63abd5fdc8876f1ccc7"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"b6eb119b5b76b3905897c727dfc9569e006802bb","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        http://192.168.33.10:8080"},{"line_number":40,"context_line":"        \"\"\""},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        client \u003d self.account_client"},{"line_number":43,"context_line":"        client._set_auth()"},{"line_number":44,"context_line":"        client.base_url \u003d \"/\".join(client.base_url.split(\"/\")[:-2])"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F%2B9I%3D","line":42,"in_reply_to":"AAAATH%2F%2FQGg%3D","updated":"2013-10-24 09:07:46.000000000","message":"Done","commit_id":"e9d7868885363c9cb767e63abd5fdc8876f1ccc7"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"41c8248fb6151a7ae7e41302e77ae232e86edada","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        self.set_account_client_url_to_endpoint()"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        resp, body \u003d self.account_client.get(\"healthcheck\", {})"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # clear the base_url for subsequent requests"},{"line_number":55,"context_line":"        self.account_client.base_url \u003d None"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2FHZg%3D","line":53,"updated":"2013-10-22 23:30:33.000000000","message":"If you\u0027re not going to use \u0027body\u0027, please rewrite this line like this:\n\nresp, _ \u003d self.account_client.get(\"healthcheck\", {})\n\n\nWhich looks more Pythonic to me: it\u0027s a convention to name throwaway variables like this.","commit_id":"e9d7868885363c9cb767e63abd5fdc8876f1ccc7"},{"author":{"_account_id":7020,"name":"Joe Hakim Rahme","email":"jhakimra@redhat.com","username":"rahmu"},"change_message_id":"b6eb119b5b76b3905897c727dfc9569e006802bb","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        self.set_account_client_url_to_endpoint()"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        resp, body \u003d self.account_client.get(\"healthcheck\", {})"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"        # clear the base_url for subsequent requests"},{"line_number":55,"context_line":"        self.account_client.base_url \u003d None"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2F%2B9E%3D","line":53,"in_reply_to":"AAAATH%2F%2FHZg%3D","updated":"2013-10-24 09:07:46.000000000","message":"Done","commit_id":"e9d7868885363c9cb767e63abd5fdc8876f1ccc7"},{"author":{"_account_id":6847,"name":"Martina Kollarova","email":"mkollaro@gmail.com","username":"mkollaro"},"change_message_id":"470df13e761985f2b7c7b9ed3cb90225c2b9ce4e","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        cls.data.teardown_all()"},{"line_number":44,"context_line":"        super(HealthcheckTest, cls).tearDownClass()"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def set_account_client_url_to_endpoint(self):"},{"line_number":47,"context_line":"        \"\"\"By default, the account client will point to a URL consisting of the"},{"line_number":48,"context_line":"        endpoint + \"v1/$(tenant_id)\"."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2F5h4%3D","line":46,"updated":"2013-10-25 14:47:15.000000000","message":"I\u0027d maybe put this into a separate function.","commit_id":"8e741e2875176514db57090f5d1a176329bde506"},{"author":{"_account_id":6847,"name":"Martina Kollarova","email":"mkollaro@gmail.com","username":"mkollaro"},"change_message_id":"470df13e761985f2b7c7b9ed3cb90225c2b9ce4e","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        resp, _ \u003d self.os_test_user.account_client.get(\"healthcheck\", {})"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        # clear the base_url for subsequent requests"},{"line_number":75,"context_line":"        self.os_test_user.account_client.base_url \u003d None"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        # The status is expected to be 200"},{"line_number":78,"context_line":"        self.assertIn(int(resp[\u0027status\u0027]), HTTP_SUCCESS)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2F5hs%3D","line":75,"updated":"2013-10-25 14:47:15.000000000","message":"This would be better in tearDown()","commit_id":"8e741e2875176514db57090f5d1a176329bde506"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"b972f4a857732ee64309781dd0a60c97b13a63b2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Copyright (C) 2013 eNovance SAS \u003clicensing@enovance.com\u003e"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAATn%2F%2FsYE%3D","line":1,"updated":"2013-10-30 11:00:48.000000000","message":"I think this line should be removed - this is done in other projects as well: https://bugs.launchpad.net/swift/+bug/1229324","commit_id":"c54a863e8781d80704b8283d0bfe584c582d93a1"}]}