)]}'
{"rally/plugins/openstack/scenarios/cinder/volumes.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f7e3754eb296c2764cecb6d2f2eb468bfae7daf0","unresolved":false,"context_lines":[{"line_number":859,"context_line":"        volume1 \u003d service.create_volume(size, **kwargs)"},{"line_number":860,"context_line":"        service.unmanage_volume(volume1)"},{"line_number":861,"context_line":"        host \u003d getattr(volume1, \"os-vol-host-attr:host\")"},{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_3856525e","line":862,"updated":"2017-05-31 16:02:47.000000000","message":"\u0027volume-\u0027 prefix is default but configurable. Please, make it configurable here too.\n\nIn a cinder code, it\u0027s generated by the following code:\ndef name(self):\n        return CONF.volume_name_template % self.name_id","commit_id":"75bc8efa81201f37b51a8a2d2f7cdfd3abd3307b"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"b65784465c310618d7be7733b5813cd847151794","unresolved":false,"context_lines":[{"line_number":859,"context_line":"        volume1 \u003d service.create_volume(size, **kwargs)"},{"line_number":860,"context_line":"        service.unmanage_volume(volume1)"},{"line_number":861,"context_line":"        host \u003d getattr(volume1, \"os-vol-host-attr:host\")"},{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_58bb4671","line":862,"in_reply_to":"df140735_3856525e","updated":"2017-05-31 16:04:09.000000000","message":"Is there a way to auto-discover(obtain) that prefix from cinder api?","commit_id":"75bc8efa81201f37b51a8a2d2f7cdfd3abd3307b"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"7c283a25255f809ae8a8cac28a04175f7c9c337f","unresolved":false,"context_lines":[{"line_number":859,"context_line":"        volume1 \u003d service.create_volume(size, **kwargs)"},{"line_number":860,"context_line":"        service.unmanage_volume(volume1)"},{"line_number":861,"context_line":"        host \u003d getattr(volume1, \"os-vol-host-attr:host\")"},{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_3d30499f","line":862,"in_reply_to":"df140735_58bb4671","updated":"2017-06-02 06:24:14.000000000","message":"It can be configured by change volume_name_template\u0027s value in /etc/cinder/cinder.conf. And I have searched https://developer.openstack.org/api-ref/block-storage/v2/, maybe it can not be obtained from cinder api.\nso, here is the question, how can i get volume_name_template\u0027s value in /etc/cinder/cinder.conf?","commit_id":"75bc8efa81201f37b51a8a2d2f7cdfd3abd3307b"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"12dd42c36b5aad38fd4335edf35dafbdfa604055","unresolved":false,"context_lines":[{"line_number":838,"context_line":"class CreateUnmanageAndManageVolume(scenario.OpenStackScenario):"},{"line_number":839,"context_line":""},{"line_number":840,"context_line":"    def run(self, size, image\u003dNone, description\u003dNone, volume_type\u003dNone,"},{"line_number":841,"context_line":"            availability_zone\u003dNone, metadata\u003dNone, bootable\u003dFalse, **kwargs):"},{"line_number":842,"context_line":"        \"\"\"Create a volume, then unmanage and manage it"},{"line_number":843,"context_line":""},{"line_number":844,"context_line":"        This scenario first creates a volume, then unmanages it, finally"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_5834f2fb","line":841,"range":{"start_line":841,"start_character":67,"end_line":841,"end_character":75},"updated":"2017-07-06 10:12:34.000000000","message":"I prefer to not use kwargs...","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"e3a540b0b4065e19ff7d2d2848f93a0102cb044c","unresolved":false,"context_lines":[{"line_number":838,"context_line":"class CreateUnmanageAndManageVolume(scenario.OpenStackScenario):"},{"line_number":839,"context_line":""},{"line_number":840,"context_line":"    def run(self, size, image\u003dNone, description\u003dNone, volume_type\u003dNone,"},{"line_number":841,"context_line":"            availability_zone\u003dNone, metadata\u003dNone, bootable\u003dFalse, **kwargs):"},{"line_number":842,"context_line":"        \"\"\"Create a volume, then unmanage and manage it"},{"line_number":843,"context_line":""},{"line_number":844,"context_line":"        This scenario first creates a volume, then unmanages it, finally"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_f69f2a4d","line":841,"range":{"start_line":841,"start_character":67,"end_line":841,"end_character":75},"in_reply_to":"3f1d235d_5834f2fb","updated":"2017-07-07 05:12:27.000000000","message":"I will remove kwargs in this patch\u0027s next review, and will modify all the create volume related scenarios in another patch.","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d6204025ae4ff9068d4743ff29e10d8762fa9461","unresolved":false,"context_lines":[{"line_number":859,"context_line":"        volume1 \u003d service.create_volume(size, **kwargs)"},{"line_number":860,"context_line":"        service.unmanage_volume(volume1)"},{"line_number":861,"context_line":"        host \u003d getattr(volume1, \"os-vol-host-attr:host\")"},{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_5ee4db0c","line":862,"updated":"2017-07-06 10:21:23.000000000","message":"Please, move volume prefix or even volume_name_template to scenario config. We can use \u0027volume-{id}\u0027 as a default value for template","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"12dd42c36b5aad38fd4335edf35dafbdfa604055","unresolved":false,"context_lines":[{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"},{"line_number":866,"context_line":"                                        volume_type\u003dvolume_type,"},{"line_number":867,"context_line":"                                        availability_zone\u003davailability_zone,"},{"line_number":868,"context_line":"                                        metadata\u003dmetadata, bootable\u003dbootable)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_7e15573f","line":865,"range":{"start_line":865,"start_character":45,"end_line":865,"end_character":49},"updated":"2017-07-06 10:12:34.000000000","message":"can we transmit random name here, so in case if L869 fail, the resource will be easier to identify?","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"ed2dc272d84bcd12c45219eab06de64a42c4ff3b","unresolved":false,"context_lines":[{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"},{"line_number":866,"context_line":"                                        volume_type\u003dvolume_type,"},{"line_number":867,"context_line":"                                        availability_zone\u003davailability_zone,"},{"line_number":868,"context_line":"                                        metadata\u003dmetadata, bootable\u003dbootable)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_75b090de","line":865,"range":{"start_line":865,"start_character":45,"end_line":865,"end_character":49},"in_reply_to":"3f1d235d_36b7e2f7","updated":"2017-07-08 03:13:53.000000000","message":"i want @Ivan suggested to move volume prefix to https://github.com/openstack/rally/blob/master/rally/plugins/openstack/cfg/cinder.py, not as argument.","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"dd9ee321a5280e4042906e7f3d620e5fd6eafab0","unresolved":false,"context_lines":[{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"},{"line_number":866,"context_line":"                                        volume_type\u003dvolume_type,"},{"line_number":867,"context_line":"                                        availability_zone\u003davailability_zone,"},{"line_number":868,"context_line":"                                        metadata\u003dmetadata, bootable\u003dbootable)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_3ec54d5c","line":865,"range":{"start_line":865,"start_character":45,"end_line":865,"end_character":49},"in_reply_to":"3f1d235d_75b090de","updated":"2017-07-10 01:52:52.000000000","message":"This file can only set values about rally. Can volume_name_template be setted here, and reflect to /etc/cinder/cinder.conf? \nMaybe it can be setted here(not in rally):\nhttps://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"},{"author":{"_account_id":18404,"name":"Ma Xinjian","email":"maxj.fnst@cn.fujitsu.com","username":"maxinjian"},"change_message_id":"e3a540b0b4065e19ff7d2d2848f93a0102cb044c","unresolved":false,"context_lines":[{"line_number":862,"context_line":"        source_name \u003d \"volume-\" + volume1.id"},{"line_number":863,"context_line":"        identifier \u003d {\"source-name\": source_name}"},{"line_number":864,"context_line":"        volume2 \u003d service.manage_volume(host\u003dhost, identifier\u003didentifier,"},{"line_number":865,"context_line":"                                        name\u003dNone, description\u003ddescription,"},{"line_number":866,"context_line":"                                        volume_type\u003dvolume_type,"},{"line_number":867,"context_line":"                                        availability_zone\u003davailability_zone,"},{"line_number":868,"context_line":"                                        metadata\u003dmetadata, bootable\u003dbootable)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f1d235d_36b7e2f7","line":865,"range":{"start_line":865,"start_character":45,"end_line":865,"end_character":49},"in_reply_to":"3f1d235d_7e15573f","updated":"2017-07-07 05:12:27.000000000","message":"Hi Andrey:\n   Again, about volume prefix... I have read Ivan\u0027s message and I know he want to pass volume_name_template manually. But can we do this in rally? Isn\u0027t it auto generated from devstack gate?","commit_id":"f8a2d357a542be2c50f2f98561c9099895ae28f2"}]}