)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c20b9c15f17c4f1c60c84d8d1e724c05a581cc86","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    check:"},{"line_number":10,"context_line":"      jobs:"},{"line_number":11,"context_line":"        - watcher-tempest-functional"},{"line_number":12,"context_line":"        - watcher-tempest-functional-jammy"},{"line_number":13,"context_line":"        - watcher-grenade"},{"line_number":14,"context_line":"        - watcher-tempest-strategies"},{"line_number":15,"context_line":"        - watcher-tempest-actuator"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"656a69d2_817b528e","line":12,"updated":"2024-10-22 11:56:51.000000000","message":"This is using Python 3.10 which is incorrect\nwe should be testing Python 3.9 as our lower-bound","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68a6044c00f59e9a9c0ac9bed0af25393a671d85","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    check:"},{"line_number":10,"context_line":"      jobs:"},{"line_number":11,"context_line":"        - watcher-tempest-functional"},{"line_number":12,"context_line":"        - watcher-tempest-functional-jammy"},{"line_number":13,"context_line":"        - watcher-grenade"},{"line_number":14,"context_line":"        - watcher-tempest-strategies"},{"line_number":15,"context_line":"        - watcher-tempest-actuator"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c030167e_6e2fcf4a","line":12,"in_reply_to":"656a69d2_817b528e","updated":"2024-10-22 18:36:46.000000000","message":"sure.","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c20b9c15f17c4f1c60c84d8d1e724c05a581cc86","unresolved":true,"context_lines":[{"line_number":180,"context_line":"    name: watcher-tempest-functional-jammy"},{"line_number":181,"context_line":"    description: This is integrated job testing on Ubuntu jammy(22.04)"},{"line_number":182,"context_line":"    parent: watcher-tempest-functional"},{"line_number":183,"context_line":"    nodeset: openstack-single-node-jammy"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"- job:"},{"line_number":186,"context_line":"    name: watcher-tempest-functional-ipv6-only"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"cb1a6370_c82df111","line":183,"updated":"2024-10-22 11:56:51.000000000","message":"+1\nthis is valid based on the smoth upgrade requiremetns when retrying a older release.\n\nhowever the comment is not entirly correct\nwe can remove this job in 2025.2\n\nthe supprot for jammy only extened to to the 2025.1 SLURP release.\nit shoudl not be included in the 2025.2 testing runtimes as 2025.1 is the release that provides the Operating system overlap.","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68a6044c00f59e9a9c0ac9bed0af25393a671d85","unresolved":false,"context_lines":[{"line_number":180,"context_line":"    name: watcher-tempest-functional-jammy"},{"line_number":181,"context_line":"    description: This is integrated job testing on Ubuntu jammy(22.04)"},{"line_number":182,"context_line":"    parent: watcher-tempest-functional"},{"line_number":183,"context_line":"    nodeset: openstack-single-node-jammy"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"- job:"},{"line_number":186,"context_line":"    name: watcher-tempest-functional-ipv6-only"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1856d198_162af822","line":183,"in_reply_to":"cb1a6370_c82df111","updated":"2024-10-22 18:36:46.000000000","message":"you are right, In the next cycle, we can remove it not when we bump default version to 26.04. I will update my comment.","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e9e551cf39a5874e3c123d8ad0bfca593527ccf6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- project:"},{"line_number":2,"context_line":"    queue: watcher"},{"line_number":3,"context_line":"    templates:"},{"line_number":4,"context_line":"      - check-requirements"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"8a219ea1_a8f5bcd3","line":1,"updated":"2024-10-23 01:10:26.000000000","message":"i know this is valid but the fact this is  not at the end of the file confused me for a bit.","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c142fbfde5407796dc80fc101913f9a9fdbe2e0c","unresolved":true,"context_lines":[{"line_number":91,"context_line":"- job:"},{"line_number":92,"context_line":"    name: watcher-tempest-multinode"},{"line_number":93,"context_line":"    parent: watcher-tempest-functional"},{"line_number":94,"context_line":"    nodeset: openstack-two-node-jammy"},{"line_number":95,"context_line":"    roles:"},{"line_number":96,"context_line":"      - zuul: openstack/tempest"},{"line_number":97,"context_line":"    group-vars:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"cb40ba7f_33428101","line":94,"range":{"start_line":94,"start_character":13,"end_line":94,"end_character":37},"updated":"2024-10-30 19:41:44.000000000","message":"we need to update this to openstack-two-node-noble","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b2b3b723913bc9954e082a21da8372dc865baeef","unresolved":false,"context_lines":[{"line_number":91,"context_line":"- job:"},{"line_number":92,"context_line":"    name: watcher-tempest-multinode"},{"line_number":93,"context_line":"    parent: watcher-tempest-functional"},{"line_number":94,"context_line":"    nodeset: openstack-two-node-jammy"},{"line_number":95,"context_line":"    roles:"},{"line_number":96,"context_line":"      - zuul: openstack/tempest"},{"line_number":97,"context_line":"    group-vars:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7cf57003_bf8362a8","line":94,"range":{"start_line":94,"start_character":13,"end_line":94,"end_character":37},"in_reply_to":"cb40ba7f_33428101","updated":"2024-10-30 19:45:29.000000000","message":"right. done","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e9e551cf39a5874e3c123d8ad0bfca593527ccf6","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    nodeset: openstack-single-node-jammy"},{"line_number":182,"context_line":"    vars:"},{"line_number":183,"context_line":"      \u003c\u003c: *base_vars"},{"line_number":184,"context_line":"      python_version: \u00273.9\u0027"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"- job:"},{"line_number":187,"context_line":"    name: watcher-tempest-functional-ipv6-only"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"8869f19d_51388f99","line":184,"updated":"2024-10-23 01:10:26.000000000","message":"+1","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"}],"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e9e551cf39a5874e3c123d8ad0bfca593527ccf6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"as default."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/932156"},{"line_number":19,"context_line":"Depends-On: https://review.opendev.org/c/openstack/watcher/+/933062"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://governance.openstack.org/tc/reference/runtimes/2025.1.html"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"0aa5ff80_08eeb4cf","line":19,"updated":"2024-10-23 01:10:26.000000000","message":"neet your using depend on to pull in the stable/2024.2 backport of the grenade/devstack venv patch to make the grenade job pass.\n\nclark told me that shoudl work but this is the first time i have seen it actully used this way correctly","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a74e8fd5b45c60122015eae1a7643ae1c85a34d5","unresolved":false,"context_lines":[{"line_number":16,"context_line":"as default."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: https://review.opendev.org/c/openstack/tempest/+/932156"},{"line_number":19,"context_line":"Depends-On: https://review.opendev.org/c/openstack/watcher/+/933062"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://governance.openstack.org/tc/reference/runtimes/2025.1.html"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"a87fd852_2da1f580","line":19,"in_reply_to":"0aa5ff80_08eeb4cf","updated":"2024-10-23 18:30:52.000000000","message":"yeah, we usually do this for greande fixes on base and target branches.","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0323bd23219aca80baf60eab4d60bf3b22b30fdc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b77c51d3_01615a02","updated":"2024-10-22 11:59:04.000000000","message":"the grenade job failed because watcher-db-manage was not found.\n\ni have not confirmed yet but this is likely related to the use of the global virutal env.\n\nill need to review the devstack log and devstack plugin code to confirm","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68a6044c00f59e9a9c0ac9bed0af25393a671d85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5dddad7c_80a224e5","in_reply_to":"b77c51d3_01615a02","updated":"2024-10-22 18:36:46.000000000","message":"yeah, its greande in venv. will fix it separatly","commit_id":"d7a867f465ef4e181717ca5521391e0a83bcaa40"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"39d5bd4a3076e42d7a5abb776756132455ce4879","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"911f4491_5998afba","updated":"2024-10-30 07:41:16.000000000","message":"i wonder why the `watcher-tempest-functional-jammy` was quite a bit faster than the `watcher-tempest-functional` but far as i can see they have the same tests \n\n\nhttps://e63f38b8e486b9ef8a45-283f569f7129297b2f76cddb21d3fd12.ssl.cf2.rackcdn.com/932963/5/check/watcher-tempest-functional/94e85d3/testr_results.html\n\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_1ed/932963/5/check/watcher-tempest-functional-jammy/1ed5623/testr_results.html","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9983bfa9b3dd6fe190d2a0b1aa3a875538cd4aca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cb3373ec_c82117ed","in_reply_to":"59cc2732_ce500db8","updated":"2024-11-04 07:47:05.000000000","message":"thanks for digging there Sean","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ee02c0eeb6e1c4ba786defaed4885e8936d93665","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"59cc2732_ce500db8","in_reply_to":"911f4491_5998afba","updated":"2024-10-30 19:45:54.000000000","message":"that is probably because fo the ci provides it ran on ovh-gra1 vs  ovh-bhs1\ncombined with the delta in base os \n\nnoble\n```\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nDevStack Component Timing\n (times are in seconds)  \n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nwait_for_service      24\nasync_wait            84\nosc                  297\napt-get              214\ntest_with_retry        5\ndbsync                 5\npip_install          338\napt-get-update         1\nrun_process           42\n-------------------------\nUnaccounted time     255\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nTotal runtime        1265\n```\n\njammy\n```\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nDevStack Component Timing\n (times are in seconds)  \n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nwait_for_service      18\nasync_wait            56\nosc                  220\napt-get              104\ntest_with_retry        5\ndbsync                 3\npip_install          200\napt-get-update         1\nrun_process           24\n-------------------------\nUnaccounted time     118\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nTotal runtime        749\n\n```\n\nthe apt and pip install times are much higher on the noble job possibel due to mirrors.","commit_id":"9ee8b7daa64a296f7d7e15a8a8160aeffc917583"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"31e54c23e1c5ffaa13658d641f7a1a65a77cd632","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d3e501bc_fe36d21a","updated":"2024-10-30 19:49:57.000000000","message":"+1 thansk for the update ill check back when ci reports.\n\nim hoping that this will either repoduce the eventlet/sqlachmy bug or show that its a local issue.","commit_id":"f1c21376f32749fa8e89944b9795455f3ccc0945"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5688c97a25aa22cc64d6ec3b3de2bd93c42d0060","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f54005d7_b0f831bb","updated":"2024-10-31 01:10:13.000000000","message":"ok this repoduces the same eventlet issue im seeing locally \n\nhttps://zuul.opendev.org/t/openstack/build/fc44a4ed60534340ae00fa2cb9cb8c1d/log/controller/logs/screen-watcher-decision-engine.txt#899\n\nhttps://zuul.opendev.org/t/openstack/build/fc44a4ed60534340ae00fa2cb9cb8c1d/log/controller/logs/screen-watcher-applier.txt?severity\u003d1#611\n\nand evenutlly the error chagnes to the max connection too.\nhttps://zuul.opendev.org/t/openstack/build/fc44a4ed60534340ae00fa2cb9cb8c1d/log/controller/logs/screen-watcher-decision-engine.txt#16421\n\ni dont think there is anything wrong with this patch but the is somethign wrong with eventlet, sqlachamy and how watcher is mixing native threads, asyncio and eventlet.","commit_id":"f1c21376f32749fa8e89944b9795455f3ccc0945"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9983bfa9b3dd6fe190d2a0b1aa3a875538cd4aca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"459fb7d8_683538cb","updated":"2024-11-04 07:47:05.000000000","message":"nice so looks like Sean\u0027s patch is good to fix the strategies job https://228fda1271fe0f503d66-4a7312796dc78ee19ab8c1340b1f0bd5.ssl.cf5.rackcdn.com/932963/7/check/watcher-tempest-strategies/c080d0d/testr_results.html","commit_id":"88b923a2033610e96accc28328d47218aec05ba4"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"7a29880229a769d4b082bb0c7846485c16774ad1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6e4a23fe_b0d8ca93","updated":"2024-11-07 15:30:32.000000000","message":"recheck","commit_id":"88b923a2033610e96accc28328d47218aec05ba4"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7c7580594f6e1c2fb8d9b447157723df0e655080","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d81bdd1d_8127b92a","updated":"2024-11-19 14:11:26.000000000","message":"note i just rebased this patch so im ok with +1ing it\nthis is still the orginal content proposed by gmann","commit_id":"8da3ce5db97b7b65fdca8bd8b3e5c32e4690c2e0"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e85961752e07a8122dd45b684212e237d2dd7c36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"7e1e5eaa_809b2867","updated":"2024-11-25 07:22:29.000000000","message":"https://10c8432188e130db822f-fccbcab9ae62b41b4b85e737441d4315.ssl.cf5.rackcdn.com/932963/13/check/openstack-tox-cover/8a37f5b/job-output.txt\n\nI thought this may be a one-off fail but after looking at the error it may well be related to Sean\u0027s patch on top since it fails on apscheduler.scheduler \n\n\n```\n2024-11-22 17:50:34.285072 | ubuntu-noble | {3} watcher.tests.decision_engine.strategy.strategies.test_storage_capacity_balance.TestStorageCapacityBalance.test_retype_fit [0.077308s] ... ok\n2024-11-22 17:50:34.328624 | ubuntu-noble | 2024-11-22 17:50:34.307 3315 ERROR apscheduler.scheduler [-] Error submitting job \"TestSchedulerMonkeyPatching.test_scheduler_eventlet_monkey_patch_called_when_should_patch.\u003clocals\u003e.work (trigger: interval[0:00:02], next run at: 2024-11-22 17:50:34 UTC)\" to executor \"default\": RuntimeError: Can not schedule new futures after being shutdown\n2024-11-22 17:50:34.328950 | ubuntu-noble | 2024-11-22 17:50:34.307 3315 ERROR apscheduler.scheduler Traceback (most recent call last):\n```","commit_id":"807b4c614ea169802e905d6d21fe3e53835ed003"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"446ed668235a201441eac9ff92e748150c7dd2f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"4ca3a87d_036c3764","updated":"2024-12-02 18:43:04.000000000","message":"both depends-on are now merged and the gate is now using noble by default so +w","commit_id":"c80c940a4fd82fe94d144a6a98af092340e2f6a7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"af1c242c4e29d99597df175b81ebb4ea5eeb1448","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"965b2882_acdf23d3","updated":"2024-11-29 02:22:17.000000000","message":"holding +w until the proceeding patches are merged and the depends on is resolved.","commit_id":"c80c940a4fd82fe94d144a6a98af092340e2f6a7"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4552ea7f30f602cfe97ca7665192449f24551598","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0cf1448c_945f8226","updated":"2024-11-30 19:25:14.000000000","message":"recheck depends-on is merged, this is good to go","commit_id":"c80c940a4fd82fe94d144a6a98af092340e2f6a7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a8901e6557ff6f18f7c3304d8631a06c8bcbe9d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"64769e6b_52cf2f09","updated":"2024-12-02 18:44:50.000000000","message":"recheck https://review.opendev.org/c/openstack/watcher/+/936654 is now merged.","commit_id":"c80c940a4fd82fe94d144a6a98af092340e2f6a7"}]}