)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"8f67219f38dece3fb3d0f1f6016809cc2ab3b1e1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add backup import/export to OpenStack Client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch implements import and export of backup metadata in the OSC."},{"line_number":10,"context_line":"BP link:https://blueprints.launchpad.net/python-openstackclient/+spec/cinder-command-support"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Co-Authored-By: jiahui.qiang \u003cjiahui.qiang@easystack.cn\u003e"},{"line_number":13,"context_line":"Change-Id: I73f903ffb2f08ddfdf731f4ab789543a238a0d8d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da4df55a_9c4bc886","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":92},"updated":"2016-12-28 02:18:03.000000000","message":"Please use the commom format:\nImplements: bp cinder-command-support","commit_id":"97644569918cac909f04373c7795a54a80784585"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"8fd8bf5d05e5862c950fab6452692f902d49ab50","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add backup import/export to OpenStack Client"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch implements import and export of backup metadata in the OSC."},{"line_number":10,"context_line":"BP link:https://blueprints.launchpad.net/python-openstackclient/+spec/cinder-command-support"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Co-Authored-By: jiahui.qiang \u003cjiahui.qiang@easystack.cn\u003e"},{"line_number":13,"context_line":"Change-Id: I73f903ffb2f08ddfdf731f4ab789543a238a0d8d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"da4df55a_eb57a457","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":92},"in_reply_to":"da4df55a_9c4bc886","updated":"2016-12-28 06:56:17.000000000","message":"Done","commit_id":"97644569918cac909f04373c7795a54a80784585"}],"doc/source/command-objects/volume-backup.rst":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":93,"context_line":".. _volume_backup_export-backup:"},{"line_number":94,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    Backup service to use for importing the backup."},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"volume backup import"},{"line_number":99,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a74a57a_21bd5302","line":96,"updated":"2016-11-27 13:55:55.000000000","message":"doc should match the help message","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    Backup to import (name or ID)"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a74a57a_a10de3e9","line":111,"range":{"start_line":111,"start_character":14,"end_line":111,"end_character":22},"updated":"2016-11-27 13:55:55.000000000","message":"backup-service?","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    Backup to import (name or ID)"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":".. describe:: \u003cbackup_url\u003e"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5a74a57a_c112df08","line":113,"updated":"2016-11-27 13:55:55.000000000","message":"doc should match the help message","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"19270015bec27adf8a6b3bc70fa21133c73c0c46","unresolved":false,"context_lines":[{"line_number":82,"context_line":"volume backup export"},{"line_number":83,"context_line":"--------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Export backup metadata record"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":".. program:: volume backup export"},{"line_number":88,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_cb4a8bac","line":85,"range":{"start_line":85,"start_character":14,"end_line":85,"end_character":29},"updated":"2016-12-19 09:11:15.000000000","message":"Metadata record is a record in Block Storage database. You can restore a volume from a backup as long as the backup\u0027s associated database information (or backup metadata) is intact in the Block Storage database. We can see more details in this  http://docs.openstack.org/admin-guide/blockstorage-volume-backups-export-import.html","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":82,"context_line":"volume backup export"},{"line_number":83,"context_line":"--------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Export backup metadata record"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":".. program:: volume backup export"},{"line_number":88,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_4cee38b5","line":85,"range":{"start_line":85,"start_character":14,"end_line":85,"end_character":29},"updated":"2016-12-10 03:50:29.000000000","message":"what\u0027s a metadata record? should this just be:\n\n  Export volume backup?","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"d113614b71934faabccfc0c7f43150af3b2997bd","unresolved":false,"context_lines":[{"line_number":82,"context_line":"volume backup export"},{"line_number":83,"context_line":"--------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Export backup metadata record"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":".. program:: volume backup export"},{"line_number":88,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_29945041","line":85,"in_reply_to":"3a71b18c_4cee38b5","updated":"2016-12-15 14:43:44.000000000","message":"I think this needs a longer explanation describing what is being exported, making it clear it is not a data export.  Is this exported to the user\u0027s local machine or is it still on the server somewhere?\n\nDitto the import.","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":93,"context_line":".. _volume_backup_export-backup:"},{"line_number":94,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    Backup to import (name or ID)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"volume backup import"},{"line_number":99,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_6c04d47d","line":96,"range":{"start_line":96,"start_character":4,"end_line":96,"end_character":10},"updated":"2016-12-10 03:50:29.000000000","message":"Call this \"Volume backup\", this occurs in many spots in the doc","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":98,"context_line":"volume backup import"},{"line_number":99,"context_line":"--------------------"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Import backup metadata record"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":".. program:: volume backup import"},{"line_number":104,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_4c4558b1","line":101,"range":{"start_line":101,"start_character":14,"end_line":101,"end_character":29},"updated":"2016-12-10 03:50:29.000000000","message":"same question and comment as on line 85","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"19270015bec27adf8a6b3bc70fa21133c73c0c46","unresolved":false,"context_lines":[{"line_number":104,"context_line":".. code:: bash"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    os volume backup import"},{"line_number":107,"context_line":"        \u003cbackup_service\u003e"},{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup-service\u003eBackup to import (name or ID)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_6b4a5729","line":108,"range":{"start_line":107,"start_character":8,"end_line":108,"end_character":20},"updated":"2016-12-19 09:11:15.000000000","message":"This two required option may be values for a metadata record.https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volume_backups.py","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":104,"context_line":".. code:: bash"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    os volume backup import"},{"line_number":107,"context_line":"        \u003cbackup_service\u003e"},{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup-service\u003eBackup to import (name or ID)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_4c1c78c1","line":108,"range":{"start_line":107,"start_character":8,"end_line":108,"end_character":20},"updated":"2016-12-10 03:50:29.000000000","message":"oh another 2 required option command, interesting.","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"d113614b71934faabccfc0c7f43150af3b2997bd","unresolved":false,"context_lines":[{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    os volume backup import"},{"line_number":107,"context_line":"        \u003cbackup_service\u003e"},{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup-service\u003eBackup to import (name or ID)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_c9ad9c85","line":108,"in_reply_to":"3a71b18c_490ee38c","updated":"2016-12-15 14:43:44.000000000","message":"URL is required?  So these commands are not symmetrical?  A user can\u0027t do an export then turn around and re-import it without hosting it somewhere?\n\nAlso, where is the backup name/ID specified, in the import data?","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"f9b15e4cd079f0e92e35550aed1cb6a2243bdccd","unresolved":false,"context_lines":[{"line_number":104,"context_line":".. code:: bash"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    os volume backup import"},{"line_number":107,"context_line":"        \u003cbackup_service\u003e"},{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup-service\u003eBackup to import (name or ID)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_490ee38c","line":108,"range":{"start_line":107,"start_character":8,"end_line":108,"end_character":20},"in_reply_to":"3a71b18c_4c1c78c1","updated":"2016-12-12 13:17:36.000000000","message":"Should these be required options?:\n    --backup-service \u003cbackup-service\u003e\n    --backup-url \u003cbackup-url\u003e","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        \u003cbackup_url\u003e"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":".. _volume_backup_import-backup:"},{"line_number":111,"context_line":".. describe:: \u003cbackup-service\u003eBackup to import (name or ID)"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"    Backup service to use for importing the backup."},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a71b18c_acb0acde","line":111,"range":{"start_line":111,"start_character":30,"end_line":111,"end_character":59},"updated":"2016-12-10 03:50:29.000000000","message":"remove this","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"954d9ad01b1fc7ab2eb824fbc18831fe6e0a42c1","unresolved":false,"context_lines":[{"line_number":94,"context_line":".. _volume_backup_export-backup:"},{"line_number":95,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    Volume backup to import (name or ID)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"volume backup import"},{"line_number":100,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_3825065c","line":97,"range":{"start_line":97,"start_character":21,"end_line":97,"end_character":27},"updated":"2016-12-19 11:05:55.000000000","message":"export?","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"aef57d190e44fa4fc0e9dc0f05197bfe8cce540f","unresolved":false,"context_lines":[{"line_number":94,"context_line":".. _volume_backup_export-backup:"},{"line_number":95,"context_line":".. describe:: \u003cbackup\u003e"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    Volume backup to import (name or ID)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"volume backup import"},{"line_number":100,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_179018ec","line":97,"range":{"start_line":97,"start_character":21,"end_line":97,"end_character":27},"in_reply_to":"1a6eadb0_3825065c","updated":"2016-12-19 11:42:16.000000000","message":"Done","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"954d9ad01b1fc7ab2eb824fbc18831fe6e0a42c1","unresolved":false,"context_lines":[{"line_number":106,"context_line":".. code:: bash"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    os volume backup import"},{"line_number":109,"context_line":"        --backup_service \u003cbackup-service\u003e"},{"line_number":110,"context_line":"        --backup_url \u003cbackup-url\u003e"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. _volume_backup_import-backup:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_98ca9a6d","line":109,"range":{"start_line":109,"start_character":10,"end_line":109,"end_character":24},"updated":"2016-12-19 11:05:55.000000000","message":"backup-service","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"aef57d190e44fa4fc0e9dc0f05197bfe8cce540f","unresolved":false,"context_lines":[{"line_number":106,"context_line":".. code:: bash"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    os volume backup import"},{"line_number":109,"context_line":"        --backup_service \u003cbackup-service\u003e"},{"line_number":110,"context_line":"        --backup_url \u003cbackup-url\u003e"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. _volume_backup_import-backup:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_57c0d0e0","line":109,"range":{"start_line":109,"start_character":10,"end_line":109,"end_character":24},"in_reply_to":"1a6eadb0_98ca9a6d","updated":"2016-12-19 11:42:16.000000000","message":"Done","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"954d9ad01b1fc7ab2eb824fbc18831fe6e0a42c1","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    os volume backup import"},{"line_number":109,"context_line":"        --backup_service \u003cbackup-service\u003e"},{"line_number":110,"context_line":"        --backup_url \u003cbackup-url\u003e"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_9891ba55","line":110,"range":{"start_line":110,"start_character":10,"end_line":110,"end_character":20},"updated":"2016-12-19 11:05:55.000000000","message":"backup-url","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"aef57d190e44fa4fc0e9dc0f05197bfe8cce540f","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    os volume backup import"},{"line_number":109,"context_line":"        --backup_service \u003cbackup-service\u003e"},{"line_number":110,"context_line":"        --backup_url \u003cbackup-url\u003e"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_77b5cc7c","line":110,"range":{"start_line":110,"start_character":10,"end_line":110,"end_character":20},"in_reply_to":"1a6eadb0_9891ba55","updated":"2016-12-19 11:42:16.000000000","message":"Done","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"954d9ad01b1fc7ab2eb824fbc18831fe6e0a42c1","unresolved":false,"context_lines":[{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    Volume backup service to use for importing the backup."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_f34bb15d","line":115,"updated":"2016-12-19 11:05:55.000000000","message":"add \"(required)\" in the end of the help message","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"aef57d190e44fa4fc0e9dc0f05197bfe8cce540f","unresolved":false,"context_lines":[{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    Volume backup service to use for importing the backup."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_97ba686d","line":115,"in_reply_to":"1a6eadb0_f34bb15d","updated":"2016-12-19 11:42:16.000000000","message":"Done","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"954d9ad01b1fc7ab2eb824fbc18831fe6e0a42c1","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    Volume backup URL for importing the backup metadata."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"volume backup list"},{"line_number":122,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_33560907","line":119,"updated":"2016-12-19 11:05:55.000000000","message":"add \"(required)\" in the end of the help message","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"aef57d190e44fa4fc0e9dc0f05197bfe8cce540f","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    Volume backup URL for importing the backup metadata."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"volume backup list"},{"line_number":122,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_d75a2050","line":119,"in_reply_to":"1a6eadb0_33560907","updated":"2016-12-19 11:42:16.000000000","message":"If option is set to (required\u003dTrue), do they all need to add \u0027required\u0027 in the help?","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"43869dd44bf29fe9da7eaaf20dbc33157fea4536","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    Volume backup URL for importing the backup metadata."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"volume backup list"},{"line_number":122,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a6eadb0_7a569107","line":119,"in_reply_to":"1a6eadb0_d75a2050","updated":"2016-12-19 11:44:23.000000000","message":"Not a hard rule but I think we\u0027d better do it to help users know more about this option","commit_id":"021eef92ee86a0dbe4b5f7a885c03623c82d5e5b"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"ee4ea91c8d89e01c13ee308bc66e109dcfc0f5b1","unresolved":false,"context_lines":[{"line_number":82,"context_line":"volume backup export"},{"line_number":83,"context_line":"--------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Export volume backup metadata record. (restoring a volume"},{"line_number":86,"context_line":"requires backup metadata intact in the Block Storage database.)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":".. program:: volume backup export"},{"line_number":89,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_04ba490a","line":86,"range":{"start_line":85,"start_character":0,"end_line":86,"end_character":63},"updated":"2016-12-21 02:01:35.000000000","message":"\"backing up a volume and exporting its metadata will allow you to restore the volume on a completely different Block Storage database, or even on a different cloud service.\"  It is difficult to contain all in one help. it still need user to read the API doc.","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":82,"context_line":"volume backup export"},{"line_number":83,"context_line":"--------------------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"Export volume backup metadata record. (restoring a volume"},{"line_number":86,"context_line":"requires backup metadata intact in the Block Storage database.)"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":".. program:: volume backup export"},{"line_number":89,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_f986ebd9","line":86,"range":{"start_line":85,"start_character":0,"end_line":86,"end_character":63},"updated":"2016-12-20 15:24:24.000000000","message":"This explanation is still not make sense for me, what about:\n\"Export volume backup metadata record, which allows you to completely restore the backup, even in the event of a catastrophic database failure\"\nI don\u0027t sure whether it is make sense or not, hope to get some more suggestions about it.","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":99,"context_line":"volume backup import"},{"line_number":100,"context_line":"--------------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Import volume backup metadata record to the Block Storage"},{"line_number":103,"context_line":"database and then restore the backup."},{"line_number":104,"context_line":""},{"line_number":105,"context_line":".. program:: volume backup import"},{"line_number":106,"context_line":".. code:: bash"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_9963371a","line":103,"range":{"start_line":102,"start_character":0,"end_line":103,"end_character":37},"updated":"2016-12-20 15:24:24.000000000","message":"This explanation seems still not enough for me, what about:\n\"Import volume backup metadata to the Block Storage database to allow to restore the volume backup\"\nI don\u0027t sure whether it is make sense or not, hope to get some more suggestions about it.","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    Volume backup service to use for importing the backup.(required)"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_ae342bfb","line":115,"range":{"start_line":115,"start_character":58,"end_line":115,"end_character":59},"updated":"2016-12-20 15:24:24.000000000","message":"nit: leave one space here","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"ee4ea91c8d89e01c13ee308bc66e109dcfc0f5b1","unresolved":false,"context_lines":[{"line_number":112,"context_line":".. _volume_backup_import-backup:"},{"line_number":113,"context_line":".. option:: --backup-service \u003cbackup-service\u003e"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    Volume backup service to use for importing the backup.(required)"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_c4f4d1d0","line":115,"range":{"start_line":115,"start_character":58,"end_line":115,"end_character":59},"in_reply_to":"1a6eadb0_ae342bfb","updated":"2016-12-21 02:01:35.000000000","message":"OK, I will fix it with the above problem in the next patch.:)","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":".. option:: --backup-url \u003cbackup-url\u003e"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    Volume backup URL for importing the backup metadata.(required)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"volume backup list"},{"line_number":122,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1a6eadb0_0e26ffcd","line":119,"range":{"start_line":119,"start_character":56,"end_line":119,"end_character":57},"updated":"2016-12-20 15:24:24.000000000","message":"nit: leave one space here","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"}],"doc/source/commands.rst":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"8f67219f38dece3fb3d0f1f6016809cc2ab3b1e1","unresolved":false,"context_lines":[{"line_number":229,"context_line":"* ``create`` (``delete``) - create a new occurrence of the specified object"},{"line_number":230,"context_line":"* ``delete`` (``create``) - delete specific occurrences of the specified objects"},{"line_number":231,"context_line":"* ``expand`` (``shrink``) - increase the capacity of a cluster"},{"line_number":232,"context_line":"* ``import`` (``export``) - import volume backup metadata record to the Block Storage database"},{"line_number":233,"context_line":"* ``issue`` (``revoke``) - issue a token"},{"line_number":234,"context_line":"* ``list`` - display summary information about multiple objects"},{"line_number":235,"context_line":"* ``lock`` (``unlock``) - lock one or more servers so that non-admin user won\u0027t be able to execute actions"}],"source_content_type":"text/x-rst","patch_set":7,"id":"da4df55a_9cef4886","line":232,"updated":"2016-12-28 02:18:03.000000000","message":"missing export","commit_id":"97644569918cac909f04373c7795a54a80784585"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"8fd8bf5d05e5862c950fab6452692f902d49ab50","unresolved":false,"context_lines":[{"line_number":229,"context_line":"* ``create`` (``delete``) - create a new occurrence of the specified object"},{"line_number":230,"context_line":"* ``delete`` (``create``) - delete specific occurrences of the specified objects"},{"line_number":231,"context_line":"* ``expand`` (``shrink``) - increase the capacity of a cluster"},{"line_number":232,"context_line":"* ``import`` (``export``) - import volume backup metadata record to the Block Storage database"},{"line_number":233,"context_line":"* ``issue`` (``revoke``) - issue a token"},{"line_number":234,"context_line":"* ``list`` - display summary information about multiple objects"},{"line_number":235,"context_line":"* ``lock`` (``unlock``) - lock one or more servers so that non-admin user won\u0027t be able to execute actions"}],"source_content_type":"text/x-rst","patch_set":7,"id":"da4df55a_6b4c94c8","line":232,"in_reply_to":"da4df55a_9cef4886","updated":"2016-12-28 06:56:17.000000000","message":"Thank you Ao!","commit_id":"97644569918cac909f04373c7795a54a80784585"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"fe38b027ef47330634e7f9820a83468c8bd1d3ed","unresolved":false,"context_lines":[{"line_number":236,"context_line":"* ``delete`` (``create``) - delete specific occurrences of the specified objects"},{"line_number":237,"context_line":"* ``expand`` (``shrink``) - increase the capacity of a cluster"},{"line_number":238,"context_line":"* ``import`` (``export``) - import volume backup metadata record to the Block Storage database"},{"line_number":239,"context_line":"* ``export`` (``import``) - Export volume backup metadata record to the Block Storage database"},{"line_number":240,"context_line":"* ``issue`` (``revoke``) - issue a token"},{"line_number":241,"context_line":"* ``list`` - display summary information about multiple objects"},{"line_number":242,"context_line":"* ``lock`` (``unlock``) - lock one or more servers so that non-admin user won\u0027t be able to execute actions"}],"source_content_type":"text/x-rst","patch_set":8,"id":"da4df55a_ea831454","line":239,"updated":"2016-12-28 13:21:07.000000000","message":"s/Export/export/","commit_id":"023094217365d9faa674501462938499ec95f886"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"4fcc41445436e8a4a040365854770e699f52668c","unresolved":false,"context_lines":[{"line_number":236,"context_line":"* ``delete`` (``create``) - delete specific occurrences of the specified objects"},{"line_number":237,"context_line":"* ``expand`` (``shrink``) - increase the capacity of a cluster"},{"line_number":238,"context_line":"* ``import`` (``export``) - import volume backup metadata record to the Block Storage database"},{"line_number":239,"context_line":"* ``export`` (``import``) - Export volume backup metadata record to the Block Storage database"},{"line_number":240,"context_line":"* ``issue`` (``revoke``) - issue a token"},{"line_number":241,"context_line":"* ``list`` - display summary information about multiple objects"},{"line_number":242,"context_line":"* ``lock`` (``unlock``) - lock one or more servers so that non-admin user won\u0027t be able to execute actions"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ba5201f7_3530b607","line":239,"in_reply_to":"ba5201f7_44cf197f","updated":"2017-01-09 22:48:19.000000000","message":"I agree. leave out metadata and just refer to this as adding a record.","commit_id":"023094217365d9faa674501462938499ec95f886"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"fe7bbbd8298d304e3cf286826e5cd7ce24901c7e","unresolved":false,"context_lines":[{"line_number":236,"context_line":"* ``delete`` (``create``) - delete specific occurrences of the specified objects"},{"line_number":237,"context_line":"* ``expand`` (``shrink``) - increase the capacity of a cluster"},{"line_number":238,"context_line":"* ``import`` (``export``) - import volume backup metadata record to the Block Storage database"},{"line_number":239,"context_line":"* ``export`` (``import``) - Export volume backup metadata record to the Block Storage database"},{"line_number":240,"context_line":"* ``issue`` (``revoke``) - issue a token"},{"line_number":241,"context_line":"* ``list`` - display summary information about multiple objects"},{"line_number":242,"context_line":"* ``lock`` (``unlock``) - lock one or more servers so that non-admin user won\u0027t be able to execute actions"}],"source_content_type":"text/x-rst","patch_set":8,"id":"ba5201f7_44cf197f","line":239,"in_reply_to":"da4df55a_ea831454","updated":"2017-01-03 22:52:34.000000000","message":"These should not be worded to be specific to the volume backup command, this is the defining list of the common actions in OSC so every resource that uses them does essentially the same thing.\n\nSo I see below that we still haven\u0027t removed that from existing actions...so how about this, rewrite it to use volume backup as an example:\n\n  - export resource information, for example a volume backup database record\n\nAlso, sort the actions into the list in alpha order.","commit_id":"023094217365d9faa674501462938499ec95f886"}],"openstackclient/tests/unit/volume/v2/test_backup.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":234,"context_line":"            )"},{"line_number":235,"context_line":""},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"class TestBackupExport(TestBackup):"},{"line_number":238,"context_line":""},{"line_number":239,"context_line":"    backup_obj \u003d volume_fakes.FakeBackup.create_one_backup()"},{"line_number":240,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_cc86c8da","line":237,"range":{"start_line":237,"start_character":6,"end_line":237,"end_character":22},"updated":"2016-12-10 03:50:29.000000000","message":"this test fails","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":258,"context_line":"        self.assertIsNone(result)"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"class TestBackupImport(TestBackup):"},{"line_number":262,"context_line":"    def setUp(self):"},{"line_number":263,"context_line":"        super(TestBackupImport, self).setUp()"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"class TestBackupList(TestBackup):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_ac834ccb","line":263,"range":{"start_line":261,"start_character":0,"end_line":263,"end_character":45},"updated":"2016-12-10 03:50:29.000000000","message":"test needs to be completed","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"}],"openstackclient/volume/v2/backup.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class CreateVolumeBackup(command.ShowOne):"},{"line_number":33,"context_line":"    \"\"\"Create new volume backup\"\"\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def get_parser(self, prog_name):"},{"line_number":36,"context_line":"        parser \u003d super(CreateVolumeBackup, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_d0cd4fb8","line":33,"range":{"start_line":33,"start_character":4,"end_line":33,"end_character":34},"updated":"2016-11-27 02:34:03.000000000","message":"undo these changes","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"class ExportVolumeBackup(command.ShowOne):"},{"line_number":173,"context_line":"    \"\"\"Export volume backup metadata record.\"\"\""},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"    def get_parser(self, prog_name):"},{"line_number":176,"context_line":"        parser \u003d super(ExportVolumeBackup, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_b03ff385","line":173,"range":{"start_line":173,"start_character":3,"end_line":173,"end_character":47},"updated":"2016-11-27 02:34:03.000000000","message":"we now use _description \u003d _(\"\u003chelp description\u003e\") instead of a docstring, so we can mark the string for translation","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        parser.add_argument("},{"line_number":178,"context_line":"            \"backup\","},{"line_number":179,"context_line":"            metavar\u003d\"\u003cbackup\u003e\","},{"line_number":180,"context_line":"            help\u003d\"ID of the backup to export.\""},{"line_number":181,"context_line":"        )"},{"line_number":182,"context_line":"        return parser"},{"line_number":183,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_104b472b","line":180,"range":{"start_line":180,"start_character":18,"end_line":180,"end_character":44},"updated":"2016-11-27 02:34:03.000000000","message":"this should work with both ID or name, since you\u0027re using the find_resource method, so the help should read:\n\n  \"Backup to export (name or ID)\"","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        parser.add_argument("},{"line_number":178,"context_line":"            \"backup\","},{"line_number":179,"context_line":"            metavar\u003d\"\u003cbackup\u003e\","},{"line_number":180,"context_line":"            help\u003d\"ID of the backup to export.\""},{"line_number":181,"context_line":"        )"},{"line_number":182,"context_line":"        return parser"},{"line_number":183,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_c129ff38","line":180,"range":{"start_line":180,"start_character":18,"end_line":180,"end_character":44},"in_reply_to":"5a74a57a_104b472b","updated":"2016-11-27 13:55:55.000000000","message":"++","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":189,"context_line":""},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"class ImportVolumeBackup(command.ShowOne):"},{"line_number":192,"context_line":"    \"\"\"Export volume backup metadata record.\"\"\""},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def get_parser(self, prog_name):"},{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_50513fd7","line":192,"range":{"start_line":192,"start_character":4,"end_line":192,"end_character":47},"updated":"2016-11-27 02:34:03.000000000","message":"same comment as above","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":194,"context_line":"    def get_parser(self, prog_name):"},{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"},{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup_service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_b05613cc","line":197,"range":{"start_line":197,"start_character":13,"end_line":197,"end_character":27},"updated":"2016-11-27 02:34:03.000000000","message":"this is how the argument appears in the help, and we always use hyphens instead of underscores, so `backup-service` and `backup-url`","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":194,"context_line":"    def get_parser(self, prog_name):"},{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"},{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup_service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_41ffafa9","line":197,"range":{"start_line":197,"start_character":13,"end_line":197,"end_character":27},"in_reply_to":"5a74a57a_b05613cc","updated":"2016-11-27 13:55:55.000000000","message":"As a positional parameter, this should be \"backup_service\". And the one below should be \"backup-service\" since it will actually appear in the help :)","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"4f27809d78bab572a46562030c1d6de9de067552","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"},{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup_service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"},{"line_number":201,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_d05b8ff4","line":198,"range":{"start_line":198,"start_character":22,"end_line":198,"end_character":36},"updated":"2016-11-27 02:34:03.000000000","message":"this should be updated to `backup-service` and `backup-url` too, when calling them via parsed_args you still refer to them by backup_service and backup_url, hope that makes sense :)","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"4517d290bfe91c0d769679d0853f8421ce06045a","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"},{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup_service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"},{"line_number":201,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":2,"id":"5a74a57a_612e6b44","line":198,"range":{"start_line":198,"start_character":22,"end_line":198,"end_character":36},"in_reply_to":"5a74a57a_d05b8ff4","updated":"2016-11-27 13:55:55.000000000","message":"This should be \"backup-service\"","commit_id":"f66ecb254bede73fec603d63d06873d0dc5b381b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"f9b15e4cd079f0e92e35550aed1cb6a2243bdccd","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        parser.add_argument("},{"line_number":178,"context_line":"            \"backup\","},{"line_number":179,"context_line":"            metavar\u003d\"\u003cbackup\u003e\","},{"line_number":180,"context_line":"            help\u003d\"Backup to export (name or ID)\""},{"line_number":181,"context_line":"        )"},{"line_number":182,"context_line":"        return parser"},{"line_number":183,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_09bcebb9","line":180,"range":{"start_line":180,"start_character":17,"end_line":180,"end_character":48},"updated":"2016-12-12 13:17:36.000000000","message":"cover all help messages into _()","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":194,"context_line":"    def get_parser(self, prog_name):"},{"line_number":195,"context_line":"        parser \u003d super(ImportVolumeBackup, self).get_parser(prog_name)"},{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup-service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_9f85b4d0","line":197,"range":{"start_line":197,"start_character":13,"end_line":197,"end_character":27},"updated":"2016-12-10 03:50:29.000000000","message":"this should be `backup-service`","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":196,"context_line":"        parser.add_argument("},{"line_number":197,"context_line":"            \"backup_service\","},{"line_number":198,"context_line":"            metavar\u003d\"\u003cbackup-service\u003e\","},{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"},{"line_number":201,"context_line":"        parser.add_argument("},{"line_number":202,"context_line":"            \"backup_url\","}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_1f728411","line":199,"range":{"start_line":199,"start_character":18,"end_line":199,"end_character":64},"updated":"2016-12-10 03:50:29.000000000","message":"what exactly is this option? anyway we can reasonably default it? like call the service API and look at the results?","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c97c6553c59a18cb7dd199bc84aabbccb0597f40","unresolved":false,"context_lines":[{"line_number":199,"context_line":"            help\u003d\"Backup service to use for importing the backup.\""},{"line_number":200,"context_line":"        )"},{"line_number":201,"context_line":"        parser.add_argument("},{"line_number":202,"context_line":"            \"backup_url\","},{"line_number":203,"context_line":"            metavar\u003d\"\u003cbackup-url\u003e\","},{"line_number":204,"context_line":"            help\u003d\"Backup URL for importing the backup metadata.\""},{"line_number":205,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a71b18c_df6bac1c","line":202,"range":{"start_line":202,"start_character":13,"end_line":202,"end_character":23},"updated":"2016-12-10 03:50:29.000000000","message":"this should be `backup-url`","commit_id":"47c418afbf4991bfca800d2c715abf4c2122fc6b"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":200,"context_line":"            \"--backup-service\","},{"line_number":201,"context_line":"            metavar\u003d\"\u003cbackup-service\u003e\","},{"line_number":202,"context_line":"            required\u003dTrue,"},{"line_number":203,"context_line":"            help\u003d_(\"Volume backup service to use for importing the backup.\")"},{"line_number":204,"context_line":"        )"},{"line_number":205,"context_line":"        parser.add_argument("},{"line_number":206,"context_line":"            \"--backup-url\","}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_594c3f70","line":203,"updated":"2016-12-20 15:24:24.000000000","message":"Help message should match the doc","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"},{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"dffedd71ade542e5670a886bef72098806469c75","unresolved":false,"context_lines":[{"line_number":206,"context_line":"            \"--backup-url\","},{"line_number":207,"context_line":"            metavar\u003d\"\u003cbackup-url\u003e\","},{"line_number":208,"context_line":"            required\u003dTrue,"},{"line_number":209,"context_line":"            help\u003d_(\"Volume backup URL for importing the backup metadata.\")"},{"line_number":210,"context_line":"        )"},{"line_number":211,"context_line":"        return parser"},{"line_number":212,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_9974179d","line":209,"updated":"2016-12-20 15:24:24.000000000","message":"Help message should match the doc","commit_id":"73c128680b275c3a4af1239189ae248bc3e6a1ed"}],"releasenotes/notes/bp-cinder-command-support-e512343fd63161f2.yaml":[{"author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"change_message_id":"8f67219f38dece3fb3d0f1f6016809cc2ab3b1e1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Add ``volume backup import`` and ``volume backup export`` command, those two commands"},{"line_number":4,"context_line":"    help user import and export metadata record."},{"line_number":5,"context_line":"    [Blueprint `cinder-command-support \u003chttps://blueprints.launchpad.net/python-openstackclient/+spec/cinder-command-support\u003e`_]"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"da4df55a_1cb27894","line":4,"range":{"start_line":4,"start_character":32,"end_line":4,"end_character":47},"updated":"2016-12-28 02:18:03.000000000","message":"nit: volume backup metadata record","commit_id":"97644569918cac909f04373c7795a54a80784585"},{"author":{"_account_id":24080,"name":"jiahui.qiang","email":"jiahui.qiang@easystack.cn","username":"qiangjiahui"},"change_message_id":"8fd8bf5d05e5862c950fab6452692f902d49ab50","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - Add ``volume backup import`` and ``volume backup export`` command, those two commands"},{"line_number":4,"context_line":"    help user import and export metadata record."},{"line_number":5,"context_line":"    [Blueprint `cinder-command-support \u003chttps://blueprints.launchpad.net/python-openstackclient/+spec/cinder-command-support\u003e`_]"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"da4df55a_ebcec42b","line":4,"range":{"start_line":4,"start_character":32,"end_line":4,"end_character":47},"in_reply_to":"da4df55a_1cb27894","updated":"2016-12-28 06:56:17.000000000","message":"Done","commit_id":"97644569918cac909f04373c7795a54a80784585"}]}