)]}'
{"brick_cinderclient_ext/volume_actions.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"474f1b02d2a51c934fb3713586e2e1204d8979fb","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        # NOTE(flaper87): mountpoint doesn\u0027t seem to be used at all, we should"},{"line_number":65,"context_line":"        # perhaps consider removing it. Unsure whether `path` exists for all the"},{"line_number":66,"context_line":"        # different drivers so, using the value of `mountpoint` as the default."},{"line_number":67,"context_line":"        mountpoint \u003d device_info.get(\u0027path\u0027, mountpoint)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        self.volumes_client.volumes.attach(self.volume_id, instance_uuid\u003dNone,"},{"line_number":70,"context_line":"                                           mountpoint\u003dmountpoint,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_bd781e2b","line":67,"range":{"start_line":67,"start_character":8,"end_line":67,"end_character":56},"updated":"2018-05-02 13:39:55.000000000","message":"So you\u0027re correct, this is NOT something we use with local attach, Setting it to path here like you\u0027re doing seems like it\u0027s the logical analogous in the local-attach case.\n\nI\u0027m not sure what you meant by \"remove it\" but I think removing it from the args list in `local-attach` makes sense.  On a side note, if we want this extension to really be usable there are a number of things missing like storing the connector etc.","commit_id":"2146a5e9f59a40fd178b65dc62b02fd4d5b6c800"}]}