)]}'
{"manilaclient/shell.py":[{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        parser.add_argument(\u0027--os-user-id\u0027,"},{"line_number":143,"context_line":"                            metavar\u003d\u0027\u003cauth-user-id\u003e\u0027,"},{"line_number":144,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_ID\u0027),"},{"line_number":145,"context_line":"                            help\u003d(\u0027Authentication user ID (Env: OS_USER_ID)\u0027))"},{"line_number":146,"context_line":"        parser.add_argument(\u0027--os_user_id\u0027,"},{"line_number":147,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_5dfd6f6c","line":145,"updated":"2015-10-09 07:57:26.000000000","message":"Inconsistent help msg. imo better: \"Authentication user ID. Defaults to env [OS_USER_ID].\"","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"86574e6156fd3768da51f3afdbe2238d6ffc76b0","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        parser.add_argument(\u0027--os-user-id\u0027,"},{"line_number":143,"context_line":"                            metavar\u003d\u0027\u003cauth-user-id\u003e\u0027,"},{"line_number":144,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_ID\u0027),"},{"line_number":145,"context_line":"                            help\u003d(\u0027Authentication user ID (Env: OS_USER_ID)\u0027))"},{"line_number":146,"context_line":"        parser.add_argument(\u0027--os_user_id\u0027,"},{"line_number":147,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_5f98f91b","line":145,"in_reply_to":"7a2fa921_5dfd6f6c","updated":"2015-10-09 12:01:47.000000000","message":"Made consistent with other - \"Defaults to env [OS_USER_ID].\".","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":143,"context_line":"                            metavar\u003d\u0027\u003cauth-user-id\u003e\u0027,"},{"line_number":144,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_ID\u0027),"},{"line_number":145,"context_line":"                            help\u003d(\u0027Authentication user ID (Env: OS_USER_ID)\u0027))"},{"line_number":146,"context_line":"        parser.add_argument(\u0027--os_user_id\u0027,"},{"line_number":147,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        parser.add_argument(\u0027--os-username\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_6898a76a","line":146,"updated":"2015-10-09 07:57:26.000000000","message":"I wouldn\u0027t add that alias. openstackclient doesn\u0027t have it and I would try to follow that.","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"86574e6156fd3768da51f3afdbe2238d6ffc76b0","unresolved":false,"context_lines":[{"line_number":143,"context_line":"                            metavar\u003d\u0027\u003cauth-user-id\u003e\u0027,"},{"line_number":144,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_ID\u0027),"},{"line_number":145,"context_line":"                            help\u003d(\u0027Authentication user ID (Env: OS_USER_ID)\u0027))"},{"line_number":146,"context_line":"        parser.add_argument(\u0027--os_user_id\u0027,"},{"line_number":147,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        parser.add_argument(\u0027--os-username\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_ffd9c5f3","line":146,"in_reply_to":"7a2fa921_6898a76a","updated":"2015-10-09 12:01:47.000000000","message":"We have aliases for existing arguments. It is made consistent with them. It is not venv var, it is manila client arg. So, it does not matter what expects openstackclient while env vars are compatible. Same below.","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":177,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":178,"context_line":"                                  \u0027 --os-tenant-name. \u0027"},{"line_number":179,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_NAME].\u0027))"},{"line_number":180,"context_line":"        parser.add_argument(\u0027--os_project_name\u0027,"},{"line_number":181,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"        parser.add_argument(\u0027--os-tenant-id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_c88e9b2d","line":180,"updated":"2015-10-09 07:57:26.000000000","message":"I wouldn\u0027t add that alias (same reason)","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":195,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":196,"context_line":"                                  \u0027 --os-tenant-id. \u0027"},{"line_number":197,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_ID].\u0027))"},{"line_number":198,"context_line":"        parser.add_argument(\u0027--os_project_id\u0027,"},{"line_number":199,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        parser.add_argument(\u0027--os-user-domain-id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_48d04bf6","line":198,"updated":"2015-10-09 07:57:26.000000000","message":"I wouldn\u0027t add that alias (same reason)","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":204,"context_line":"                            help\u003d(\u0027OpenStack user domain ID. \u0027"},{"line_number":205,"context_line":"                                  \u0027Defaults to env[OS_USER_DOMAIN_ID].\u0027))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        parser.add_argument(\u0027--os_user_domain_id\u0027,"},{"line_number":208,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        parser.add_argument(\u0027--os-user-domain-name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_9d630742","line":207,"updated":"2015-10-09 07:57:26.000000000","message":"I wouldn\u0027t add that alias (same reason)","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"1acbe295af1d2f13a3045e4f623c6021fe87735b","unresolved":false,"context_lines":[{"line_number":212,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_DOMAIN_NAME\u0027),"},{"line_number":213,"context_line":"                            help\u003d(\u0027OpenStack user domain name. \u0027"},{"line_number":214,"context_line":"                                  \u0027Defaults to env[OS_USER_DOMAIN_NAME].\u0027))"},{"line_number":215,"context_line":"        parser.add_argument(\u0027--os_user_domain_name\u0027,"},{"line_number":216,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        parser.add_argument(\u0027--os-project-domain-id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a2fa921_1d585787","line":215,"updated":"2015-10-09 07:57:26.000000000","message":"I wouldn\u0027t add that alias (same reason)","commit_id":"c709029379c1e141c0ee2665f8caa45a3ff9aa25"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"3d3a288aa795b4e8dfd6c8e409c92c38f267b78f","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_DOMAIN_ID\u0027),"},{"line_number":204,"context_line":"                            help\u003d(\u0027OpenStack user domain ID. \u0027"},{"line_number":205,"context_line":"                                  \u0027Defaults to env[OS_USER_DOMAIN_ID].\u0027))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        parser.add_argument(\u0027--os_user_domain_id\u0027,"},{"line_number":208,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_37262017","line":206,"updated":"2015-10-13 04:17:23.000000000","message":"please remove empty line here to have argument and alias in one block","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"20ee97bd05adba43e34716b02841475ad3db4242","unresolved":false,"context_lines":[{"line_number":203,"context_line":"                            default\u003dcliutils.env(\u0027OS_USER_DOMAIN_ID\u0027),"},{"line_number":204,"context_line":"                            help\u003d(\u0027OpenStack user domain ID. \u0027"},{"line_number":205,"context_line":"                                  \u0027Defaults to env[OS_USER_DOMAIN_ID].\u0027))"},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"        parser.add_argument(\u0027--os_user_domain_id\u0027,"},{"line_number":208,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":209,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_a84c0a77","line":206,"in_reply_to":"7a2fa921_37262017","updated":"2015-10-13 07:38:45.000000000","message":"Done","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"3d3a288aa795b4e8dfd6c8e409c92c38f267b78f","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        parser.add_argument(\u0027--os_user_domain_name\u0027,"},{"line_number":216,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        parser.add_argument(\u0027--os-project-domain-id\u0027,"},{"line_number":219,"context_line":"                            metavar\u003d\u0027\u003cauth-project-domain-id\u003e\u0027,"},{"line_number":220,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_ID\u0027),"},{"line_number":221,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_ID].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_b711d0b1","line":218,"updated":"2015-10-13 04:17:23.000000000","message":"As discussed in patchset3, alias should be added. So lease add one for --os-project-domain-id","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"20ee97bd05adba43e34716b02841475ad3db4242","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        parser.add_argument(\u0027--os_user_domain_name\u0027,"},{"line_number":216,"context_line":"                            help\u003dargparse.SUPPRESS)"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"        parser.add_argument(\u0027--os-project-domain-id\u0027,"},{"line_number":219,"context_line":"                            metavar\u003d\u0027\u003cauth-project-domain-id\u003e\u0027,"},{"line_number":220,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_ID\u0027),"},{"line_number":221,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_ID].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_88e10e01","line":218,"in_reply_to":"7a2fa921_b711d0b1","updated":"2015-10-13 07:38:45.000000000","message":"Done","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"3d3a288aa795b4e8dfd6c8e409c92c38f267b78f","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_ID\u0027),"},{"line_number":221,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_ID].\u0027)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        parser.add_argument(\u0027--os-project-domain-name\u0027,"},{"line_number":224,"context_line":"                            metavar\u003d\u0027\u003cauth-project-domain-name\u003e\u0027,"},{"line_number":225,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_NAME\u0027),"},{"line_number":226,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_NAME].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_f71f48b5","line":223,"updated":"2015-10-13 04:17:23.000000000","message":"same here. alias missing","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"20ee97bd05adba43e34716b02841475ad3db4242","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_ID\u0027),"},{"line_number":221,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_ID].\u0027)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"        parser.add_argument(\u0027--os-project-domain-name\u0027,"},{"line_number":224,"context_line":"                            metavar\u003d\u0027\u003cauth-project-domain-name\u003e\u0027,"},{"line_number":225,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_DOMAIN_NAME\u0027),"},{"line_number":226,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_DOMAIN_NAME].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_88afce93","line":223,"in_reply_to":"7a2fa921_f71f48b5","updated":"2015-10-13 07:38:45.000000000","message":"Done","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":7102,"name":"Thomas Bechtold","email":"thomas.bechtold@chainguard.dev","username":"toabctl"},"change_message_id":"3d3a288aa795b4e8dfd6c8e409c92c38f267b78f","unresolved":false,"context_lines":[{"line_number":300,"context_line":"                            default\u003d0,"},{"line_number":301,"context_line":"                            help\u003d\u0027Number of retries.\u0027)"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        parser.add_argument(\u0027--os-cert\u0027,"},{"line_number":304,"context_line":"                            metavar\u003d\u0027\u003ccertificate\u003e\u0027,"},{"line_number":305,"context_line":"                            default\u003dcliutils.env(\u0027OS_CERT\u0027),"},{"line_number":306,"context_line":"                            help\u003d\u0027Defaults to env[OS_CERT].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_b778f0f9","line":303,"updated":"2015-10-13 04:17:23.000000000","message":"alias missing","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"20ee97bd05adba43e34716b02841475ad3db4242","unresolved":false,"context_lines":[{"line_number":300,"context_line":"                            default\u003d0,"},{"line_number":301,"context_line":"                            help\u003d\u0027Number of retries.\u0027)"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        parser.add_argument(\u0027--os-cert\u0027,"},{"line_number":304,"context_line":"                            metavar\u003d\u0027\u003ccertificate\u003e\u0027,"},{"line_number":305,"context_line":"                            default\u003dcliutils.env(\u0027OS_CERT\u0027),"},{"line_number":306,"context_line":"                            help\u003d\u0027Defaults to env[OS_CERT].\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"7a2fa921_48921659","line":303,"in_reply_to":"7a2fa921_b778f0f9","updated":"2015-10-13 07:38:45.000000000","message":"Done","commit_id":"3ef9d721a33c60953436516115df4a1a4e894d65"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"fa77d0816a3f244302c277cc1e9148829bcfe0af","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                            metavar\u003d\u0027\u003cauth-project-name\u003e\u0027,"},{"line_number":175,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_NAME\u0027),"},{"line_number":176,"context_line":"                            help\u003d(\u0027Another way to specify tenant name. \u0027"},{"line_number":177,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":178,"context_line":"                                  \u0027 --os-tenant-name. \u0027"},{"line_number":179,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_NAME].\u0027))"},{"line_number":180,"context_line":"        parser.add_argument(\u0027--os_project_name\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_c554ef55","line":177,"updated":"2015-10-13 17:55:53.000000000","message":"An extra space between with and  --os-...","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"fdcf3d0681598fe12f8913c447e8b1bdc5a546dd","unresolved":false,"context_lines":[{"line_number":174,"context_line":"                            metavar\u003d\u0027\u003cauth-project-name\u003e\u0027,"},{"line_number":175,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_NAME\u0027),"},{"line_number":176,"context_line":"                            help\u003d(\u0027Another way to specify tenant name. \u0027"},{"line_number":177,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":178,"context_line":"                                  \u0027 --os-tenant-name. \u0027"},{"line_number":179,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_NAME].\u0027))"},{"line_number":180,"context_line":"        parser.add_argument(\u0027--os_project_name\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_dbc13082","line":177,"in_reply_to":"5a2ca52d_c554ef55","updated":"2015-10-13 18:17:02.000000000","message":"Done","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"fa77d0816a3f244302c277cc1e9148829bcfe0af","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                            metavar\u003d\u0027\u003cauth-project-id\u003e\u0027,"},{"line_number":193,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_ID\u0027),"},{"line_number":194,"context_line":"                            help\u003d(\u0027Another way to specify tenant ID. \u0027"},{"line_number":195,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":196,"context_line":"                                  \u0027 --os-tenant-id. \u0027"},{"line_number":197,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_ID].\u0027))"},{"line_number":198,"context_line":"        parser.add_argument(\u0027--os_project_id\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_65ccfb35","line":195,"updated":"2015-10-13 17:55:53.000000000","message":"same here","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"fdcf3d0681598fe12f8913c447e8b1bdc5a546dd","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                            metavar\u003d\u0027\u003cauth-project-id\u003e\u0027,"},{"line_number":193,"context_line":"                            default\u003dcliutils.env(\u0027OS_PROJECT_ID\u0027),"},{"line_number":194,"context_line":"                            help\u003d(\u0027Another way to specify tenant ID. \u0027"},{"line_number":195,"context_line":"                                  \u0027This option is mutually exclusive with \u0027"},{"line_number":196,"context_line":"                                  \u0027 --os-tenant-id. \u0027"},{"line_number":197,"context_line":"                                  \u0027Defaults to env[OS_PROJECT_ID].\u0027))"},{"line_number":198,"context_line":"        parser.add_argument(\u0027--os_project_id\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_3b3704b2","line":195,"in_reply_to":"5a2ca52d_65ccfb35","updated":"2015-10-13 18:17:02.000000000","message":"Done","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"fa77d0816a3f244302c277cc1e9148829bcfe0af","unresolved":false,"context_lines":[{"line_number":481,"context_line":"                \"project_domain_name or project_domain_id) via \""},{"line_number":482,"context_line":"                \"  --os-tenant-name (env[OS_TENANT_NAME]),\""},{"line_number":483,"context_line":"                \"  --os-tenant-id (env[OS_TENANT_ID]),\""},{"line_number":484,"context_line":"                \"  --os-project-id (env[OS_PROJECT_ID])\""},{"line_number":485,"context_line":"                \"  --os-project-name (env[OS_PROJECT_NAME]),\""},{"line_number":486,"context_line":"                \"  --os-project-domain-id (env[OS_PROJECT_DOMAIN_ID])\""},{"line_number":487,"context_line":"                \"  --os-project-domain-name (env[OS_PROJECT_DOMAIN_NAME])\""}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_a57e93ac","line":484,"updated":"2015-10-13 17:55:53.000000000","message":"I\u0027m not sure if you intentionally missed a comma here to suggest some grouping or if it was a mistake.","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"fdcf3d0681598fe12f8913c447e8b1bdc5a546dd","unresolved":false,"context_lines":[{"line_number":481,"context_line":"                \"project_domain_name or project_domain_id) via \""},{"line_number":482,"context_line":"                \"  --os-tenant-name (env[OS_TENANT_NAME]),\""},{"line_number":483,"context_line":"                \"  --os-tenant-id (env[OS_TENANT_ID]),\""},{"line_number":484,"context_line":"                \"  --os-project-id (env[OS_PROJECT_ID])\""},{"line_number":485,"context_line":"                \"  --os-project-name (env[OS_PROJECT_NAME]),\""},{"line_number":486,"context_line":"                \"  --os-project-domain-id (env[OS_PROJECT_DOMAIN_ID])\""},{"line_number":487,"context_line":"                \"  --os-project-domain-name (env[OS_PROJECT_DOMAIN_NAME])\""}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_9b52d8dc","line":484,"in_reply_to":"5a2ca52d_a57e93ac","updated":"2015-10-13 18:17:02.000000000","message":"Done","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":11047,"name":"Mark Sturdevant","email":"mark.sturdevant@ibm.com","username":"markstur"},"change_message_id":"fa77d0816a3f244302c277cc1e9148829bcfe0af","unresolved":false,"context_lines":[{"line_number":483,"context_line":"                \"  --os-tenant-id (env[OS_TENANT_ID]),\""},{"line_number":484,"context_line":"                \"  --os-project-id (env[OS_PROJECT_ID])\""},{"line_number":485,"context_line":"                \"  --os-project-name (env[OS_PROJECT_NAME]),\""},{"line_number":486,"context_line":"                \"  --os-project-domain-id (env[OS_PROJECT_DOMAIN_ID])\""},{"line_number":487,"context_line":"                \"  --os-project-domain-name (env[OS_PROJECT_DOMAIN_NAME])\""},{"line_number":488,"context_line":"            )"},{"line_number":489,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_a0b6813b","line":486,"updated":"2015-10-13 17:55:53.000000000","message":"and here.  I think perhaps you should not use commas and do one list for \"must provide a tenant... or project... via: \" and then a second list for \"... and a ... domain... via: \".  Does that make sense?  If not just list the options (probably no commas?) and rely on help to explain usage better.","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"fdcf3d0681598fe12f8913c447e8b1bdc5a546dd","unresolved":false,"context_lines":[{"line_number":483,"context_line":"                \"  --os-tenant-id (env[OS_TENANT_ID]),\""},{"line_number":484,"context_line":"                \"  --os-project-id (env[OS_PROJECT_ID])\""},{"line_number":485,"context_line":"                \"  --os-project-name (env[OS_PROJECT_NAME]),\""},{"line_number":486,"context_line":"                \"  --os-project-domain-id (env[OS_PROJECT_DOMAIN_ID])\""},{"line_number":487,"context_line":"                \"  --os-project-domain-name (env[OS_PROJECT_DOMAIN_NAME])\""},{"line_number":488,"context_line":"            )"},{"line_number":489,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"5a2ca52d_7b033cb9","line":486,"in_reply_to":"5a2ca52d_a0b6813b","updated":"2015-10-13 18:17:02.000000000","message":"It is one-liner. So, commas should exist. Added missed commas.","commit_id":"f3ca88de0ef56c36e1321d521b1615db59c8d4e5"}]}