)]}'
{"nova/test.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"d47adefd427813192f9b6f628d1a28109b5823c0","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        # COMPUTE_RESOURCE_SEMAPHORE. Note that we cannot import"},{"line_number":108,"context_line":"        # COMPUTE_RESOURCE_SEMAPHORE as that would apply the decorators we want"},{"line_number":109,"context_line":"        # to poison here."},{"line_number":110,"context_line":"        if ((args[0] \u003d\u003d \"compute_resources\" or"},{"line_number":111,"context_line":"                kwargs.get(\"name\") \u003d\u003d \"compute_resources\") and"},{"line_number":112,"context_line":"                not kwargs.get(\"fair\", False)):"},{"line_number":113,"context_line":"            raise AssertionError("}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_61ec76f3","line":110,"updated":"2020-03-12 22:10:26.000000000","message":"Looks like this needs to be resilient to the case where the first arg (name) is passed as a keyword arg and not a positional arg:\n\n if ((args[0] \u003d\u003d \"compute_resources\" or\n    IndexError: tuple index out of range\n\nhttps://zuul.opendev.org/t/openstack/build/24ad3e6fdb3a4990a251af93cf1c0b94/log/job-output.txt#11480","commit_id":"f275a510eb12060b20f954a208023bc3367411c5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"68902ba4ffd9df64e790566926e900986374aab8","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        # COMPUTE_RESOURCE_SEMAPHORE. Note that we cannot import"},{"line_number":108,"context_line":"        # COMPUTE_RESOURCE_SEMAPHORE as that would apply the decorators we want"},{"line_number":109,"context_line":"        # to poison here."},{"line_number":110,"context_line":"        if ((args[0] \u003d\u003d \"compute_resources\" or"},{"line_number":111,"context_line":"                kwargs.get(\"name\") \u003d\u003d \"compute_resources\") and"},{"line_number":112,"context_line":"                not kwargs.get(\"fair\", False)):"},{"line_number":113,"context_line":"            raise AssertionError("}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_be190089","line":110,"in_reply_to":"1fa4df85_61ec76f3","updated":"2020-03-16 19:12:24.000000000","message":"Good point. Done.\nI tried to keep it simple but our unit tests are more complicated than I thought.","commit_id":"f275a510eb12060b20f954a208023bc3367411c5"}]}
