)]}'
{"trove/cmd/manage.py":[{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":94,"context_line":"            \u0027db_sync\u0027, description\u003d\u0027Populate the database structure with the \u0027"},{"line_number":95,"context_line":"            \u0027current version needed by Trove.\u0027)"},{"line_number":96,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_663d892c","line":95,"updated":"2014-07-16 20:27:34.000000000","message":"Suggestion: \"Create and populate the database.\" \n\nNot sure we need to mention version here.","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":94,"context_line":"            \u0027db_sync\u0027, description\u003d\u0027Populate the database structure with the \u0027"},{"line_number":95,"context_line":"            \u0027current version needed by Trove.\u0027)"},{"line_number":96,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_9732c5f0","line":95,"in_reply_to":"baada198_663d892c","updated":"2014-07-16 21:23:14.000000000","message":"+1 to Gregs\u0027 concern","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"2eca88f2b84caa8e30515924d76691146dca9829","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":94,"context_line":"            \u0027db_sync\u0027, description\u003d\u0027Populate the database structure with the \u0027"},{"line_number":95,"context_line":"            \u0027current version needed by Trove.\u0027)"},{"line_number":96,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_373f39d8","line":95,"in_reply_to":"baada198_663d892c","updated":"2014-07-16 22:15:48.000000000","message":"We don\u0027t actually create the database - it must exist.  Changing it to \u0027Populate the database\u0027 would make me think we\u0027re adding data, which is not true.  I\u0027ll change it to \u0027Populate the database structure\u0027","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            \u0027db_upgrade\u0027, description\u003d\u0027Upgrade the database to the \u0027"},{"line_number":100,"context_line":"            \u0027specified version.\u0027)"},{"line_number":101,"context_line":"        parser.add_argument("},{"line_number":102,"context_line":"            \u0027--version\u0027, help\u003d\u0027Version to upgrade to. Defaults to the \u0027"},{"line_number":103,"context_line":"            \u0027current version.\u0027)"},{"line_number":104,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_e610b9ad","line":102,"updated":"2014-07-16 20:27:34.000000000","message":"Suggestion: \"Version to upgrade to.\" --\u003e \"Target version.\"","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            \u0027db_upgrade\u0027, description\u003d\u0027Upgrade the database to the \u0027"},{"line_number":100,"context_line":"            \u0027specified version.\u0027)"},{"line_number":101,"context_line":"        parser.add_argument("},{"line_number":102,"context_line":"            \u0027--version\u0027, help\u003d\u0027Version to upgrade to. Defaults to the \u0027"},{"line_number":103,"context_line":"            \u0027current version.\u0027)"},{"line_number":104,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_3772d9ad","line":102,"in_reply_to":"baada198_e610b9ad","updated":"2014-07-16 21:23:14.000000000","message":"Actually i\u0027m good with proposed naming here","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":107,"context_line":"            \u0027db_downgrade\u0027, description\u003d\u0027Downgrade the database to the \u0027"},{"line_number":108,"context_line":"            \u0027specified version.\u0027)"},{"line_number":109,"context_line":"        parser.add_argument(\u0027version\u0027, help\u003d\u0027Version to downgrade to.\u0027)"},{"line_number":110,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_260b917e","line":109,"updated":"2014-07-16 20:27:34.000000000","message":"target version?","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":107,"context_line":"            \u0027db_downgrade\u0027, description\u003d\u0027Downgrade the database to the \u0027"},{"line_number":108,"context_line":"            \u0027specified version.\u0027)"},{"line_number":109,"context_line":"        parser.add_argument(\u0027version\u0027, help\u003d\u0027Version to downgrade to.\u0027)"},{"line_number":110,"context_line":"        parser.add_argument(\u0027--repo_path\u0027, help\u003drepo_path_help)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_b75d0918","line":109,"in_reply_to":"baada198_260b917e","updated":"2014-07-16 21:23:14.000000000","message":"same","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":113,"context_line":"            \u0027datastore_update\u0027, description\u003d\u0027Add a datastore to Trove. \u0027"},{"line_number":114,"context_line":"            \u0027If the datastore exists, the default version will be updated.\u0027)"},{"line_number":115,"context_line":"        parser.add_argument("},{"line_number":116,"context_line":"            \u0027datastore_name\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":117,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_e65ef978","line":114,"updated":"2014-07-16 20:27:34.000000000","message":"Suggestion: \"Add or update a datastore. If the datastore already exists, the default version will be updated.\"\n\nI\u0027d drop \"to Trove\" here.","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":113,"context_line":"            \u0027datastore_update\u0027, description\u003d\u0027Add a datastore to Trove. \u0027"},{"line_number":114,"context_line":"            \u0027If the datastore exists, the default version will be updated.\u0027)"},{"line_number":115,"context_line":"        parser.add_argument("},{"line_number":116,"context_line":"            \u0027datastore_name\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":117,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_978e4578","line":114,"in_reply_to":"baada198_e65ef978","updated":"2014-07-16 21:23:14.000000000","message":"+1","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        parser.add_argument("},{"line_number":116,"context_line":"            \u0027datastore_name\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":117,"context_line":"        parser.add_argument("},{"line_number":118,"context_line":"            \u0027default_version\u0027, help\u003d\u0027The name or ID of the default datastore \u0027"},{"line_number":119,"context_line":"            \u0027version. If adding a new datastore, pass in an empty string.\u0027)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_668749b7","line":118,"updated":"2014-07-16 20:27:34.000000000","message":"The datastore version must already exist, right? Maybe: \n\"Name or ID of an existing datastore version to set as the default. When adding a new datastore, use an empty string\"","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        parser.add_argument("},{"line_number":116,"context_line":"            \u0027datastore_name\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":117,"context_line":"        parser.add_argument("},{"line_number":118,"context_line":"            \u0027default_version\u0027, help\u003d\u0027The name or ID of the default datastore \u0027"},{"line_number":119,"context_line":"            \u0027version. If adding a new datastore, pass in an empty string.\u0027)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        parser \u003d subparser.add_parser("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_37ce59b5","line":118,"in_reply_to":"baada198_668749b7","updated":"2014-07-16 21:23:14.000000000","message":"Correct. Please add \u0027existing\u0027 word","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":122,"context_line":"            \u0027datastore_version_update\u0027, description\u003d\u0027Add a datastore version \u0027"},{"line_number":123,"context_line":"            \u0027to trove. If the datastore version already exists, all values \u0027"},{"line_number":124,"context_line":"            \u0027except the datastore name and version will be updated.\u0027)"},{"line_number":125,"context_line":"        parser.add_argument(\u0027datastore\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":126,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_c68e55d0","line":123,"updated":"2014-07-16 20:27:34.000000000","message":"Suggestion: \"Add or update a datastore version. If the datastore version already exists...\"","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        parser \u003d subparser.add_parser("},{"line_number":122,"context_line":"            \u0027datastore_version_update\u0027, description\u003d\u0027Add a datastore version \u0027"},{"line_number":123,"context_line":"            \u0027to trove. If the datastore version already exists, all values \u0027"},{"line_number":124,"context_line":"            \u0027except the datastore name and version will be updated.\u0027)"},{"line_number":125,"context_line":"        parser.add_argument(\u0027datastore\u0027, help\u003d\u0027The name of the datastore.\u0027)"},{"line_number":126,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_77bff1fc","line":123,"in_reply_to":"baada198_c68e55d0","updated":"2014-07-16 21:23:14.000000000","message":"+1","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            \u0027manager\u0027, help\u003d\u0027The name of the manager that Trove should use \u0027"},{"line_number":130,"context_line":"            \u0027to administer the datastore.\u0027)"},{"line_number":131,"context_line":"        parser.add_argument("},{"line_number":132,"context_line":"            \u0027image_id\u0027, help\u003d\u0027The Glance ID of the image used to create a new \u0027"},{"line_number":133,"context_line":"            \u0027Trove instance of the datastore version.\u0027)"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027packages\u0027, help\u003d\u0027The packages of the datastore version that are \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_86ab7d1d","line":132,"updated":"2014-07-16 20:27:34.000000000","message":"I haven\u0027t seen \u0027glance id\u0027 used elsewhere in the help, so maybe:\n\n\"ID of the image used to create an instance of the datastore version.\"","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            \u0027manager\u0027, help\u003d\u0027The name of the manager that Trove should use \u0027"},{"line_number":130,"context_line":"            \u0027to administer the datastore.\u0027)"},{"line_number":131,"context_line":"        parser.add_argument("},{"line_number":132,"context_line":"            \u0027image_id\u0027, help\u003d\u0027The Glance ID of the image used to create a new \u0027"},{"line_number":133,"context_line":"            \u0027Trove instance of the datastore version.\u0027)"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027packages\u0027, help\u003d\u0027The packages of the datastore version that are \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_37457917","line":132,"in_reply_to":"baada198_86ab7d1d","updated":"2014-07-16 21:23:14.000000000","message":"+1","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9749,"name":"Greg Lucas","email":"greg@glucas.net","username":"glucas"},"change_message_id":"cdfda8e4e4d3c959bb610f233ec0facd3c7361e3","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            \u0027image_id\u0027, help\u003d\u0027The Glance ID of the image used to create a new \u0027"},{"line_number":133,"context_line":"            \u0027Trove instance of the datastore version.\u0027)"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027packages\u0027, help\u003d\u0027The packages of the datastore version that are \u0027"},{"line_number":136,"context_line":"            \u0027installed on the guest image.\u0027)"},{"line_number":137,"context_line":"        parser.add_argument("},{"line_number":138,"context_line":"            \u0027active\u0027, help\u003d\u0027Whether the datastore version is active or not.  \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_463fa5d6","line":135,"updated":"2014-07-16 20:27:34.000000000","message":"\"that are installed\" is a bit misleading. How about \"to be installed\" or \"that are required\"?","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"2eca88f2b84caa8e30515924d76691146dca9829","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            \u0027image_id\u0027, help\u003d\u0027The Glance ID of the image used to create a new \u0027"},{"line_number":133,"context_line":"            \u0027Trove instance of the datastore version.\u0027)"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027packages\u0027, help\u003d\u0027The packages of the datastore version that are \u0027"},{"line_number":136,"context_line":"            \u0027installed on the guest image.\u0027)"},{"line_number":137,"context_line":"        parser.add_argument("},{"line_number":138,"context_line":"            \u0027active\u0027, help\u003d\u0027Whether the datastore version is active or not.  \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_52ae7b30","line":135,"in_reply_to":"baada198_463fa5d6","updated":"2014-07-16 22:15:48.000000000","message":"I\u0027m not entirely sure how this variable is used, but I\u0027m pretty sure that it doesn\u0027t control what will be installed on the VM.  I\u0027ll add the \u0027required\u0027 keyword and see if it\u0027s better.","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"51e2611febfec41c6ed71a1f232983ff491e48b0","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            \u0027image_id\u0027, help\u003d\u0027The Glance ID of the image used to create a new \u0027"},{"line_number":133,"context_line":"            \u0027Trove instance of the datastore version.\u0027)"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027packages\u0027, help\u003d\u0027The packages of the datastore version that are \u0027"},{"line_number":136,"context_line":"            \u0027installed on the guest image.\u0027)"},{"line_number":137,"context_line":"        parser.add_argument("},{"line_number":138,"context_line":"            \u0027active\u0027, help\u003d\u0027Whether the datastore version is active or not.  \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_1725d55b","line":135,"in_reply_to":"baada198_463fa5d6","updated":"2014-07-16 21:23:14.000000000","message":"What about \u0027packages to install on VM\u0027?","commit_id":"caf95b5e13f75b6f5c99f7d1f8a41f2eb8964d77"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"14e511ad5e326a7fe7c357f51c72a1659a6527d2","unresolved":false,"context_lines":[{"line_number":132,"context_line":"        parser.add_argument("},{"line_number":133,"context_line":"            \u0027image_id\u0027, help\u003d\u0027ID of the image used to create an instance of \u0027"},{"line_number":134,"context_line":"            \u0027the datastore version.\u0027)"},{"line_number":135,"context_line":"        parser.add_argument("},{"line_number":136,"context_line":"            \u0027packages\u0027, help\u003d\u0027Packages required by the datastore version that \u0027"},{"line_number":137,"context_line":"            \u0027are installed on the guest image.\u0027)"},{"line_number":138,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_4fa4114b","line":135,"updated":"2014-07-17 02:06:08.000000000","message":"just curious why some arguments have the -- and others don\u0027t.","commit_id":"6c21b15f3efd034eefbeb6ffcbc927e7724c861b"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"70a382bcfdf7847ca3fa2d44f1d20d637c8be448","unresolved":false,"context_lines":[{"line_number":132,"context_line":"        parser.add_argument("},{"line_number":133,"context_line":"            \u0027image_id\u0027, help\u003d\u0027ID of the image used to create an instance of \u0027"},{"line_number":134,"context_line":"            \u0027the datastore version.\u0027)"},{"line_number":135,"context_line":"        parser.add_argument("},{"line_number":136,"context_line":"            \u0027packages\u0027, help\u003d\u0027Packages required by the datastore version that \u0027"},{"line_number":137,"context_line":"            \u0027are installed on the guest image.\u0027)"},{"line_number":138,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_d108bb2a","line":135,"in_reply_to":"baada198_4fa4114b","updated":"2014-07-17 07:49:49.000000000","message":"Agreed, all argumets for subparsers should have -- before argument name.","commit_id":"6c21b15f3efd034eefbeb6ffcbc927e7724c861b"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"20811aaa35491f846637b4801ad4f39dbd864427","unresolved":false,"context_lines":[{"line_number":132,"context_line":"        parser.add_argument("},{"line_number":133,"context_line":"            \u0027image_id\u0027, help\u003d\u0027ID of the image used to create an instance of \u0027"},{"line_number":134,"context_line":"            \u0027the datastore version.\u0027)"},{"line_number":135,"context_line":"        parser.add_argument("},{"line_number":136,"context_line":"            \u0027packages\u0027, help\u003d\u0027Packages required by the datastore version that \u0027"},{"line_number":137,"context_line":"            \u0027are installed on the guest image.\u0027)"},{"line_number":138,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_7e988c6a","line":135,"in_reply_to":"baada198_d108bb2a","updated":"2014-07-17 14:48:37.000000000","message":"The \u0027--\u0027 means that the argument is optional whereas not having it means it is required.  I purposely didn\u0027t change that on any argument as I knew it would change the behaviour of the utility and thus be out of scope of this bug (plus it would break redstack).  If either of you feel that the selections are incorrect (my vote goes for default_version of datastore_update), feel free to submit a bug and fix it.\n\nAs per the commit message, no DocImpact tag is necessary as there currently is no documentation on this utility.  Once this patchset is merged, an in-flight patchset https://review.openstack.org/#/c/107484/ is set to update the docs.\n\nOf course if someone does submit a bug and change the argument requirements, a DocImpact will then be necessary.\n\nI hope this addressed all your concerns.","commit_id":"6c21b15f3efd034eefbeb6ffcbc927e7724c861b"}]}