)]}'
{"novaclient/v2/shell.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"1696b4f8826800178297f93ed18576a034ed1cc4","unresolved":false,"context_lines":[{"line_number":4386,"context_line":"    \"\"\"List the default quotas for a tenant.\"\"\""},{"line_number":4387,"context_line":""},{"line_number":4388,"context_line":"    if args.tenant:"},{"line_number":4389,"context_line":"        project_id \u003d args.tenant"},{"line_number":4390,"context_line":"    elif isinstance(cs.client, client.SessionClient):"},{"line_number":4391,"context_line":"        auth \u003d cs.client.auth"},{"line_number":4392,"context_line":"        project_id \u003d auth.get_auth_ref(cs.client.session).project_id"}],"source_content_type":"text/x-python","patch_set":2,"id":"7af24918_429aabc6","line":4389,"range":{"start_line":4389,"start_character":8,"end_line":4389,"end_character":32},"updated":"2016-03-06 18:54:41.000000000","message":"What about this one?","commit_id":"43800f822c87836820e7e17f6a8066ef988e4398"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"328f8f5b21c578bb84aaa34e6ea9f207e6f3151d","unresolved":false,"context_lines":[{"line_number":4523,"context_line":"           \u0027reserved exceeds the new quota.\u0027))"},{"line_number":4524,"context_line":"def do_quota_update(cs, args):"},{"line_number":4525,"context_line":"    \"\"\"Update the quotas for a tenant/user.\"\"\""},{"line_number":4526,"context_line":"    if args.tenant.isalpha():"},{"line_number":4527,"context_line":"        raise exceptions.CommandError(_(\"tenant-id \u0027%s\u0027 not found.\")"},{"line_number":4528,"context_line":"                                      % args.tenant)"},{"line_number":4529,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"5aef4532_05a836da","line":4526,"updated":"2016-03-09 17:18:05.000000000","message":"Hmmm…I see two potential problems here.  The first is that tenant names could potentially contain numbers, so that would fail this test.  The second is that I could see a tenant ID consisting exclusively of alphabetical characters; it should happen rarely, but I could see it happening.\n\nI think the only way to really resolve this sort of thing may be to explicitly query the authentication service.  This code might be a reasonable means of dealing with the issue in the interim, but I really want to see some way to force novaclient to go ahead and make the request even if this test matches, so we don\u0027t leave an admin high and dry if they run into a situation such as a tenant ID being all alphabetical characters.","commit_id":"964597967f7a4000133649827b1fb3f7b6a20e5f"}]}