)]}'
{"keystoneclient/shell.py":[{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"e34d9a40f159832a544633ecb0c231673c9831d5","unresolved":false,"context_lines":[{"line_number":285,"context_line":"                    \u0027env[OS_AUTH_URL]\u0027)"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"            if (args.os_username or args.os_password or"},{"line_number":288,"context_line":"               args.os_tenant_name or args.os_tenant_id):"},{"line_number":289,"context_line":"                if not args.os_username:"},{"line_number":290,"context_line":"                    raise exc.CommandError("},{"line_number":291,"context_line":"                        \u0027Expecting a username provided via either \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_54135264","line":288,"updated":"2015-03-20 06:12:06.000000000","message":"need one more space for the indention to align","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"69238a21413e0971a97d1841ae163988eb7c3eae","unresolved":false,"context_lines":[{"line_number":285,"context_line":"                    \u0027env[OS_AUTH_URL]\u0027)"},{"line_number":286,"context_line":""},{"line_number":287,"context_line":"            if (args.os_username or args.os_password or"},{"line_number":288,"context_line":"               args.os_tenant_name or args.os_tenant_id):"},{"line_number":289,"context_line":"                if not args.os_username:"},{"line_number":290,"context_line":"                    raise exc.CommandError("},{"line_number":291,"context_line":"                        \u0027Expecting a username provided via either \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"da9b358b_701eab64","line":288,"in_reply_to":"9a80dd14_54135264","updated":"2015-03-31 13:56:08.000000000","message":"tox -epep8 fails if I do that.","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"c885b958652817b98be2500384f90d55054082c2","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":297,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":298,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"                if not args.os_password:"},{"line_number":301,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_1600a065","line":298,"updated":"2015-01-20 08:02:59.000000000","message":"This isn\u0027t required. You can have a default_project_id set on your user and so you will automatically get a tenant_id (and scoped token + catalog) even if you don\u0027t provide one.\n\nThere was a similar patch from memory that tried to prevent people using the shell with an unscoped token.","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"1e361c0138be2d80dced0d312a7c01faca5195b9","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":297,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":298,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"                if not args.os_password:"},{"line_number":301,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_61c4b560","line":298,"in_reply_to":"3a961159_1600a065","updated":"2015-01-20 13:47:51.000000000","message":"The bug report (https://bugs.launchpad.net/python-keystoneclient/+bug/1411625) says otherwise, and I have been able to reproduce the bug.\n\nIf you do not have this \"default_project_id\" and do not pass a tenant name/id, your keystone command will fail without an error message (as would be expected).\n\nMaybe I should check here whether the given user has a defined default_project_id? How could I do this? It seems to be v3-related, isn\u0027t it?","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"e7d9dc5ffbf90e36e9a37da579106c7c1e336c45","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":297,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":298,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"                if not args.os_password:"},{"line_number":301,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_6648fdcc","line":298,"in_reply_to":"3a961159_61c4b560","updated":"2015-03-18 18:26:08.000000000","message":"I think you\u0027d need to grab the user reference from Keystone and inspect the response to find the default project id? Also, does this need a test?","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"e34d9a40f159832a544633ecb0c231673c9831d5","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":297,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":298,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"                if not args.os_password:"},{"line_number":301,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_a80fd22c","line":298,"in_reply_to":"9a80dd14_6648fdcc","updated":"2015-03-20 06:12:06.000000000","message":"does\u0027t default_project_id only applies to v3? keystoneclient cli is only for v2.","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"36aae83007a57a49a86548d4eb02f521c530ceb8","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":297,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":298,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"                if not args.os_password:"},{"line_number":301,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":1,"id":"da9b358b_9cef38b0","line":298,"in_reply_to":"9a80dd14_a80fd22c","updated":"2015-03-30 14:59:02.000000000","message":"@Lin Hua Cheng: indeed, \"git grep default_project_id\" only returns stuff from v3/\n\n@Lance bragstad: not sure how I am supposed to do that. Indeed, I could probably add a test.","commit_id":"b3e5b19e37f39220e66c8713d0df5f5df8cd8f61"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"c3bf0c0d96347c78d8aed2a6b880a610230b3156","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"                if not (args.os_tenant_name or args.os_tenant_id):"},{"line_number":300,"context_line":"                    raise exc.CommandError("},{"line_number":301,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":303,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":304,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_e668af89","line":301,"updated":"2015-04-08 16:09:25.000000000","message":"should be a space or \\n at the end of this string","commit_id":"8f3b634d5c71ea6824ec871761a2c1caa8887a05"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"112fbaae0a66c9170b49dbd904c3d4c0cfbf1431","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"                if not (args.os_tenant_name or args.os_tenant_id):"},{"line_number":300,"context_line":"                    raise exc.CommandError("},{"line_number":301,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via\u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":303,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":304,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_665d7514","line":301,"in_reply_to":"9aa53dc9_e668af89","updated":"2015-04-10 09:25:27.000000000","message":"NIce catch!","commit_id":"8f3b634d5c71ea6824ec871761a2c1caa8887a05"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"8c7bba16c50c74fdcc5ae8b6770ad429c2137e80","unresolved":false,"context_lines":[{"line_number":290,"context_line":"                    \u0027env[OS_AUTH_URL]\u0027)"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"            if (args.os_username or args.os_password or"},{"line_number":293,"context_line":"               args.os_tenant_name or args.os_tenant_id):"},{"line_number":294,"context_line":"                if not args.os_username:"},{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a username provided via either \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_de7b9a7a","line":293,"updated":"2015-04-10 23:10:17.000000000","message":"I wonder why pep8 didn\u0027t catch this, missing an extra space for the indention","commit_id":"1353b4b57dbd5e4f2271055368f2da6d6326e237"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"7d480d8224a2573ff40c5a3a191a88a01c7c9475","unresolved":false,"context_lines":[{"line_number":290,"context_line":"                    \u0027env[OS_AUTH_URL]\u0027)"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"            if (args.os_username or args.os_password or"},{"line_number":293,"context_line":"               args.os_tenant_name or args.os_tenant_id):"},{"line_number":294,"context_line":"                if not args.os_username:"},{"line_number":295,"context_line":"                    raise exc.CommandError("},{"line_number":296,"context_line":"                        \u0027Expecting a username provided via either \u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_487217e7","line":293,"in_reply_to":"7aaa499b_de7b9a7a","updated":"2015-04-13 09:30:43.000000000","message":"If I add a space, as you suggest, and then run \"tox -epep8\", I get:\n\n./keystoneclient/shell.py:293:17: E129 visually indented line with same indent as next logical line\n                args.os_tenant_name or args.os_tenant_id):","commit_id":"1353b4b57dbd5e4f2271055368f2da6d6326e237"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"8c7bba16c50c74fdcc5ae8b6770ad429c2137e80","unresolved":false,"context_lines":[{"line_number":299,"context_line":"                if not (args.os_tenant_name or args.os_tenant_id):"},{"line_number":300,"context_line":"                    raise exc.CommandError("},{"line_number":301,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via \u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":303,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"                if not args.os_password:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_fe865664","line":302,"updated":"2015-04-10 23:10:17.000000000","message":"--os-tenant-name \"or\" env[OS_TENANT_NAME]","commit_id":"1353b4b57dbd5e4f2271055368f2da6d6326e237"},{"author":{"_account_id":9276,"name":"David Charles Kennedy","email":"david.c.kennedy@hpe.com","username":"darfed"},"change_message_id":"0bda00cb146c56a7e36b9bf9c28485f203494c90","unresolved":false,"context_lines":[{"line_number":299,"context_line":"                if not (args.os_tenant_name or args.os_tenant_id):"},{"line_number":300,"context_line":"                    raise exc.CommandError("},{"line_number":301,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via \u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":303,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"                if not args.os_password:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aa745d1_6a8dc230","line":302,"in_reply_to":"7aaa499b_8b049915","updated":"2015-04-16 16:09:09.000000000","message":"I think this is always going to sound awkward but my best bet would be:\n\nExpecting a tenant name provided via --os-tenant-name or  env[OS_TENANT_NAME] or a tenant id provided via --os-tenant-id or env[OS_TENANT_ID]\n\nThis uses the tenant name/tenant id to break the ors up a bit.","commit_id":"1353b4b57dbd5e4f2271055368f2da6d6326e237"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"7d480d8224a2573ff40c5a3a191a88a01c7c9475","unresolved":false,"context_lines":[{"line_number":299,"context_line":"                if not (args.os_tenant_name or args.os_tenant_id):"},{"line_number":300,"context_line":"                    raise exc.CommandError("},{"line_number":301,"context_line":"                        \u0027Expecting a tenant name or a tenant id provided via \u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME], \u0027"},{"line_number":303,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"                if not args.os_password:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7aaa499b_8b049915","line":302,"in_reply_to":"7aaa499b_fe865664","updated":"2015-04-13 09:30:43.000000000","message":"This (\"a or b, c or d\") sounds weird to me, but I\u0027m not a native speaker. Are you? What do other people think ?","commit_id":"1353b4b57dbd5e4f2271055368f2da6d6326e237"},{"author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"change_message_id":"61a092ee22bf535f683a6bcf7abdee47897c04cd","unresolved":false,"context_lines":[{"line_number":301,"context_line":"                        \u0027Expecting a tenant name provided via \u0027"},{"line_number":302,"context_line":"                        \u0027--os-tenant-name, env[OS_TENANT_NAME] \u0027"},{"line_number":303,"context_line":"                        \u0027or a tenant id provided via \u0027"},{"line_number":304,"context_line":"                        \u0027--os-tenant-id or env[OS_TENANT_ID]\u0027)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"                if not args.os_password:"},{"line_number":307,"context_line":"                    # No password, If we\u0027ve got a tty, try prompting for it"}],"source_content_type":"text/x-python","patch_set":4,"id":"5ad0253b_f4a551b5","line":304,"updated":"2015-04-28 19:32:01.000000000","message":"this isn\u0027t always the case - you can do some commands without specifying a tenant.","commit_id":"411b5ede91bc82745ba3283a0897271094c41f60"}]}