)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cb6a6a44bfad56140adf597a3c26ca36abd5f34a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4ce14ecf_ab51171b","updated":"2022-12-14 01:12:42.000000000","message":"All the unit tests are failing.  See, for example, https://zuul.opendev.org/t/openstack/build/b8744cbdf00747b5bf0317c987e63a26\n\nI could not get those errors to happen for me locally until I ran stestr with \u0027--concurrency\u003d4\u0027.  With concurrency\u003d4, I had to split the tests into separate stestr runs (instead of a single run with group_regex for TestDBPersistence and TestMemoryDBPersistence).  Next patch set will run 3 sets of tests: \"regular\", TestDBPersistence, and TestMemoryDBPersistence.","commit_id":"d28b7b37c623eaa12b15ae6718d4d69adcecccd2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"40e3d8e2583f348dfba14fb1a80421c7c5800787","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9dd179a2_5be2116f","updated":"2022-12-14 01:44:21.000000000","message":"Not going to affect the gate, but I have {posargs} in the wrong place in tox.ini.  New PS up in a minute.","commit_id":"0e91f3fdf2de4a90cd64f86430087922ca4d3b97"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c9c7dc65787babc97fa55e6951be4795fca9456e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b78e288d_d323d234","updated":"2022-12-15 15:53:51.000000000","message":"I\u0027m +2 on the part that Rajat wrote!","commit_id":"728b933bf2b0f74953ac7f2be0214d2a13bb7acd"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a58ca1079c074a4b2a2f797bcefa184bfe36ace1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f4e27f55_1764f6d1","updated":"2022-12-15 14:02:32.000000000","message":"I\u0027m the co-author and it looks like a workaround to pass the tests but it\u0027s needed to release cinderlib which is a bigger priority. We can clean this up later when we\u0027ve a better way to handle it. Thanks Brian for working on this, LGTM.","commit_id":"728b933bf2b0f74953ac7f2be0214d2a13bb7acd"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"c49d8d491c5ef7383c57b871743f4896b36a2f53","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7e2b6541_109a0731","updated":"2022-12-14 14:29:44.000000000","message":"There is one typo in a comment.  Don\u0027t know that it is worth holding the patch for.  ","commit_id":"728b933bf2b0f74953ac7f2be0214d2a13bb7acd"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a72a55b346fb60ffe8d71e61685e68894b287090","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a636594c_882e985e","in_reply_to":"7e2b6541_109a0731","updated":"2022-12-14 14:37:42.000000000","message":"Yeah, since it\u0027s a TODO comment, I\u0027m inclined to let it slide as (hopefully) it will be addressed soon.","commit_id":"728b933bf2b0f74953ac7f2be0214d2a13bb7acd"}],"tox.ini":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"c49d8d491c5ef7383c57b871743f4896b36a2f53","unresolved":true,"context_lines":[{"line_number":40,"context_line":"    stestr run --exclude-regex cinderlib.tests.unit.persistence.test_dbms {posargs}"},{"line_number":41,"context_line":"    stestr run --combine {posargs} cinderlib.tests.unit.persistence.test_dbms.TestDBPersistence"},{"line_number":42,"context_line":"    stestr run --combine {posargs} cinderlib.tests.unit.persistence.test_dbms.TestMemoryDBPersistence"},{"line_number":43,"context_line":"    # TODO: figure out how to arrange the above so that you can run a"},{"line_number":44,"context_line":"    # a single test.  For example, invoking"},{"line_number":45,"context_line":"    #   tox -e py310 -- cinderlib.tests.unit.test_cinderlib.TestCinderlib.test__set_priv_helper"},{"line_number":46,"context_line":"    # runs that test and then TestDBPersistence tests and then TestMemoryDBPersistence tests"}],"source_content_type":"text/x-properties","patch_set":7,"id":"0270b04f_83495241","line":43,"range":{"start_line":43,"start_character":68,"end_line":43,"end_character":69},"updated":"2022-12-14 14:29:44.000000000","message":"nit:  repeated \u0027a\u0027","commit_id":"728b933bf2b0f74953ac7f2be0214d2a13bb7acd"}]}