)]}'
{"nova/api/openstack/compute/servers.py":[{"author":{"_account_id":25113,"name":"Fan Zhang","email":"zh.f@outlook.com","username":"fanzhang"},"change_message_id":"6f173f133e76687ea0504b6a7e2440f5b4ebc368","unresolved":false,"context_lines":[{"line_number":190,"context_line":"                msg \u003d _(\"Invalid filter field: %s.\") % search_opt"},{"line_number":191,"context_line":"                raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        # Only admin can use vm_state query param."},{"line_number":194,"context_line":"        if \u0027vm_state\u0027 in req.GET.keys():"},{"line_number":195,"context_line":"            if not context.is_admin:"},{"line_number":196,"context_line":"                msg \u003d _(\"Only administrators may list instances via vm_state\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_0ec1d62f","line":193,"range":{"start_line":193,"start_character":10,"end_line":193,"end_character":50},"updated":"2019-12-26 02:02:21.000000000","message":"This part is a little confusing, \"This parameter is only valid when specified by administrators. If non-admin users specify this parameter, it is ignored\". But currently, there is no restraint as doc [1] says. So I prefer to add this check and maybe I should modify the doc BTW.\n\n\n[1]. https://docs.openstack.org/api-ref/compute/?expanded\u003dlist-servers-detail#list-server-request","commit_id":"81d99f1a703a80cea698a1278c19f84c689664e6"}]}
