)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b556e111d618dbe9a64d1a2658640de3327910e0","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-07-15 18:57:45 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add \"get_current\" method to VersionManager"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I36e84680c578a88d5c274f8ca24ecd18d1f3a179"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a41bdd9_7875c12f","line":8,"updated":"2015-07-16 06:13:08.000000000","message":"we need reference to the bp","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"e1b65548a7452d8e17dfab8fde4d8545e6e56db5","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-07-15 18:57:45 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add \"get_current\" method to VersionManager"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I36e84680c578a88d5c274f8ca24ecd18d1f3a179"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a41bdd9_aaae817e","line":8,"in_reply_to":"9a41bdd9_7875c12f","updated":"2015-07-16 11:43:33.000000000","message":"Done","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"}],"novaclient/v2/versions.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"353db38bfe02f78acc201a5393a1b9ef230c11e0","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def get_current(self):"},{"line_number":40,"context_line":"        \"\"\"Returns info about current version.\"\"\""},{"line_number":41,"context_line":"        url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":42,"context_line":"        for version in self.list():"},{"line_number":43,"context_line":"            for link in version.links:"},{"line_number":44,"context_line":"                if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_98176544","line":41,"updated":"2015-07-16 06:00:35.000000000","message":"this doesn\u0027t works for SessionClient.\n\nAnd you can access to endpoint version info directly, like this:\n\nendpoint \u003d self.api.client.get_endpoint()\n\nurl \u003d \u0027/\u0027.join(endpoint.split(\u0027/\u0027)[:-1])\n\nreturn self._get(url, \"version\")","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"a2d690ca81be19acd2e0f834d43f3e9dbb5aa14d","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def get_current(self):"},{"line_number":40,"context_line":"        \"\"\"Returns info about current version.\"\"\""},{"line_number":41,"context_line":"        url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":42,"context_line":"        for version in self.list():"},{"line_number":43,"context_line":"            for link in version.links:"},{"line_number":44,"context_line":"                if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_76be2daf","line":41,"in_reply_to":"9a41bdd9_023443f9","updated":"2015-07-16 09:19:56.000000000","message":"yea, we can learn the check at line 51, then doing different process","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"e1b65548a7452d8e17dfab8fde4d8545e6e56db5","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def get_current(self):"},{"line_number":40,"context_line":"        \"\"\"Returns info about current version.\"\"\""},{"line_number":41,"context_line":"        url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":42,"context_line":"        for version in self.list():"},{"line_number":43,"context_line":"            for link in version.links:"},{"line_number":44,"context_line":"                if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_eab809ca","line":41,"in_reply_to":"9a41bdd9_76be2daf","updated":"2015-07-16 11:43:33.000000000","message":"Done","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"cd4f4c567e66653897aa6bcd2cee82be5a6136d2","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def get_current(self):"},{"line_number":40,"context_line":"        \"\"\"Returns info about current version.\"\"\""},{"line_number":41,"context_line":"        url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":42,"context_line":"        for version in self.list():"},{"line_number":43,"context_line":"            for link in version.links:"},{"line_number":44,"context_line":"                if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_023443f9","line":41,"in_reply_to":"9a41bdd9_98176544","updated":"2015-07-16 08:02:55.000000000","message":"`get_endpoint` is not implemented by HTTPClient. It looks like we have very different interfaces for SessionClient and HTTPClient.","commit_id":"d999e391fe3147e608129f7a123b80e8b8ba0377"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b0f385306b79345e484945aef9d551d8c6665a8f","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        else:"},{"line_number":45,"context_line":"            # NOTE(andreykurilin): HTTPClient doesn\u0027t have ability to send get"},{"line_number":46,"context_line":"            # request without token in the url, so `self._get` doesn\u0027t work."},{"line_number":47,"context_line":"            url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":48,"context_line":"            for version in self.list():"},{"line_number":49,"context_line":"                for link in version.links:"},{"line_number":50,"context_line":"                    if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_467d1cb0","line":47,"updated":"2015-07-21 07:13:32.000000000","message":"In the begin, if there isn\u0027t any call trigger, the management_url is None","commit_id":"3e016c9d1cccf42d8efba8edac6ae34e50e23515"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"9b5ed7df3a7221bfb0ccb8f0520b94207286f723","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        else:"},{"line_number":45,"context_line":"            # NOTE(andreykurilin): HTTPClient doesn\u0027t have ability to send get"},{"line_number":46,"context_line":"            # request without token in the url, so `self._get` doesn\u0027t work."},{"line_number":47,"context_line":"            url \u003d self.client.management_url.rsplit(\"/\", 1)[0]"},{"line_number":48,"context_line":"            for version in self.list():"},{"line_number":49,"context_line":"                for link in version.links:"},{"line_number":50,"context_line":"                    if link[\"href\"] \u003d\u003d url:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_6fdcabe4","line":47,"in_reply_to":"3a50d1a3_467d1cb0","updated":"2015-07-21 08:30:20.000000000","message":"Done","commit_id":"3e016c9d1cccf42d8efba8edac6ae34e50e23515"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c7e47154d4c42c47e6764a6d1efaf7318ce1fb4d","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        \"\"\"Returns info about current version.\"\"\""},{"line_number":41,"context_line":"        if self._is_session_client():"},{"line_number":42,"context_line":"            url \u003d self.api.client.get_endpoint().rsplit(\"/\", 1)[0]"},{"line_number":43,"context_line":"            self._get(url, \"version\")"},{"line_number":44,"context_line":"        else:"},{"line_number":45,"context_line":"            # NOTE(andreykurilin): HTTPClient doesn\u0027t have ability to send get"},{"line_number":46,"context_line":"            # request without token in the url, so `self._get` doesn\u0027t work."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_172bceec","line":43,"updated":"2015-07-25 06:33:32.000000000","message":"missing a return at here","commit_id":"983614660b87b49c9b73c00622d84b8045b2c5cc"}]}