)]}'
{"id":"openstack%2Fwatcher~938429","triplet_id":"openstack%2Fwatcher~stable%2F2024.1~I672c183274b0a17cb40d7b5ab8c313197760b5a0","project":"openstack/watcher","branch":"stable/2024.1","topic":"migrate-to-noble-stable/2024.1","attention_set":{},"removed_from_attention_set":{"34720":{"account":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"last_update":"2025-01-10 02:23:51.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I672c183274b0a17cb40d7b5ab8c313197760b5a0","subject":"Fix incompatiablity between apscheduler and eventlet","status":"ABANDONED","created":"2025-01-06 05:36:48.000000000","updated":"2025-01-10 02:23:51.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"cherry_pick_of_change":933890,"cherry_pick_of_patch_set":11,"meta_rev_id":"dd33504eefb0f06fe17f62809d3f23b8a0a44303","_number":938429,"virtual_id_number":938429,"owner":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2025-01-06 06:06:06.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"all":[{"value":-1,"date":"2025-01-07 07:50:44.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},{"value":-1,"date":"2025-01-07 11:48:05.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Standard Change","+1":"Important Change","+2":"High Priority Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-01-06 05:36:48.000000000","updated_by":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"CC"},{"updated":"2025-01-06 06:06:06.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2025-01-07 07:50:44.000000000","updated_by":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"reviewer":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"state":"REVIEWER"},{"updated":"2025-01-07 11:48:05.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"}],"messages":[{"id":"a7461f15490293650284446ec527a2bcdb88fae4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"date":"2025-01-06 05:36:48.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"2b13f0efdfaa71b4efdfa740c2871851339bb79a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-01-06 06:06:06.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/c0108ded1bce43b99905428acb203f7c\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/fb3389f92b264131a11d0f91947d5206 : FAILURE in 3m 40s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8f7f4ac05c3a420a826dd9a389f19600 : SUCCESS in 3m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/4ae1766b415b482284827c654504dea4 : FAILURE in 3m 38s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/571dd2329d2140a2893e5f70cbaf126b : FAILURE in 3m 45s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/0239b3b012e94a83875308512920c7f9 : FAILURE in 4m 33s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/f64a06ea101b4608a07778fbf196247c : FAILURE in 16m 33s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/5cd163fa0a4c40588be535806b113826 : FAILURE in 28m 04s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/f721fa5443f64458bb7f6b47850bffde : FAILURE in 17m 40s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/ebf37fa930db414988656d19a83c1260 : FAILURE in 18m 15s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/00621d810d7d4c6685aaa0a8e6515795 : FAILURE in 15m 53s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/40c33f7281d843b9bb1f8491ad4f466d : FAILURE in 17m 29s","accounts_in_message":[],"_revision_number":1},{"id":"3c6d8499b2602250b31a342611f7f0a0485ca0eb","author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"date":"2025-01-07 07:50:44.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"784fe852304fe5845791843187746a7c20961252","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-01-07 11:48:05.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"dd33504eefb0f06fe17f62809d3f23b8a0a44303","tag":"autogenerated:gerrit:abandon","author":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"date":"2025-01-10 02:23:51.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"afd65bc64295fdd2b0e00b557b73d43d2af1f6a3","revisions":{"afd65bc64295fdd2b0e00b557b73d43d2af1f6a3":{"kind":"REWORK","_number":1,"created":"2025-01-06 05:36:48.000000000","uploader":{"_account_id":34720,"name":"JamesLin","display_name":"jneo8","email":"james.lin@canonical.com","username":"jneo8"},"ref":"refs/changes/29/938429/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/29/938429/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/29/938429/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/29/938429/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/29/938429/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/29/938429/1"}}},"commit":{"parents":[{"commit":"6b433b3547466f592c9e5ed22477bd8e534512b9","subject":"Fix oslo.db \u003e\u003d 15.0.0 compatibility","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6b433b3547466f592c9e5ed22477bd8e534512b9"}]}],"author":{"name":"Sean Mooney","email":"work@seanmooney.info","date":"2024-11-19 01:10:00.000000000","tz":0},"committer":{"name":"JamesLin","email":"james.lin@canonical.com","date":"2025-01-06 05:36:48.000000000","tz":0},"subject":"Fix incompatiablity between apscheduler and eventlet","message":"Fix incompatiablity between apscheduler and eventlet\n\nThe apscheduler background scheduler spawns a native thread\nwhich is not monkey patched which interacts with shared module\nlevel objects like the module level LOG instances and sqlachmey\nengine facades.\n\nThis is unsafe and leads to mixing patched and unpatched\ncode in the same thread.\n\nThis manifests in 2 ways:\n1.) https://paste.opendev.org/show/bGPgfURx1cZYOsgmtDyw/\nsqlalchmey calls can fail due to a time.sleep(0) in oslo.db being invoked\nusing the unpatched time modules in an eventlet greenthrad.\n2.) https://paste.opendev.org/show/b5C2Zz4A4BFIGbKLKrQU/\nover time that caused the sqlalchmy connection queuepool to fill up preventing\nbackgound tasks form running like reconsiling audits.\n\nThis change adresses this by overloading the background scheduler _main_loop\nto monkey patch the main loop if the calling thread was monkey patched.\n\nCloses-Bug: #2086710\nChange-Id: I672c183274b0a17cb40d7b5ab8c313197760b5a0\n(cherry picked from commit f07694ba6c75315dac1407c3751645de35ed14f4)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/afd65bc64295fdd2b0e00b557b73d43d2af1f6a3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/afd65bc64295fdd2b0e00b557b73d43d2af1f6a3"}]},"branch":"refs/heads/stable/2024.1"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}