)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"change_message_id":"a6492ecfcd2724d9cdad798a43d317250621b1aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2d9b6b1f_cf389222","updated":"2024-11-28 06:16:18.000000000","message":"LGTM","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ed27f221015b58795c7ddded4992a8e808b4c44a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2fd38017_29f331d8","updated":"2024-11-26 17:25:15.000000000","message":"job\u0027s result looks good [1]\n\nLGTM\n\n[1] https://827fc7bbd35003a9e6fe-a12b18ccd704b72d5d1cd1c70d4b1c51.ssl.cf2.rackcdn.com/934394/2/check/openstack-tox-pep8/e1e5ff3/tox/pep8/1-commands%5B0%5D.log","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f47cf1b139619bb97c957007923e60344d433e6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6e8982de_6e5a5683","updated":"2024-12-02 15:13:24.000000000","message":"recheck https://review.opendev.org/c/openstack/watcher/+/936654 is now merged.\n\nnote because we did nto get the follow up patches merged before the defaults change to use noble we may need to temporally pin the jobs back to jammy and then revert that afterword to be able to proceed with this series.\n\nthe alternitive would be to squash the 3 patches.\n\nlest see who the ci jobs work and revisit that based on the results","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6a3e0d36a45c7fa0d335c093380ba3dd3fe6e289","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4310508f_3a5a8e57","updated":"2024-12-02 16:48:39.000000000","message":"recheck test_execute_vm_workload_consolidation_strategy has a race in metric setup\n\nok so the test setup is calling \n\n_create_one_instance_per_host_with_statistic\n\nwhich boots a vm and calls \n\nmake_instance_statistic\ncalls create_resource here\n\nhttps://github.com/openstack/watcher-tempest-plugin/blob/9add899f95319e0d5dde85edfe420dae18241511/watcher_tempest_plugin/tests/scenario/base.py#L357\n\nwhich end up raising a 409 conflict as it already exists.\n\nthis look like a race condtion with the actul gnochii intergartion and teh test setup.\n\n\nlooking at the specfic excpetion the resouce that is conflicting is the memory\n\nDetails: {\u0027code\u0027: 400, \u0027title\u0027: \u0027Bad Request\u0027, \u0027description\u0027: {\u0027cause\u0027: \u0027Named metric already exists\u0027, \u0027detail\u0027: \u0027memory\u0027}}\n\nhttps://github.com/openstack/watcher-tempest-plugin/blob/master/watcher_tempest_plugin/tests/scenario/test_execute_strategies.py#L112-L114\n\n\nso yes this is an existing race condition in the tests.\n\nnote this race is present for all tests that use this pattern so ill file a bug with this info sepreately.\n\nif this continues to be unstable we may need to make the job non voting until that can be resovled in the plugin properly.","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"773f06d4aba44b1c5400b032b2e761206602fde9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a42e02e4_8578b634","updated":"2024-11-28 19:19:39.000000000","message":"the ci failure is now tracked here https://bugs.launchpad.net/watcher/+bug/2089866\nand seam to be adressed by https://review.opendev.org/c/openstack/watcher/+/936654","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"26d385de060f45ec4a1ca165893bc4f721dce9aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"daf88fba_56671466","updated":"2024-12-02 16:38:01.000000000","message":"the failure in teh stagy plugin seams to be unrelated\n\nDetails: {\u0027code\u0027: 400, \u0027title\u0027: \u0027Bad Request\u0027, \u0027description\u0027: {\u0027cause\u0027: \u0027Named metric already exists\u0027, \u0027detail\u0027: \u0027memory\u0027}}\n\nso that implies there is a test instablity in the tempest plugin.\n\nill quickly see if i can check fi that is a know issue or not before rechecking but this is likely a bug we shoudl fix in the tempepst plugin","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"da51d178798a42c212557041a6e61c656f5a5646","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"474ecec1_7f4815c6","in_reply_to":"4310508f_3a5a8e57","updated":"2024-12-02 18:41:44.000000000","message":"note i filed https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2090854 to track the test instablity.","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1c8a2407dbbde5f7f52f5a89e6c8c1c8bfb48ad2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"743c866f_340f760e","in_reply_to":"daf88fba_56671466","updated":"2024-12-02 16:38:34.000000000","message":"https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_49d/934394/2/check/watcher-tempest-strategies/49dbaa5/testr_results.html\n\nthat the relevent log for context","commit_id":"9abec18c8b0211d0c448b9c21fbebdc28f894275"}]}