)]}'
{"openstackclient/common/clientmanager.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e53cc51e0e56e3fdeb4c944619417d450b42e5ee","unresolved":false,"context_lines":[{"line_number":135,"context_line":"        # NOTE(jcross): Cinder did some interesting things with their service"},{"line_number":136,"context_line":"        #               name so we need to figure out which version to look"},{"line_number":137,"context_line":"        #               for when calling is_service_available()"},{"line_number":138,"context_line":"        volume_version \u003d volume_client.api_version.ver_major"},{"line_number":139,"context_line":"        return self.is_service_available("},{"line_number":140,"context_line":"            \"volumev%s\" % volume_version) is not False"},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_b6b776f5","line":138,"updated":"2018-04-03 04:18:08.000000000","message":"this is the only part that concerns me -- it doesn\u0027t address the case where the service name is just \"volume\".\n\nthe volumev1, volumev2, volumev3 stuff is a failure in the way we deploy devstack (IMO) \n\nsee the install instructions (https://docs.openstack.org/mitaka/install-guide-rdo/cinder-controller-install.html) -- we actually recommend folks deploy it the service as just \"volume\"\n\n  $ openstack service create --name cinder \\\n  --description \"OpenStack Block Storage\" volume","commit_id":"fe4d8a6184b29bb2c954cd5de20763a9218f85fd"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"acd9fad6b6c10ab88ed83e5cf47d35fb243a1716","unresolved":false,"context_lines":[{"line_number":135,"context_line":"        # NOTE(jcross): Cinder did some interesting things with their service"},{"line_number":136,"context_line":"        #               name so we need to figure out which version to look"},{"line_number":137,"context_line":"        #               for when calling is_service_available()"},{"line_number":138,"context_line":"        volume_version \u003d volume_client.api_version.ver_major"},{"line_number":139,"context_line":"        return self.is_service_available("},{"line_number":140,"context_line":"            \"volumev%s\" % volume_version) is not False"},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_576ff429","line":138,"in_reply_to":"bf659307_b6b776f5","updated":"2018-04-03 07:29:17.000000000","message":"Good point! I will come up with a better way to to this.","commit_id":"fe4d8a6184b29bb2c954cd5de20763a9218f85fd"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"3579912922bf562523ee5533682d1a00713c28ec","unresolved":false,"context_lines":[{"line_number":135,"context_line":"        # NOTE(jcross): Cinder did some interesting things with their service"},{"line_number":136,"context_line":"        #               name so we need to figure out which version to look"},{"line_number":137,"context_line":"        #               for when calling is_service_available()"},{"line_number":138,"context_line":"        volume_version \u003d volume_client.api_version.ver_major"},{"line_number":139,"context_line":"        return self.is_service_available("},{"line_number":140,"context_line":"            \"volumev%s\" % volume_version) is not False"},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_899a54b6","line":138,"in_reply_to":"bf659307_b6b776f5","updated":"2018-04-03 07:49:24.000000000","message":"I fixed this, I am not sure if it is the best way. LMK what you think.","commit_id":"fe4d8a6184b29bb2c954cd5de20763a9218f85fd"}]}