)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"b6dcbb56a923c166f106847525be407d6ce3382c","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This change simply increases service_down_time to 6 seconds to avoid"},{"line_number":15,"context_line":"this race, ensuring disabled but alive computes are able to initially"},{"line_number":16,"context_line":"report in and continue to be marked as up by the servicegroup API."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Closes-Bug: #1940741"},{"line_number":19,"context_line":"Change-Id: I61fded6f513c18dbd0248ed54fc58a318dc05416"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1ef91ba8_82f8667a","line":16,"updated":"2021-08-25 01:42:35.000000000","message":"Eh... I dunno about using real service down times to reflect a service as down given how flaky things can be with slow CI nodes. Usually we prefer to force down the service but I see that you are doing that as a separate test and differentiating between forced down and just down. For the latter I would have thought it might be better to \"with mock.patch(is_up()):\" to make it return False to take away the possibility of timing differences.","commit_id":"5e8267f703d040b6e8b56d5c0f3ed669170cbb2d"}]}