)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"483d28da97621f31b24388315dd1986fb2657ff4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fda3895a_dfd228fc","updated":"2023-04-18 11:49:29.000000000","message":"recheck\nThe test failed on the job tempest-multinode-full-py3 - let\u0027s see if the failure is consistent.\nhttps://5ac2e0019b2d88b7bbf6-2bdaf7ecf5927d28353faaebb26c863b.ssl.cf1.rackcdn.com/880719/2/check/tempest-multinode-full-py3/26dbac8/testr_results.html\nAssertionError: 8.8 not less than or equal to 5.0 : Downtime of 8.8 seconds is higher than expected \u00275.0\u0027","commit_id":"50075de75dced4f7c61fd537c3cb5c01b34a0fba"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"e72bd2228a44c9748c6b45d3049de71282525085","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"da32cbbe_e1babefa","updated":"2023-04-18 14:22:45.000000000","message":"recheck\nfirst time, it failed\nsecond time, it passed\nrunning a third attempt","commit_id":"50075de75dced4f7c61fd537c3cb5c01b34a0fba"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"1c6f443d4a326e952c67b291c9d5673456f1dcff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9cd63789_31bf1a71","updated":"2023-04-25 13:31:57.000000000","message":"In some setups (specifically in BGP downstream jobs), this test should have failed due to rhbz#2187651 but it didn\u0027t.\nI can\u0027t tell it the results of this test will become less stable due to this change in upstream CI.\nWith this patch, all jobs were executed three times and only one job failed once.","commit_id":"f7b002131ce7d2c12c097c1363764bf2ecffbfff"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9b1f07196faeb03f289be2216327a6013cec103b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a8d97871_af6976dd","updated":"2023-04-28 18:16:35.000000000","message":"agree with the idea of waiting before downtime is measured but let\u0027s make wait to be success ping based instead of time.","commit_id":"f7b002131ce7d2c12c097c1363764bf2ecffbfff"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"c327376477a8cdfc51d16504eb51b59172923928","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9ededa23_0c49d605","updated":"2023-06-16 09:21:24.000000000","message":"I have opened a bug for some of the tests failing (not related to my patch):\nhttps://bugs.launchpad.net/neutron/+bug/2024160\n\nI am afraid any patches are blocked until this bug is fixed.","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3036b8912f108223d5a3c8fc41a8b8669db5ce4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"07facbbe_25e79b24","updated":"2023-06-16 03:24:24.000000000","message":"lgtm","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f4d9b73503cbca17f2e71b01dc419cb37025ad19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"004c5d6c_f3a68530","updated":"2023-06-15 20:37:00.000000000","message":"looks good to me, thanks","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"530f01b035aa0e46adad39d3616653ccbfd1bc62","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"99b2bdf9_78a3e6f4","updated":"2023-06-15 23:48:39.000000000","message":"recheck - the fails are unrelated to this change, the change looks good to me","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b59527bb780dd382874075e3cde3bf847ba3a2de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7674601d_1fa1ceb3","updated":"2023-06-15 20:37:12.000000000","message":"recheck to get the tests logs","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"88c3e6ca2ade0fe9bc39a7e647005c5c51777599","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"67c6b957_17889fcb","updated":"2023-06-19 22:39:06.000000000","message":"recheck unrelated port status failure","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"88c3e6ca2ade0fe9bc39a7e647005c5c51777599","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"bb277d7e_7911bef4","in_reply_to":"9ededa23_0c49d605","updated":"2023-06-19 22:39:06.000000000","message":"I think there are some timing issue on port status but that is not 100% failing so this change should be good to go.\n\nThanks for checking and filling the bug it really help to make gate stable.","commit_id":"ddc561e100dcf249086ceb6691fd8e21ce3938c2"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"1708256c00840532e3635a6c98021174c51c5c49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1bc90cfc_45c231fb","updated":"2023-06-23 05:55:16.000000000","message":"recheck","commit_id":"a9c221efc1d11da522c6cddf331d01d63dc18e7c"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"782d2d18c458cb48d03299016525bf047e7f60c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"461cde0e_4dbb6128","updated":"2023-06-26 10:35:08.000000000","message":"recheck","commit_id":"a9c221efc1d11da522c6cddf331d01d63dc18e7c"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"82815a15a15b4e56770683fbf4fa806e74bf608e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"846e6666_5a2e4f6f","updated":"2023-06-26 15:46:14.000000000","message":"recheck\n\nThis didn\u0027t fail before and looks concerning:\nhttps://f2f41ad0c14cfbdbf8b2-95baeb13f84d1cb0954af27128e2e856.ssl.cf5.rackcdn.com/880719/6/check/tempest-full-enforce-scope-new-defaults/df049a2/testr_results.html\n\nAll setups failed with:\n```\ntempest.lib.exceptions.IdentityError: Got identity error\nDetails: Unexpected status code 500\n\n```\n\nLet\u0027s see if it happens again or not.","commit_id":"a9c221efc1d11da522c6cddf331d01d63dc18e7c"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"2fcabdf20d3145c58ef6a7c511f5a88f0123a52e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1d375fa4_1bb0d841","updated":"2023-06-27 07:07:17.000000000","message":"recheck\ntimeouts","commit_id":"a9c221efc1d11da522c6cddf331d01d63dc18e7c"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"a35d46e11c07d6673bda4de4ecc1f232fa27d625","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"da1c98b5_93b920c1","updated":"2023-07-01 11:18:42.000000000","message":"recheck","commit_id":"03ec2ae242de472e7763bffb84a15e531e7d448a"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"37e062b26850d2690e2dda1d9c96730c3306ac28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"eccc8c3d_a2b1d878","updated":"2023-07-01 15:49:41.000000000","message":"recheck","commit_id":"03ec2ae242de472e7763bffb84a15e531e7d448a"}],"tempest/scenario/test_network_advanced_server_ops.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9b1f07196faeb03f289be2216327a6013cec103b","unresolved":true,"context_lines":[{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        new_host \u003d self.get_host_for_server(server[\u0027id\u0027])"},{"line_number":272,"context_line":"        self.assertNotEqual(old_host, new_host, \u0027Server did not migrate\u0027)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        allowed_downtime \u003d CONF.validation.allowed_network_downtime"},{"line_number":275,"context_line":"        LOG.debug(\"Sleeping for %f seconds in order to verify properly that \""},{"line_number":276,"context_line":"                  \"downtime is lower than that time\", allowed_downtime)"},{"line_number":277,"context_line":"        time.sleep(allowed_downtime)"},{"line_number":278,"context_line":"        downtime \u003d downtime_meter.get_downtime()"},{"line_number":279,"context_line":"        self.assertIsNotNone(downtime)"},{"line_number":280,"context_line":"        LOG.debug(\"Downtime seconds measured with downtime_meter \u003d %r\","}],"source_content_type":"text/x-python","patch_set":4,"id":"a85b5602_80ed76a7","line":277,"range":{"start_line":273,"start_character":0,"end_line":277,"end_character":36},"updated":"2023-04-28 18:16:35.000000000","message":"instead of wait here which can still give false result if server is still not up why do not we ping server in get_downtime() and if that is success then measure the downtime ?\n\nhttps://github.com/openstack/tempest/blob/master/tempest/common/utils/net_downtime.py#L53","commit_id":"f7b002131ce7d2c12c097c1363764bf2ecffbfff"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"c141a4db1fc6bf858e995bf5595df4c0bf76ce2d","unresolved":false,"context_lines":[{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        new_host \u003d self.get_host_for_server(server[\u0027id\u0027])"},{"line_number":272,"context_line":"        self.assertNotEqual(old_host, new_host, \u0027Server did not migrate\u0027)"},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        allowed_downtime \u003d CONF.validation.allowed_network_downtime"},{"line_number":275,"context_line":"        LOG.debug(\"Sleeping for %f seconds in order to verify properly that \""},{"line_number":276,"context_line":"                  \"downtime is lower than that time\", allowed_downtime)"},{"line_number":277,"context_line":"        time.sleep(allowed_downtime)"},{"line_number":278,"context_line":"        downtime \u003d downtime_meter.get_downtime()"},{"line_number":279,"context_line":"        self.assertIsNotNone(downtime)"},{"line_number":280,"context_line":"        LOG.debug(\"Downtime seconds measured with downtime_meter \u003d %r\","}],"source_content_type":"text/x-python","patch_set":4,"id":"f7ee652c_ec97da39","line":277,"range":{"start_line":273,"start_character":0,"end_line":277,"end_character":36},"in_reply_to":"a85b5602_80ed76a7","updated":"2023-05-16 11:08:05.000000000","message":"Done","commit_id":"f7b002131ce7d2c12c097c1363764bf2ecffbfff"}]}