)]}'
{"doc/source/contributor/functional_tests.rst":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"4b2ccdd0d465954867e0ed8e978037d5f13a71d9","unresolved":true,"context_lines":[{"line_number":5,"context_line":"Cinderclient contains a suite of functional tests, in the cinderclient/"},{"line_number":6,"context_line":"tests/functional directory."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"These are currently non-voting.however, that these tests"},{"line_number":9,"context_line":"are investigated in the case of a failure."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Running the tests"},{"line_number":12,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"eadbac10_f488d040","line":9,"range":{"start_line":8,"start_character":0,"end_line":9,"end_character":42},"updated":"2021-02-19 15:23:31.000000000","message":"I am ok with the change regarding ostestr, but this change isn\u0027t an improvement.  I would like it put back to the way it was.","commit_id":"8fbe1e974ea1ff4820b36405b5bffd55dbee4d44"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"79c0bdf42816b9f41fe55c05b77b5332d475a265","unresolved":true,"context_lines":[{"line_number":5,"context_line":"Cinderclient contains a suite of functional tests, in the cinderclient/"},{"line_number":6,"context_line":"tests/functional directory."},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"These are currently non-voting, meaning that Jenkins will not reject a"},{"line_number":9,"context_line":"patched based on failure of the functional tests. It is highly recommended,"},{"line_number":10,"context_line":"however, that these tests are investigated in the case of a failure."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ade17496_4ac3fcfd","line":8,"range":{"start_line":8,"start_character":45,"end_line":8,"end_character":52},"updated":"2021-02-22 08:54:30.000000000","message":"If I read correctly the previous comment, the idea there was to not remove the entire sentence as it happened in the previous review, but that doesn\u0027t mean you can\u0027t just replace \"Jenkins\" with \"Zuul\" and leave the rest of the text as it is. In fact, I think you can do that.\nI\u0027d let Jay comment though.","commit_id":"30754a312da7504422d4040e5a67617eec12d2f5"}]}