)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"a7e75b5f3d921ec71a5f4f0859f3d8ae1525b7b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3af99519_3cccdd93","updated":"2023-01-25 13:24:42.000000000","message":"recheck, random failure","commit_id":"25c107d710a03024243463be7e1d8404f1375232"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"27e770a9fd27430aac47d0e9d677314536859be6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5784b17a_8c609682","updated":"2023-01-25 11:43:39.000000000","message":"recheck, random failure","commit_id":"25c107d710a03024243463be7e1d8404f1375232"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"2e72be74195d9f724501ded299a11b493f1246de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b4af3ad_d37cdef9","updated":"2023-01-25 15:51:04.000000000","message":"recheck, random failure","commit_id":"25c107d710a03024243463be7e1d8404f1375232"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"10a43d2144d9684086941ce29b84d6fcf25af284","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cdcad069_f0cf1e9b","updated":"2023-01-25 20:08:25.000000000","message":"recheck, random failure","commit_id":"25c107d710a03024243463be7e1d8404f1375232"}],"openstack_dashboard/test/integration_tests/basewebobject.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c2db8758357df085987c055745dcf046363f4612","unresolved":true,"context_lines":[{"line_number":157,"context_line":"            try:"},{"line_number":158,"context_line":"                self._wait_till_element_visible(locator)"},{"line_number":159,"context_line":"            except Exceptions.NoSuchElementException:"},{"line_number":160,"context_line":"                pass"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def wait_till_spinner_disappears(self):"},{"line_number":163,"context_line":"        def getter():"}],"source_content_type":"text/x-python","patch_set":2,"id":"593886b1_79b94612","line":160,"updated":"2023-01-26 05:46:13.000000000","message":"question: Why can we ignore NoSuchElementException here?\nThe method waits until an element specified by \"locator\" is visible.\nWhat is the expected behavior if the specified element does not exist?\n\nI know this is a backport and it is not a good place to discuss, but I would like to clarify why we can ignore this exception.","commit_id":"25c107d710a03024243463be7e1d8404f1375232"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"25f783d957123fc142f152b777d007b7e41014b9","unresolved":true,"context_lines":[{"line_number":157,"context_line":"            try:"},{"line_number":158,"context_line":"                self._wait_till_element_visible(locator)"},{"line_number":159,"context_line":"            except Exceptions.NoSuchElementException:"},{"line_number":160,"context_line":"                pass"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def wait_till_spinner_disappears(self):"},{"line_number":163,"context_line":"        def getter():"}],"source_content_type":"text/x-python","patch_set":2,"id":"c9661c87_6d8e03ef","line":160,"in_reply_to":"593886b1_79b94612","updated":"2023-01-26 10:11:48.000000000","message":"This function works only as a stabilizing time element (Tests were unstable before. Because selenium was \"faster\" than website). The function in default state does not affect the test. It is only smarter \"wait\" that does not increase duration of tests by a lot of time but slows down selenium against web \u003d\u003e tests does not fail in situation when selenium is trying to read data that are not  loaded on the web yet.\nEverything else (checking of elements/texts/states) that should or should not be on UI is not primary purpose of this function. \nIt can of course be used for this purpose but it is not primary purpose.","commit_id":"25c107d710a03024243463be7e1d8404f1375232"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2203c3fc7ca88a99ef228ee882baa2a9a3eb6fa6","unresolved":true,"context_lines":[{"line_number":157,"context_line":"            try:"},{"line_number":158,"context_line":"                self._wait_till_element_visible(locator)"},{"line_number":159,"context_line":"            except Exceptions.NoSuchElementException:"},{"line_number":160,"context_line":"                pass"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def wait_till_spinner_disappears(self):"},{"line_number":163,"context_line":"        def getter():"}],"source_content_type":"text/x-python","patch_set":2,"id":"71865a1f_51ef2d68","line":160,"in_reply_to":"c9661c87_6d8e03ef","updated":"2023-02-03 05:36:49.000000000","message":"Thanks for the clarification. It makes sense to me. This backport looks ready to go as-is.\n\nApart from this backport, I think it would be better to add some inline comment to note what you explained here (as L.150 above does) in the master branch. Let\u0027s discuss it separately.","commit_id":"25c107d710a03024243463be7e1d8404f1375232"}]}