)]}'
{".zuul.yaml":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e2e5e0bd0e33aab088db6c52c6bfd001356cbfc1","unresolved":true,"context_lines":[{"line_number":140,"context_line":"    irrelevant-files: *irrelevent_files"},{"line_number":141,"context_line":"    timeout: 7800"},{"line_number":142,"context_line":"    required-projects: \u0026base_sg_required_projects"},{"line_number":143,"context_line":"      - openstack/aodh"},{"line_number":144,"context_line":"      - openstack/ceilometer"},{"line_number":145,"context_line":"      - openstack/tempest"},{"line_number":146,"context_line":"      - openstack-k8s-operators/sg-core"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"db1341c0_78fd2cd7","side":"PARENT","line":143,"updated":"2025-07-21 09:45:59.000000000","message":"aodh project from https://github.com/openstack/watcher/blob/master/devstack/local.conf.controller#L48 also?","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f78fcba0e76514c52b0d22ce5a6ceee69099926e","unresolved":false,"context_lines":[{"line_number":140,"context_line":"    irrelevant-files: *irrelevent_files"},{"line_number":141,"context_line":"    timeout: 7800"},{"line_number":142,"context_line":"    required-projects: \u0026base_sg_required_projects"},{"line_number":143,"context_line":"      - openstack/aodh"},{"line_number":144,"context_line":"      - openstack/ceilometer"},{"line_number":145,"context_line":"      - openstack/tempest"},{"line_number":146,"context_line":"      - openstack-k8s-operators/sg-core"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"481fbf3f_72759d76","side":"PARENT","line":143,"in_reply_to":"9a7312ec_9b20e3e8","updated":"2025-07-21 11:40:09.000000000","message":"Acknowledged","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"f4ab7429b1b2766b50f072600ca5c7095f6e7a2a","unresolved":true,"context_lines":[{"line_number":140,"context_line":"    irrelevant-files: *irrelevent_files"},{"line_number":141,"context_line":"    timeout: 7800"},{"line_number":142,"context_line":"    required-projects: \u0026base_sg_required_projects"},{"line_number":143,"context_line":"      - openstack/aodh"},{"line_number":144,"context_line":"      - openstack/ceilometer"},{"line_number":145,"context_line":"      - openstack/tempest"},{"line_number":146,"context_line":"      - openstack-k8s-operators/sg-core"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9a7312ec_9b20e3e8","side":"PARENT","line":143,"in_reply_to":"db1341c0_78fd2cd7","updated":"2025-07-21 09:59:08.000000000","message":"That depends. I\u0027m happy to include the deletion in this patch, but I saw you have some docs about auditing using Aodh alarms https://docs.openstack.org/watcher/latest/user/event_type_audit.html . For that you\u0027ll need Aodh running. So while it makes sense to me to remove Aodh from the zuul job as Aodh isn\u0027t used there, maybe you still want to keep it in the sample local.conf just in case somebody wants to work on that feature?","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e2e5e0bd0e33aab088db6c52c6bfd001356cbfc1","unresolved":true,"context_lines":[{"line_number":173,"context_line":"      devstack_localrc:"},{"line_number":174,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":175,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":176,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":177,"context_line":"        # avoid collecting real host cpu metrics since tests"},{"line_number":178,"context_line":"        # will inject fake metrics when needed"},{"line_number":179,"context_line":"        NODE_EXPORTER_COLLECTOR_EXCLUDE: \"cpu\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b2913feb_f4a73a4f","side":"PARENT","line":176,"range":{"start_line":176,"start_character":8,"end_line":176,"end_character":34},"updated":"2025-07-21 09:45:59.000000000","message":"Can we also drop this setting from here https://github.com/openstack/watcher/blob/master/devstack/local.conf.controller#L21?","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a321d5349a13eb123a63d9365d9b3d1e2a86b3bb","unresolved":false,"context_lines":[{"line_number":173,"context_line":"      devstack_localrc:"},{"line_number":174,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":175,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":176,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":177,"context_line":"        # avoid collecting real host cpu metrics since tests"},{"line_number":178,"context_line":"        # will inject fake metrics when needed"},{"line_number":179,"context_line":"        NODE_EXPORTER_COLLECTOR_EXCLUDE: \"cpu\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1de9bf34_5023c91b","side":"PARENT","line":176,"range":{"start_line":176,"start_character":8,"end_line":176,"end_character":34},"in_reply_to":"64bdc964_fc08e371","updated":"2025-07-21 14:31:10.000000000","message":"Done","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"f4ab7429b1b2766b50f072600ca5c7095f6e7a2a","unresolved":true,"context_lines":[{"line_number":173,"context_line":"      devstack_localrc:"},{"line_number":174,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":175,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":176,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":177,"context_line":"        # avoid collecting real host cpu metrics since tests"},{"line_number":178,"context_line":"        # will inject fake metrics when needed"},{"line_number":179,"context_line":"        NODE_EXPORTER_COLLECTOR_EXCLUDE: \"cpu\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c300813f_c45b7c82","side":"PARENT","line":176,"range":{"start_line":176,"start_character":8,"end_line":176,"end_character":34},"in_reply_to":"b2913feb_f4a73a4f","updated":"2025-07-21 09:59:08.000000000","message":"As this is used for tempest testing, this makes sense.","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f78fcba0e76514c52b0d22ce5a6ceee69099926e","unresolved":true,"context_lines":[{"line_number":173,"context_line":"      devstack_localrc:"},{"line_number":174,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":175,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":176,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":177,"context_line":"        # avoid collecting real host cpu metrics since tests"},{"line_number":178,"context_line":"        # will inject fake metrics when needed"},{"line_number":179,"context_line":"        NODE_EXPORTER_COLLECTOR_EXCLUDE: \"cpu\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"64bdc964_fc08e371","side":"PARENT","line":176,"range":{"start_line":176,"start_character":8,"end_line":176,"end_character":34},"in_reply_to":"c300813f_c45b7c82","updated":"2025-07-21 11:40:09.000000000","message":"thanks! Sounds good to drop it.","commit_id":"374750847f3ca2284b455792c414808a55e181ff"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ae3539459180e971ad18c63271de6389cf09c877","unresolved":true,"context_lines":[{"line_number":151,"context_line":"      configure_swap_size: 8192"},{"line_number":152,"context_line":"      devstack_plugins:"},{"line_number":153,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":154,"context_line":"        sg-core: https://github.com/openstack-k8s-operators/sg-core"},{"line_number":155,"context_line":"        watcher: https://opendev.org/openstack/watcher"},{"line_number":156,"context_line":"        devstack-plugin-prometheus: https://opendev.org/openstack/devstack-plugin-prometheus"},{"line_number":157,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0fddb091_a1f4d5f1","line":154,"range":{"start_line":154,"start_character":0,"end_line":154,"end_character":67},"updated":"2025-07-21 18:03:39.000000000","message":"as an aside when ever celiomerter complete deocuemation fo the\nnon sg core base workflow with the promethus scrap config endpoint\nwe shoudl probably consider adopting it so we can remove sg core.\n\nuntil that feature is properly documented however we cant really use it in the job.","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"5a90155359b82481df45f1f89e0ab13ec5291017","unresolved":false,"context_lines":[{"line_number":151,"context_line":"      configure_swap_size: 8192"},{"line_number":152,"context_line":"      devstack_plugins:"},{"line_number":153,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":154,"context_line":"        sg-core: https://github.com/openstack-k8s-operators/sg-core"},{"line_number":155,"context_line":"        watcher: https://opendev.org/openstack/watcher"},{"line_number":156,"context_line":"        devstack-plugin-prometheus: https://opendev.org/openstack/devstack-plugin-prometheus"},{"line_number":157,"context_line":"      devstack_services:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d596f381_adac2b84","line":154,"range":{"start_line":154,"start_character":0,"end_line":154,"end_character":67},"in_reply_to":"0fddb091_a1f4d5f1","updated":"2025-08-12 21:46:28.000000000","message":"Agreed. We\u0027ll discuss further during the PTG.","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"}],"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ae3539459180e971ad18c63271de6389cf09c877","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Looking at previous builds of watcher-prometheus-integration job,"},{"line_number":10,"context_line":"it doesn\u0027t seem like Aodh is being used at all. So it seems like"},{"line_number":11,"context_line":"it doesn\u0027t need to be deployed in the job."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The CEILOMETER_ALARM_THRESHOLD variable is used to set a threshold"},{"line_number":14,"context_line":"for autoscaling when executing the autoscaling scenario from the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"393c2ed9_86514a51","line":11,"updated":"2025-07-21 18:03:39.000000000","message":"its not currently beut that only becasue we do not have coverage\nof event based audits yet.\n\nwe could disable it for now but it will eventually need to be added again.","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"5a90155359b82481df45f1f89e0ab13ec5291017","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Looking at previous builds of watcher-prometheus-integration job,"},{"line_number":10,"context_line":"it doesn\u0027t seem like Aodh is being used at all. So it seems like"},{"line_number":11,"context_line":"it doesn\u0027t need to be deployed in the job."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The CEILOMETER_ALARM_THRESHOLD variable is used to set a threshold"},{"line_number":14,"context_line":"for autoscaling when executing the autoscaling scenario from the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a0107444_a6ebeb97","line":11,"in_reply_to":"393c2ed9_86514a51","updated":"2025-08-12 21:46:28.000000000","message":"Acknowledged","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ae3539459180e971ad18c63271de6389cf09c877","unresolved":true,"context_lines":[{"line_number":14,"context_line":"for autoscaling when executing the autoscaling scenario from the"},{"line_number":15,"context_line":"telemetry-tempest-plugin. This scenario isn\u0027t being run in the"},{"line_number":16,"context_line":"watcher-prometheus-integration job, so the variable is unused and"},{"line_number":17,"context_line":"doesn\u0027t need to be there."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I271a891b9c7be239348486818d4cedc43aa74515"},{"line_number":20,"context_line":"Signed-off-by: Jaromir Wysoglad \u003cjwysogla@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e2ce8528_48c12b0c","line":17,"updated":"2025-07-21 18:03:39.000000000","message":"ack that makes sense to remove","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"5a90155359b82481df45f1f89e0ab13ec5291017","unresolved":false,"context_lines":[{"line_number":14,"context_line":"for autoscaling when executing the autoscaling scenario from the"},{"line_number":15,"context_line":"telemetry-tempest-plugin. This scenario isn\u0027t being run in the"},{"line_number":16,"context_line":"watcher-prometheus-integration job, so the variable is unused and"},{"line_number":17,"context_line":"doesn\u0027t need to be there."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I271a891b9c7be239348486818d4cedc43aa74515"},{"line_number":20,"context_line":"Signed-off-by: Jaromir Wysoglad \u003cjwysogla@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6bca022c_80008bf8","line":17,"in_reply_to":"e2ce8528_48c12b0c","updated":"2025-08-12 21:46:28.000000000","message":"Acknowledged","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e2e5e0bd0e33aab088db6c52c6bfd001356cbfc1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"030b25e8_7a9cbda1","updated":"2025-07-21 09:45:59.000000000","message":"Thank you @jwysogla@redhat.com for improving the zuul job. The following project aodh and setting is mentioned in https://github.com/openstack/watcher/blob/master/devstack/local.conf.controller#L48, I think we can remove it from there also.","commit_id":"9c815657f98c5f6047e1b729aede5714e16a95d8"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a321d5349a13eb123a63d9365d9b3d1e2a86b3bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4788a445_4f021328","in_reply_to":"030b25e8_7a9cbda1","updated":"2025-07-21 14:31:10.000000000","message":"Done","commit_id":"9c815657f98c5f6047e1b729aede5714e16a95d8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ae3539459180e971ad18c63271de6389cf09c877","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca694dbc_b27e6303","updated":"2025-07-21 18:03:39.000000000","message":"for now this is ok we can readd aodh when we add testing of the event based audits.\n\n+1 while others review.","commit_id":"16a0173874cdbf2db263c9b901ce3aff82fbd7ee"}]}