)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":"- job:"},{"line_number":213,"context_line":"    name: watcher-sg-core-tempest-base"},{"line_number":214,"context_line":"    parent: devstack-tempest"},{"line_number":215,"context_line":"    description: |"},{"line_number":216,"context_line":"      This job is for testing watcher and sg-core/prometheus installation"},{"line_number":217,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"2774c448_cabb5901","line":214,"updated":"2024-11-15 17:08:47.000000000","message":"so this is fine to use as a base but it uses a singel node nodeset by default \n\neventually we will want to use at least\nnodeset: openstack-two-node-noble \n\nhowever we may want tree ndoes\n\nwhen i added openstack-two-node-noble i did nto add a 3 node version\n\nhttps://github.com/openstack/devstack/blob/master/.zuul.yaml#L154-L182\n\nfor now without my executor patch to resole the eventlet issue on nbole notble wont work so initally i would added\n\n```\n nodeset: openstack-two-node-jammy\n```\n\ninstead  and we can update this job to noble in a follow up patch.","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":211,"context_line":""},{"line_number":212,"context_line":"- job:"},{"line_number":213,"context_line":"    name: watcher-sg-core-tempest-base"},{"line_number":214,"context_line":"    parent: devstack-tempest"},{"line_number":215,"context_line":"    description: |"},{"line_number":216,"context_line":"      This job is for testing watcher and sg-core/prometheus installation"},{"line_number":217,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d9a05b12_a99a4ad0","line":214,"in_reply_to":"2774c448_cabb5901","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":220,"context_line":"    required-projects: \u0026base_sg_required_projects"},{"line_number":221,"context_line":"      - openstack/aodh"},{"line_number":222,"context_line":"      - openstack/ceilometer"},{"line_number":223,"context_line":"      - openstack/heat-tempest-plugin"},{"line_number":224,"context_line":"      - openstack/heat"},{"line_number":225,"context_line":"      - openstack/diskimage-builder"},{"line_number":226,"context_line":"      - openstack/tempest"},{"line_number":227,"context_line":"      - gnocchixyz/gnocchi"},{"line_number":228,"context_line":"      - infrawatch/sg-core"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9858d67b_39ec15ab","line":225,"range":{"start_line":223,"start_character":5,"end_line":225,"end_character":35},"updated":"2024-11-15 17:08:47.000000000","message":"as far as i am awer we do not need heat or diskimage builder","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":220,"context_line":"    required-projects: \u0026base_sg_required_projects"},{"line_number":221,"context_line":"      - openstack/aodh"},{"line_number":222,"context_line":"      - openstack/ceilometer"},{"line_number":223,"context_line":"      - openstack/heat-tempest-plugin"},{"line_number":224,"context_line":"      - openstack/heat"},{"line_number":225,"context_line":"      - openstack/diskimage-builder"},{"line_number":226,"context_line":"      - openstack/tempest"},{"line_number":227,"context_line":"      - gnocchixyz/gnocchi"},{"line_number":228,"context_line":"      - infrawatch/sg-core"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"2e4e62be_a2ab4df9","line":225,"range":{"start_line":223,"start_character":5,"end_line":225,"end_character":35},"in_reply_to":"9858d67b_39ec15ab","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":224,"context_line":"      - openstack/heat"},{"line_number":225,"context_line":"      - openstack/diskimage-builder"},{"line_number":226,"context_line":"      - openstack/tempest"},{"line_number":227,"context_line":"      - gnocchixyz/gnocchi"},{"line_number":228,"context_line":"      - infrawatch/sg-core"},{"line_number":229,"context_line":"      - openstack/watcher"},{"line_number":230,"context_line":"      - openstack/python-watcherclient"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b05f3a98_b42eb30d","line":227,"range":{"start_line":227,"start_character":6,"end_line":227,"end_character":26},"updated":"2024-11-15 17:08:47.000000000","message":"we also shoudl nto require gnocchi","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":224,"context_line":"      - openstack/heat"},{"line_number":225,"context_line":"      - openstack/diskimage-builder"},{"line_number":226,"context_line":"      - openstack/tempest"},{"line_number":227,"context_line":"      - gnocchixyz/gnocchi"},{"line_number":228,"context_line":"      - infrawatch/sg-core"},{"line_number":229,"context_line":"      - openstack/watcher"},{"line_number":230,"context_line":"      - openstack/python-watcherclient"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"24a7999c_e809f0b8","line":227,"range":{"start_line":227,"start_character":6,"end_line":227,"end_character":26},"in_reply_to":"b05f3a98_b42eb30d","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":232,"context_line":"    vars:"},{"line_number":233,"context_line":"      configure_swap_size: 8192"},{"line_number":234,"context_line":"      devstack_plugins:"},{"line_number":235,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":236,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":237,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":238,"context_line":"        sg-core: https://github.com/infrawatch/sg-core"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d7da836d_7689d2a2","line":235,"range":{"start_line":235,"start_character":7,"end_line":235,"end_character":48},"updated":"2024-11-15 17:08:47.000000000","message":"so we can remove this","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"fe849bacc0465043d4ddbd0630431490dabdd377","unresolved":false,"context_lines":[{"line_number":232,"context_line":"    vars:"},{"line_number":233,"context_line":"      configure_swap_size: 8192"},{"line_number":234,"context_line":"      devstack_plugins:"},{"line_number":235,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":236,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":237,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":238,"context_line":"        sg-core: https://github.com/infrawatch/sg-core"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"eddcb660_371a257a","line":235,"range":{"start_line":235,"start_character":7,"end_line":235,"end_character":48},"in_reply_to":"d7da836d_7689d2a2","updated":"2024-11-15 20:17:36.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":237,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":238,"context_line":"        sg-core: https://github.com/infrawatch/sg-core"},{"line_number":239,"context_line":"        watcher: https://opendev.org/openstack/watcher"},{"line_number":240,"context_line":"      devstack_services:"},{"line_number":241,"context_line":"        watcher-api: true"},{"line_number":242,"context_line":"        watcher-decision-engine: true"},{"line_number":243,"context_line":"        watcher-applier: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"eb3db8df_f120de80","line":240,"updated":"2024-11-15 17:08:47.000000000","message":"for sevices we do not need swift or in the shorterm cidner.\n\nhttps://github.com/openstack/nova/blob/master/.zuul.yaml#L195-L203\n\nso i woudl copy htis\n\n```\n        # NOTE(sean-k-mooney) this job is testing ml2/ovs with\n        # hybrid plug we do not need to test cinder and we want to ensure\n        # that we have at least one job without cinder in our gate so we\n        # disable it in this job\n        # Cinder services\n        c-api: false\n        c-bak: false\n        c-sch: false\n        c-vol: false\n        # we also do not need swift in this job so disable it for speed\n        s-account: false\n        s-container: false\n        s-object: false\n        s-proxy: false\n```\n     \nthe base devstack jobs form which devstack-tempest inherits\n\nhttps://github.com/openstack/devstack/blob/master/.zuul.yaml#L482\n\ndoes not enable heat by default so we shoudl nto need to disabel it here.","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":237,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":238,"context_line":"        sg-core: https://github.com/infrawatch/sg-core"},{"line_number":239,"context_line":"        watcher: https://opendev.org/openstack/watcher"},{"line_number":240,"context_line":"      devstack_services:"},{"line_number":241,"context_line":"        watcher-api: true"},{"line_number":242,"context_line":"        watcher-decision-engine: true"},{"line_number":243,"context_line":"        watcher-applier: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"feca72e2_d36c4189","line":240,"in_reply_to":"eb3db8df_f120de80","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":true,"context_lines":[{"line_number":244,"context_line":"        tempest: true"},{"line_number":245,"context_line":"      devstack_localrc:"},{"line_number":246,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":247,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""},{"line_number":248,"context_line":"        CEILOMETER_BACKENDS: \"gnocchi,sg-core\""},{"line_number":249,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":250,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7cf1fd99_7d408085","line":247,"updated":"2024-11-15 18:17:26.000000000","message":"removing this as well - no Gnocchi","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ef1999852ce384e59a5339a61d621ff670e75de1","unresolved":false,"context_lines":[{"line_number":244,"context_line":"        tempest: true"},{"line_number":245,"context_line":"      devstack_localrc:"},{"line_number":246,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":247,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""},{"line_number":248,"context_line":"        CEILOMETER_BACKENDS: \"gnocchi,sg-core\""},{"line_number":249,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":250,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"23f58dfc_795381dd","line":247,"in_reply_to":"7cf1fd99_7d408085","updated":"2024-11-26 11:58:46.000000000","message":"Acknowledged","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":245,"context_line":"      devstack_localrc:"},{"line_number":246,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":247,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""},{"line_number":248,"context_line":"        CEILOMETER_BACKENDS: \"gnocchi,sg-core\""},{"line_number":249,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":250,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":251,"context_line":"        GLOBAL_VENV: False"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"880a2f31_5db569ea","line":248,"range":{"start_line":248,"start_character":30,"end_line":248,"end_character":37},"updated":"2024-11-15 17:08:47.000000000","message":"and remove gnocchi here","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":245,"context_line":"      devstack_localrc:"},{"line_number":246,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":247,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""},{"line_number":248,"context_line":"        CEILOMETER_BACKENDS: \"gnocchi,sg-core\""},{"line_number":249,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":250,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":251,"context_line":"        GLOBAL_VENV: False"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"54f15d4d_7dccc6ec","line":248,"range":{"start_line":248,"start_character":30,"end_line":248,"end_character":37},"in_reply_to":"880a2f31_5db569ea","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[{"line_number":255,"context_line":"            service_available:"},{"line_number":256,"context_line":"              sg_core: True"},{"line_number":257,"context_line":"            telemetry_services:"},{"line_number":258,"context_line":"              metric_backends: gnocchi,prometheus"},{"line_number":259,"context_line":"            telemetry:"},{"line_number":260,"context_line":"              disable_ssl_certificate_validation: True"},{"line_number":261,"context_line":"              ceilometer_polling_interval: 15"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6deab9ef_86ee44ae","line":258,"range":{"start_line":258,"start_character":31,"end_line":258,"end_character":38},"updated":"2024-11-15 17:08:47.000000000","message":"and here","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            service_available:"},{"line_number":256,"context_line":"              sg_core: True"},{"line_number":257,"context_line":"            telemetry_services:"},{"line_number":258,"context_line":"              metric_backends: gnocchi,prometheus"},{"line_number":259,"context_line":"            telemetry:"},{"line_number":260,"context_line":"              disable_ssl_certificate_validation: True"},{"line_number":261,"context_line":"              ceilometer_polling_interval: 15"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"177e978e_ccf36802","line":258,"range":{"start_line":258,"start_character":31,"end_line":258,"end_character":38},"in_reply_to":"6deab9ef_86ee44ae","updated":"2024-11-15 18:17:26.000000000","message":"Done","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"85d9afef7af10f4032dbf7247e4f534914da8b3f","unresolved":true,"context_lines":[{"line_number":262,"context_line":"      tempest_plugins:"},{"line_number":263,"context_line":"        - watcher-tempest-plugin"},{"line_number":264,"context_line":"      # TODO (rlandy) modify as tests come on line"},{"line_number":265,"context_line":"      tempest_test_regex: watcher_tempest_plugin.tests.api"},{"line_number":266,"context_line":"      tox_envlist: all"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f0ca9fd_7159602c","line":265,"updated":"2024-11-15 18:17:26.000000000","message":"Changed to watcher_tempest_plugin.tests.* ... might run too long - we\u0027ll see","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ef1999852ce384e59a5339a61d621ff670e75de1","unresolved":false,"context_lines":[{"line_number":262,"context_line":"      tempest_plugins:"},{"line_number":263,"context_line":"        - watcher-tempest-plugin"},{"line_number":264,"context_line":"      # TODO (rlandy) modify as tests come on line"},{"line_number":265,"context_line":"      tempest_test_regex: watcher_tempest_plugin.tests.api"},{"line_number":266,"context_line":"      tox_envlist: all"},{"line_number":267,"context_line":""},{"line_number":268,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ec550947_0fc9a812","line":265,"in_reply_to":"1f0ca9fd_7159602c","updated":"2024-11-26 11:58:46.000000000","message":"Acknowledged","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"1ee4c6574cb2b9acaab5b1f9bd08b1be571b8155","unresolved":true,"context_lines":[{"line_number":256,"context_line":"        # Gnocchi"},{"line_number":257,"context_line":"        gnocchi-api: false"},{"line_number":258,"context_line":"        gnocchi-metricd: false"},{"line_number":259,"context_line":"     devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"f4a2b487_a1d92f68","line":259,"range":{"start_line":259,"start_character":5,"end_line":259,"end_character":21},"updated":"2024-11-15 20:33:02.000000000","message":"this needs one more space to be correctly indented to avoid the error message, it\u0027s one too much on the left now","commit_id":"1984de0ed49b93fd2f666cf800934b6fe1428d96"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"e795fe2e95bafefef72f62c474ab5c18865273d5","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        # Gnocchi"},{"line_number":257,"context_line":"        gnocchi-api: false"},{"line_number":258,"context_line":"        gnocchi-metricd: false"},{"line_number":259,"context_line":"     devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"f2319f0a_4ed63f51","line":259,"range":{"start_line":259,"start_character":5,"end_line":259,"end_character":21},"in_reply_to":"f4a2b487_a1d92f68","updated":"2024-11-15 20:38:38.000000000","message":"Done","commit_id":"1984de0ed49b93fd2f666cf800934b6fe1428d96"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"fac03e94588eeec7ea56c6dc83e0df6fc6fa4015","unresolved":true,"context_lines":[{"line_number":254,"context_line":"        s-object: false"},{"line_number":255,"context_line":"        s-proxy: false"},{"line_number":256,"context_line":"      devstack_localrc:"},{"line_number":257,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":258,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":259,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":260,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1abb6992_8324cab9","line":257,"updated":"2024-11-25 07:01:16.000000000","message":"perhaps we are we missing some aodh config here (no idea just guessing), the aodh-api failed to start https://f1a145115186f00be5c6-01ce71cda54df73c195ec13ca394b56d.ssl.cf1.rackcdn.com/935371/10/check/watcher-prometheus-integration/ef43876/compute1/logs/screen-aodh-api.txt\n\n```\nNov 25 02:25:55.004748 np0039152760 devstack@aodh-api.service[56394]: bind(): Cannot assign requested address [core/socket.c line 769]\nNov 25 02:25:55.005450 np0039152760 systemd[1]: devstack@aodh-api.service: Main process exited, code\u003dexited, status\u003d1/FAILURE\nNov 25 02:25:55.005487 np0039152760 systemd[1]: devstack@aodh-api.service: Failed with result \u0027exit-code\u0027.\nNov 25 02:25:55.006225 np0039152760 systemd[1]: Failed to start Devstack devstack@aodh-api.service.\nNov 25 02:25:55.235967 np0039152760 systemd[1]: devstack@aodh-api.service: Scheduled restart job, restart counter is at 4.\n```","commit_id":"533850db654a322da147d25018818c94a988f751"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"e0ada1505049dbef78bf80904e87b4b9da9f111d","unresolved":false,"context_lines":[{"line_number":254,"context_line":"        s-object: false"},{"line_number":255,"context_line":"        s-proxy: false"},{"line_number":256,"context_line":"      devstack_localrc:"},{"line_number":257,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":258,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":259,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":260,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"d02ade8e_f767be19","line":257,"in_reply_to":"1abb6992_8324cab9","updated":"2024-12-08 22:56:34.000000000","message":"Done","commit_id":"533850db654a322da147d25018818c94a988f751"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2bb9766a5d313a3ebb6154ff3e054a26a7640323","unresolved":false,"context_lines":[{"line_number":284,"context_line":"    group-vars:"},{"line_number":285,"context_line":"      subnode:"},{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"1c39b66d_e630b172","line":287,"updated":"2024-11-26 08:00:44.000000000","message":"nope that is not a typo :D  \nhttps://opendev.org/openstack/ceilometer/src/commit/a11acf020451ccb69612a9fdbf479e17d4b56796/devstack/settings#L3","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ef1999852ce384e59a5339a61d621ff670e75de1","unresolved":false,"context_lines":[{"line_number":284,"context_line":"    group-vars:"},{"line_number":285,"context_line":"      subnode:"},{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"560ef350_0c1f1349","line":287,"in_reply_to":"1c39b66d_e630b172","updated":"2024-11-26 11:58:46.000000000","message":"The A stands for agent but it really is just odd.","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2bb9766a5d313a3ebb6154ff3e054a26a7640323","unresolved":true,"context_lines":[{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"- job:"},{"line_number":292,"context_line":"    name: watcher-prometheus-integration"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"847f52b4_d55c3897","line":289,"updated":"2024-11-26 08:00:44.000000000","message":"it didn\u0027t like that \nhttps://9a8601bd1ae2889ac6d3-dc95df9dc6d9c4be8998bc6e49021df1.ssl.cf2.rackcdn.com/935371/13/check/watcher-prometheus-integration/2655c61/job-output.txt\n\n```\n2024-11-25 19:20:20.683137 | compute1 | /opt/stack/ceilometer/devstack/settings: line 14: CEILOMETER_BACKEND: \"\": syntax error: operand expected (error token is \"\"\"\")\n\n```","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b3c152305a6febe84ce63a498f17b2b88d2a9a66","unresolved":true,"context_lines":[{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"- job:"},{"line_number":292,"context_line":"    name: watcher-prometheus-integration"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9b096f06_0acadb35","line":289,"in_reply_to":"19991a59_67352328","updated":"2024-11-27 07:44:05.000000000","message":"ok thanks for the extra context then i think setting them both \"none\" should be OK looking at https://opendev.org/openstack/ceilometer/src/commit/27faf0e97c52aa1379aafa9ac02ccced7a2c749d/devstack/settings#L12-L25 \n\n```\n# Gnocchi is the default backind if both \"CEILOMETER_BACKEND\"\n# and \"CEILOMETER_BACKENDS\" are empty\nCEILOMETER_BACKEND\u003d${CEILOMETER_BACKEND:\"\"}\nif ! [[ \"$CEILOMETER_BACKENDS\" \u003d~ \"$CEILOMETER_BACKEND\" ]]; then\n    CEILOMETER_BACKENDS+\u003d\",\"$CEILOMETER_BACKEND\nfi\nCEILOMETER_BACKENDS\u003d${CEILOMETER_BACKENDS:-\"gnocchi\"}\n\nif [[ \"$CEILOMETER_BACKENDS\" \u003d~ \"gnocchi\" ]]; then\n    enable_service gnocchi-api gnocchi-metricd\nfi\nif [[ \"$CEILOMETER_BACKENDS\" \u003d~ \"sg-core\" ]]; then\n    enable_service sg-core\nfi\n\n```","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"965130475c073db9e8b06505db04ec0259390079","unresolved":true,"context_lines":[{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"- job:"},{"line_number":292,"context_line":"    name: watcher-prometheus-integration"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"c9a2cf9f_bd61ed89","line":289,"in_reply_to":"847f52b4_d55c3897","updated":"2024-11-26 10:36:30.000000000","message":"+1 to marios\u0027s comment. Based on https://opendev.org/openstack/ceilometer/src/branch/master/devstack/settings#L14 `CEILOMETER_BACKEND\u003d${CEILOMETER_BACKEND:\"\"}`\n\nI think setting `CEILOMETER_BACKEND: \"\"` will do the job.","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7ef7b94c7b9877089a1ca91f5b168c4b308888f6","unresolved":false,"context_lines":[{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"- job:"},{"line_number":292,"context_line":"    name: watcher-prometheus-integration"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"f4061be2_9dd216fa","line":289,"in_reply_to":"9b096f06_0acadb35","updated":"2024-12-09 13:43:06.000000000","message":"Done","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"ef1999852ce384e59a5339a61d621ff670e75de1","unresolved":true,"context_lines":[{"line_number":286,"context_line":"        devstack_services:"},{"line_number":287,"context_line":"          ceilometer-acompute: true"},{"line_number":288,"context_line":"        devstack_localrc:"},{"line_number":289,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"- job:"},{"line_number":292,"context_line":"    name: watcher-prometheus-integration"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"19991a59_67352328","line":289,"in_reply_to":"c9a2cf9f_bd61ed89","updated":"2024-11-26 11:58:46.000000000","message":"so the value i used locally was\n\nCEILOMETER_BACKEND\u003d\"none\"\n\nthe quotes are important in this context i belive as this is a string.\n\nnone is one of the documetned values\n\nhttps://opendev.org/openstack/ceilometer/src/commit/cf2471219359591e920ad122ff2d94a284296e70/devstack/plugin.sh#L37\n\nwe are tryign to make sure gnocci and sg core are not installed on teh comptue nodes so i dont think `CEILOMETER_BACKEND: \"\"` will work\n\nwe need to ensure that this code does not enable either service\nhttps://opendev.org/openstack/ceilometer/src/branch/master/devstack/settings#L12-L25\n\nwe coudl set CEILOMETER_BACKENDS to none as well directly to ensure that.","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b3c152305a6febe84ce63a498f17b2b88d2a9a66","unresolved":true,"context_lines":[{"line_number":295,"context_line":"          ceilometer-acompute: true"},{"line_number":296,"context_line":"        devstack_localrc:"},{"line_number":297,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":298,"context_line":"        devstack_local_conf:"},{"line_number":299,"context_line":"          post-config:"},{"line_number":300,"context_line":"            $WATCHER_CONF:"},{"line_number":301,"context_line":"              watcher_cluster_data_model_collectors.compute:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"f2f1e7c8_f82bd073","line":298,"updated":"2024-11-27 07:44:05.000000000","message":"https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9ad/935371/14/check/watcher-prometheus-integration/9ad4b13/job-output.txt\n\n2024-11-26 20:20:27.695432 | compute1 | /opt/stack/ceilometer/devstack/settings: line 14: CEILOMETER_BACKEND: \"\": syntax error: operand expected (error token is \"\"\"\")\n\n\n\nthe last run fails on the same issue @rlandy@redhat.com you can try:\n\n```\n          CEILOMETER_BACKEND: \"none\"\n          CEILOMETER_BACKENDS: \"none\"\n```\n\n(as a string/quoted as above).","commit_id":"18d2603dd0afcae5ea4c16bbd1a9d1739ede8435"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"7f2ef2c3eadcbd7635a3b49f90d31bb4939890b3","unresolved":false,"context_lines":[{"line_number":295,"context_line":"          ceilometer-acompute: true"},{"line_number":296,"context_line":"        devstack_localrc:"},{"line_number":297,"context_line":"          CEILOMETER_BACKEND: none"},{"line_number":298,"context_line":"        devstack_local_conf:"},{"line_number":299,"context_line":"          post-config:"},{"line_number":300,"context_line":"            $WATCHER_CONF:"},{"line_number":301,"context_line":"              watcher_cluster_data_model_collectors.compute:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"72b0f2f3_30d5da92","line":298,"in_reply_to":"f2f1e7c8_f82bd073","updated":"2024-11-27 11:32:32.000000000","message":"Acknowledged","commit_id":"18d2603dd0afcae5ea4c16bbd1a9d1739ede8435"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b5e14f1474be00e90819c79ad173fa06dd2901cd","unresolved":true,"context_lines":[{"line_number":241,"context_line":"        # hybrid plug we do not need to test cinder and we want to ensure"},{"line_number":242,"context_line":"        # that we have at least one job without cinder in our gate so we"},{"line_number":243,"context_line":"        # disable it in this job"},{"line_number":244,"context_line":"        # Cinder services"},{"line_number":245,"context_line":"        c-api: false"},{"line_number":246,"context_line":"        c-bak: false"},{"line_number":247,"context_line":"        c-sch: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"4eeb9f5c_3c706325","line":244,"updated":"2024-11-28 07:50:49.000000000","message":"for some reason though cinder is trying to start and failing, specifically db setup\n\n```\n2024-11-27 22:40:58.948731 | compute1 | + lib/cinder:init_cinder:490               :   /opt/stack/data/venv/bin/cinder-manage --config-file /etc/cinder/cinder.conf db sync\n2024-11-27 22:40:58.948756 | compute1 | INFO dbcounter [\u001b[00;36m-] \u001b[01;35mRegistered counter for database cinder\u001b[00m\n2024-11-27 22:40:58.948845 | compute1 | CRITICAL cinder [\u001b[00;36m-] \u001b[01;35mUnhandled error\u001b[00m: oslo_db.exception.DBNonExistentDatabase: (pymysql.err.OperationalError) (1049, \"Unknown database \u0027cinder\u0027\")\n```\n\nand then unstack/clean runs shortly after that so i **think** this is the fatal error in https://655f829b5314efd0a430-298f4365cc798f0001a632f171eb41d6.ssl.cf1.rackcdn.com/935371/16/check/watcher-prometheus-integration/9d863ad/job-output.txt","commit_id":"109aac76e3f7019be9304b873af33d28cc960e35"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e4537deb6766b45b63fef21a0bed003277d7f3f0","unresolved":true,"context_lines":[{"line_number":241,"context_line":"        # hybrid plug we do not need to test cinder and we want to ensure"},{"line_number":242,"context_line":"        # that we have at least one job without cinder in our gate so we"},{"line_number":243,"context_line":"        # disable it in this job"},{"line_number":244,"context_line":"        # Cinder services"},{"line_number":245,"context_line":"        c-api: false"},{"line_number":246,"context_line":"        c-bak: false"},{"line_number":247,"context_line":"        c-sch: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"83dab659_d9d51f57","line":244,"in_reply_to":"4eeb9f5c_3c706325","updated":"2024-11-28 10:59:35.000000000","message":"that is failing on the compute not the contoler.\n\nwe need to sepreatly disable cnder on the subnode\nas cidner volume is enabled there currently.\n\n\nwe could just remove this and add cidner to the job, i recommended we turn it off before i found out that the workload balancing stagey has the ability to blance cinder storage pools\n\nso we will proably want to enabel it so that we can fix \n\nhttps://bugs.launchpad.net/watcher/+bug/2088118\n\nalthough we can do that in the review that actull adresses the bug.\n\nbut ya the issue is its enabled on the compute wihtout the contoler services being enabled.","commit_id":"109aac76e3f7019be9304b873af33d28cc960e35"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7ef7b94c7b9877089a1ca91f5b168c4b308888f6","unresolved":false,"context_lines":[{"line_number":241,"context_line":"        # hybrid plug we do not need to test cinder and we want to ensure"},{"line_number":242,"context_line":"        # that we have at least one job without cinder in our gate so we"},{"line_number":243,"context_line":"        # disable it in this job"},{"line_number":244,"context_line":"        # Cinder services"},{"line_number":245,"context_line":"        c-api: false"},{"line_number":246,"context_line":"        c-bak: false"},{"line_number":247,"context_line":"        c-sch: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"2b063382_c84824bf","line":244,"in_reply_to":"83dab659_d9d51f57","updated":"2024-12-09 13:43:06.000000000","message":"Done","commit_id":"109aac76e3f7019be9304b873af33d28cc960e35"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e4537deb6766b45b63fef21a0bed003277d7f3f0","unresolved":true,"context_lines":[{"line_number":291,"context_line":"    group-vars:"},{"line_number":292,"context_line":"      subnode:"},{"line_number":293,"context_line":"        devstack_services:"},{"line_number":294,"context_line":"          ceilometer-acompute: true"},{"line_number":295,"context_line":"        devstack_localrc:"},{"line_number":296,"context_line":"          CEILOMETER_BACKEND: \"none\""},{"line_number":297,"context_line":"          CEILOMETER_BACKENDS: \"none\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"92894691_7aa00010","line":294,"updated":"2024-11-28 10:59:35.000000000","message":"so we would disable c-vol and c-back here too.","commit_id":"109aac76e3f7019be9304b873af33d28cc960e35"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"e0ada1505049dbef78bf80904e87b4b9da9f111d","unresolved":false,"context_lines":[{"line_number":291,"context_line":"    group-vars:"},{"line_number":292,"context_line":"      subnode:"},{"line_number":293,"context_line":"        devstack_services:"},{"line_number":294,"context_line":"          ceilometer-acompute: true"},{"line_number":295,"context_line":"        devstack_localrc:"},{"line_number":296,"context_line":"          CEILOMETER_BACKEND: \"none\""},{"line_number":297,"context_line":"          CEILOMETER_BACKENDS: \"none\""}],"source_content_type":"text/x-yaml","patch_set":16,"id":"bd0a88fb_64f44712","line":294,"in_reply_to":"92894691_7aa00010","updated":"2024-12-08 22:56:34.000000000","message":"Done","commit_id":"109aac76e3f7019be9304b873af33d28cc960e35"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c6a29e56ca1dfc8bb8ad1f6c4f40bbb6d4812bc7","unresolved":true,"context_lines":[{"line_number":234,"context_line":"      - openstack/aodh"},{"line_number":235,"context_line":"      - openstack/ceilometer"},{"line_number":236,"context_line":"      - openstack/tempest"},{"line_number":237,"context_line":"      - infrawatch/sg-core"},{"line_number":238,"context_line":"      - openstack/watcher"},{"line_number":239,"context_line":"      - openstack/python-watcherclient"},{"line_number":240,"context_line":"      - openstack/watcher-tempest-plugin"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"1e0e89ff_7cc1bb62","line":237,"range":{"start_line":237,"start_character":0,"end_line":237,"end_character":2},"updated":"2025-01-09 22:49:57.000000000","message":"if we can update this to use the other sg-core repo i think we can merge this and follow up with other patches to adress the other comments.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3129ce3f526c28ec00530fc7bd81d96993dead0","unresolved":false,"context_lines":[{"line_number":234,"context_line":"      - openstack/aodh"},{"line_number":235,"context_line":"      - openstack/ceilometer"},{"line_number":236,"context_line":"      - openstack/tempest"},{"line_number":237,"context_line":"      - infrawatch/sg-core"},{"line_number":238,"context_line":"      - openstack/watcher"},{"line_number":239,"context_line":"      - openstack/python-watcherclient"},{"line_number":240,"context_line":"      - openstack/watcher-tempest-plugin"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"dd366ccd_3793ea91","line":237,"range":{"start_line":237,"start_character":0,"end_line":237,"end_character":2},"in_reply_to":"1e0e89ff_7cc1bb62","updated":"2025-01-10 13:49:54.000000000","message":"Done","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"eb5ff624e8da92f13fd4952f2fc3f5d3c990607f","unresolved":true,"context_lines":[{"line_number":257,"context_line":"        s-object: false"},{"line_number":258,"context_line":"        s-proxy: false"},{"line_number":259,"context_line":"      devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"3d1e62ce_3a7be8fa","line":260,"updated":"2024-12-09 11:38:29.000000000","message":"I think we can drop TEMPEST_PLUGINS from here and use `tempest_plugins` var provided by [write-devstack-local-conf](https://docs.openstack.org/devstack/latest/zuul_roles.html#rolevar-write-devstack-local-conf.tempest_plugins) role like this.\n```\n      tempest_plugins:\n        - watcher-tempest-plugin\n```\n[The same](https://opendev.org/openstack/watcher/src/branch/master/.zuul.yaml#L168) is used in other jobs also.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2abc2eb9693ea65772c7e9677855ea118ac5182f","unresolved":false,"context_lines":[{"line_number":257,"context_line":"        s-object: false"},{"line_number":258,"context_line":"        s-proxy: false"},{"line_number":259,"context_line":"      devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"dce4d45f_ae04997c","line":260,"in_reply_to":"3d1e62ce_3a7be8fa","updated":"2024-12-09 12:56:14.000000000","message":"Sorry already added at line 290. this flag TEMPEST_PLUGINS is needed. It is used in all the repos. https://codesearch.openstack.org/?q\u003dTEMPEST_PLUGINS\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7ef7b94c7b9877089a1ca91f5b168c4b308888f6","unresolved":true,"context_lines":[{"line_number":257,"context_line":"        s-object: false"},{"line_number":258,"context_line":"        s-proxy: false"},{"line_number":259,"context_line":"      devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"b3e81498_3d5e7af2","line":260,"in_reply_to":"3d1e62ce_3a7be8fa","updated":"2024-12-09 13:43:06.000000000","message":"let defer that to a followup.\n\nyou may be correct that we can remove this and use \n```\ntempest_plugins:\n        - watcher-tempest-plugin\n```\ninsetad.\n\nwe already have that directive so its trvial to delete this line test that in a separate follow up patch that just does that change.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3129ce3f526c28ec00530fc7bd81d96993dead0","unresolved":false,"context_lines":[{"line_number":257,"context_line":"        s-object: false"},{"line_number":258,"context_line":"        s-proxy: false"},{"line_number":259,"context_line":"      devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"8b566cc5_fc60eb19","line":260,"in_reply_to":"77b5bc2b_3db48b60","updated":"2025-01-10 13:49:54.000000000","message":"Done","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"44f1504b0b4afeaa9f95b1d3f7634b5e9c1e4c8a","unresolved":true,"context_lines":[{"line_number":257,"context_line":"        s-object: false"},{"line_number":258,"context_line":"        s-proxy: false"},{"line_number":259,"context_line":"      devstack_localrc:"},{"line_number":260,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/watcher-tempest-plugin\"\u0027"},{"line_number":261,"context_line":"        CEILOMETER_BACKENDS: \"sg-core\""},{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"77b5bc2b_3db48b60","line":260,"in_reply_to":"b3e81498_3d5e7af2","updated":"2024-12-10 14:42:46.000000000","message":"can test that in one of the follow up jobs.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"b5ec2b12a599f0c904061434eccb97378711aeee","unresolved":true,"context_lines":[{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":264,"context_line":"      devstack_local_conf:"},{"line_number":265,"context_line":"        post-config:"},{"line_number":266,"context_line":"          $WATCHER_CONF:"},{"line_number":267,"context_line":"            watcher_cluster_data_model_collectors.compute:"},{"line_number":268,"context_line":"              period: 120"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"da257b64_d4d81ff2","line":265,"range":{"start_line":265,"start_character":0,"end_line":265,"end_character":2},"updated":"2024-12-12 12:21:11.000000000","message":"At some point (once we have an initial version of the datasource merged), I\u0027d say we should set also: \n\n```\nwatcher_datasources:\n  datasources: prometheus\n```\n\nTo make sure the prometheus datasource is the only used one?","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3129ce3f526c28ec00530fc7bd81d96993dead0","unresolved":true,"context_lines":[{"line_number":262,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":263,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":264,"context_line":"      devstack_local_conf:"},{"line_number":265,"context_line":"        post-config:"},{"line_number":266,"context_line":"          $WATCHER_CONF:"},{"line_number":267,"context_line":"            watcher_cluster_data_model_collectors.compute:"},{"line_number":268,"context_line":"              period: 120"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"7ea129a4_9e2ce526","line":265,"range":{"start_line":265,"start_character":0,"end_line":265,"end_character":2},"in_reply_to":"da257b64_d4d81ff2","updated":"2025-01-10 13:49:54.000000000","message":"this can be changed when both are merged we wanted to merge the job without depending on the data souce","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1163721d7c2952582c6b2df225afe4018fe093dd","unresolved":true,"context_lines":[{"line_number":292,"context_line":"      # TODO (rlandy): enable all scenario tests"},{"line_number":293,"context_line":"      # when tests can be run with prometheus data source."},{"line_number":294,"context_line":"      # https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2090853"},{"line_number":295,"context_line":"      tempest_test_regex: ^watcher_tempest_plugin.tests\\.(api|client_functional|scenario.test_execute_dummy_optim)"},{"line_number":296,"context_line":"      tempest_concurrency: 1"},{"line_number":297,"context_line":"      tox_envlist: all"},{"line_number":298,"context_line":"    group-vars:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"3d19e2a1_42494a8e","line":295,"updated":"2024-12-11 04:45:21.000000000","message":"I found this nice example of making tempest_test_regex more readable: https://opendev.org/x/whitebox-neutron-tempest-plugin/src/branch/master/zuul.d/master_jobs.yaml#L442\n\nWe can adopt the same as a followup patch if possible:\n```\ntempest_test_regex: \"\\\n  (^watcher_tempest_plugin.tests.api)|\\\n  (^watcher_tempest_plugin.tests.client_functional)|\\\n  (^watcher_tempest_plugin.tests.scenario.test_execute_dummy_optim)\"\n```","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0941d5fe5dcd3921c311bf8572af6a127bfff186","unresolved":true,"context_lines":[{"line_number":292,"context_line":"      # TODO (rlandy): enable all scenario tests"},{"line_number":293,"context_line":"      # when tests can be run with prometheus data source."},{"line_number":294,"context_line":"      # https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2090853"},{"line_number":295,"context_line":"      tempest_test_regex: ^watcher_tempest_plugin.tests\\.(api|client_functional|scenario.test_execute_dummy_optim)"},{"line_number":296,"context_line":"      tempest_concurrency: 1"},{"line_number":297,"context_line":"      tox_envlist: all"},{"line_number":298,"context_line":"    group-vars:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"94c8bd76_0bfb20d6","line":295,"in_reply_to":"3d19e2a1_42494a8e","updated":"2024-12-18 11:41:37.000000000","message":"done in follow up","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"412529f9b806cc39ff716f425fd9622d23ee952b","unresolved":true,"context_lines":[{"line_number":297,"context_line":"      tox_envlist: all"},{"line_number":298,"context_line":"    group-vars:"},{"line_number":299,"context_line":"      subnode:"},{"line_number":300,"context_line":"        devstack_plugins:"},{"line_number":301,"context_line":"          ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":302,"context_line":"        devstack_services:"},{"line_number":303,"context_line":"          ceilometer-acompute: true"},{"line_number":304,"context_line":"        devstack_localrc:"},{"line_number":305,"context_line":"          CEILOMETER_BACKEND: \"none\""},{"line_number":306,"context_line":"          CEILOMETER_BACKENDS: \"none\""},{"line_number":307,"context_line":"        devstack_local_conf:"},{"line_number":308,"context_line":"          post-config:"},{"line_number":309,"context_line":"            $WATCHER_CONF:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"ae35b729_afa6d6b8","line":306,"range":{"start_line":300,"start_character":8,"end_line":306,"end_character":37},"updated":"2024-12-17 06:32:45.000000000","message":"https://review.opendev.org/c/openstack/watcher/+/937625 drops ceilometer support. We need to keep an eye otherwise it will break this job.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"7bb362b745b50511a0d9628fd67db654e52f817d","unresolved":true,"context_lines":[{"line_number":297,"context_line":"      tox_envlist: all"},{"line_number":298,"context_line":"    group-vars:"},{"line_number":299,"context_line":"      subnode:"},{"line_number":300,"context_line":"        devstack_plugins:"},{"line_number":301,"context_line":"          ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":302,"context_line":"        devstack_services:"},{"line_number":303,"context_line":"          ceilometer-acompute: true"},{"line_number":304,"context_line":"        devstack_localrc:"},{"line_number":305,"context_line":"          CEILOMETER_BACKEND: \"none\""},{"line_number":306,"context_line":"          CEILOMETER_BACKENDS: \"none\""},{"line_number":307,"context_line":"        devstack_local_conf:"},{"line_number":308,"context_line":"          post-config:"},{"line_number":309,"context_line":"            $WATCHER_CONF:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"20f02a8f_0f4a7428","line":306,"range":{"start_line":300,"start_character":8,"end_line":306,"end_character":37},"in_reply_to":"3b2a13e4_c36df506","updated":"2024-12-17 11:48:17.000000000","message":"I also believe that removing the datasource will not affect the job. Gnocci datasource also needs ceilometer running, so we will continue to have jobs deploying ceilometer too.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c907b6688f19ac39d192864ed1ee8a0bf0fd55e7","unresolved":true,"context_lines":[{"line_number":297,"context_line":"      tox_envlist: all"},{"line_number":298,"context_line":"    group-vars:"},{"line_number":299,"context_line":"      subnode:"},{"line_number":300,"context_line":"        devstack_plugins:"},{"line_number":301,"context_line":"          ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":302,"context_line":"        devstack_services:"},{"line_number":303,"context_line":"          ceilometer-acompute: true"},{"line_number":304,"context_line":"        devstack_localrc:"},{"line_number":305,"context_line":"          CEILOMETER_BACKEND: \"none\""},{"line_number":306,"context_line":"          CEILOMETER_BACKENDS: \"none\""},{"line_number":307,"context_line":"        devstack_local_conf:"},{"line_number":308,"context_line":"          post-config:"},{"line_number":309,"context_line":"            $WATCHER_CONF:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"3b2a13e4_c36df506","line":306,"range":{"start_line":300,"start_character":8,"end_line":306,"end_character":37},"in_reply_to":"ae35b729_afa6d6b8","updated":"2024-12-17 07:51:27.000000000","message":"hmmm this is a good point, but maybe? I mean, 937625 removes the ceilometer datasource for watcher. it should not affect any ceilometer service things we setup for this job (like the plugins and sg-core backend etc). \n\nalso i believe this job is not actually configuring or using any datasources that will be wired up in following patches, so ... maybe? ;) I think it should be OK... we could rebase onto 937625 if we wanted to check","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3129ce3f526c28ec00530fc7bd81d96993dead0","unresolved":true,"context_lines":[{"line_number":275,"context_line":"              datasources: prometheus"},{"line_number":276,"context_line":"            prometheus_client:"},{"line_number":277,"context_line":"              prometheus_host_port: \"localhost:3000\""},{"line_number":278,"context_line":"              prometheus_fqdn_label: fqdn"},{"line_number":279,"context_line":"        test-config:"},{"line_number":280,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":281,"context_line":"            compute:"}],"source_content_type":"text/x-yaml","patch_set":32,"id":"050c7ad0_2acbef03","line":278,"updated":"2025-01-10 13:49:54.000000000","message":"this is not in sync with the lastest version fo the datasource patch.","commit_id":"a8e6a111998f1a69639599860c0b9b706b338d3a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"849f7b9a1a6dc516fbfe5d04ac9d0011a1c6a7bd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4c65630b_986fca48","updated":"2024-11-15 17:08:47.000000000","message":"thats a good start several comments inline but mostly disabling services that watcher does not interact with to get an inial base jobs that we can then extend as needed.","commit_id":"56574d3a0522482cc846fbd7e7723e7e8592f011"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"fe849bacc0465043d4ddbd0630431490dabdd377","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fcfad5be_372c9cbd","updated":"2024-11-15 20:17:36.000000000","message":"Looks like Gnocchi is enabled:\nhttps://github.com/openstack/watcher/blob/master/devstack/local.conf.controller#L44\n\n# Enable the Gnocchi plugin\nenable_plugin gnocchi https://github.com/gnocchixyz/gnocchi\n\n(1049, \"Unknown database \u0027gnocchi\u0027\")\n(Background on this error at: https://sqlalche.me/e/20/e3q8)","commit_id":"291e2cb67b1405eb5908c2a6f63696564bc99cb4"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"2ce8315b742eccb20c00f0a8b87e1bfdae87609c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6e0d18e8_dc5db20c","updated":"2024-11-17 23:07:53.000000000","message":"https://github.com/openstack/watcher/blob/master/devstack/local.conf.controller#L44 enables gnocchi - error is looking for gnocchi","commit_id":"dad8a213a3173a48e3a54c79994d8ea3940440a3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2bb9766a5d313a3ebb6154ff3e054a26a7640323","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"20202def_cd17e69c","updated":"2024-11-26 08:00:44.000000000","message":"for the watcher-prometheus-integration there are a few things going on (inline comment about the None and the aodh-api is still failing to start but i also noticed an issue with the sql password there https://9a8601bd1ae2889ac6d3-dc95df9dc6d9c4be8998bc6e49021df1.ssl.cf2.rackcdn.com/935371/13/check/watcher-prometheus-integration/2655c61/job-output.txt\n \n\n```\n2024-11-25 19:26:34.470733 | compute1 | + lib/databases/mysql:configure_database_mysql:111 :   sudo mysqladmin -u root password secretdatabase\n2024-11-25 19:26:34.489340 | compute1 | \u0007mysqladmin: connect to server at \u0027localhost\u0027 failed\n2024-11-25 19:26:34.489384 | compute1 | error: \u0027Access denied for user \u0027root\u0027@\u0027localhost\u0027 (using password: NO)\u0027\n\n```\n\n\nFor the other two jobs openstack-tox-cover and tox-py39 they are failing because of my patch. So you can ignore those ones I will deal with them in the parent patch.","commit_id":"c3a19992a6d993c0ce7de2b57cacb337e38d458a"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"110d36bbc340fe0f69ebe52e2434db0d72b728bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"40339d12_ba84245f","updated":"2024-11-27 11:56:10.000000000","message":"recheck","commit_id":"91352e49bbce24670124f08fd8898ca244b69504"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"2e44fd23f3dec644c6024fec9c797b3e7220bbf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"346a9b83_d63fc401","updated":"2024-12-03 20:25:15.000000000","message":"recheck","commit_id":"7c452372a7b216286be3c1ddb9341bc5080cb628"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"4896c49ff5c28cca680ecec5f34a9bba598903ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"7de3cd86_ad9f0c3f","updated":"2024-12-03 19:22:09.000000000","message":"recheck","commit_id":"7c452372a7b216286be3c1ddb9341bc5080cb628"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9ded546c1b15f6ace34789ca76e5a070015d8ec9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"bcf7d5b3_d4744784","updated":"2024-12-04 00:55:39.000000000","message":"i am pretty confused\n\n\nboth the passing stragey job and the failing new one both seam to be trigging\n\nDec 03 22:44:17.291817 np0039246418 watcher-applier[92709]:   File \"/opt/stack/data/venv/lib/python3.12/site-packages/oslo_db/sqlalchemy/engines.py\", line 52, in _thread_yield\nDec 03 22:44:17.291817 np0039246418 watcher-applier[92709]:     time.sleep(0)\nDec 03 22:44:17.291817 np0039246418 watcher-applier[92709]:   File \"/opt/stack/data/venv/lib/python3.12/site-packages/eventlet/greenthread.py\", line 35, in sleep\nDec 03 22:44:17.291817 np0039246418 watcher-applier[92709]:     raise RuntimeError(\u0027do not call blocking functions from the mainloop\u0027)\nDec 03 22:44:17.291817 np0039246418 watcher-applier[92709]: RuntimeError: do not call blocking functions from the mainloop\n\n\n\none of the jobs passed\nthe other failed.\n\nbut  but run the same code and both shoudl have my fix patch inclcuded...\n\nso we still have that failure but apprently its not always fatal?","commit_id":"2234377238fe666315c39e39f5d3bd80479dbb42"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"f4595a2a03c89e7f7a759d4b55b0ff54584de69a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"799c10e1_49ffdaa3","updated":"2024-12-04 09:10:23.000000000","message":"recheck","commit_id":"2234377238fe666315c39e39f5d3bd80479dbb42"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"26a83ff494d2ff4fe9b1016ad7be306732d44282","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"9fd8fab1_aa5b1227","in_reply_to":"bcf7d5b3_d4744784","updated":"2024-12-04 00:56:48.000000000","message":"https://zuul.opendev.org/t/openstack/build/f87cf4d825ad4f318a6511b389f3561e/log/controller/logs/screen-watcher-applier.txt#1140","commit_id":"2234377238fe666315c39e39f5d3bd80479dbb42"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0ad4e374e9a36dc601220b997383b5400601d98d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"7fb9cd15_7f8d1e68","updated":"2024-12-04 12:55:33.000000000","message":"recheck","commit_id":"78a8afbabc6d7de08b8f5057952e7b41cfbf7aad"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9b9be69a09ee2e21f4d333419254efb2825e24f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"6ac18cfe_f1deaf9c","updated":"2024-12-13 15:46:19.000000000","message":"+1 on getting this job up and running for future integration with prometheus datasource. Tks","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"eb5ff624e8da92f13fd4952f2fc3f5d3c990607f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"c3718bcb_be798f47","updated":"2024-12-09 11:38:29.000000000","message":"Great job Ronelle on adding this job and making it passing. Left a small improvement, rest looks good.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"b5ec2b12a599f0c904061434eccb97378711aeee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"f4a88539_41a34b04","updated":"2024-12-12 12:21:11.000000000","message":"Just a reminder for future change needed to enable prometheus datasource","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1163721d7c2952582c6b2df225afe4018fe093dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"ba7d4d2e_2662f136","updated":"2024-12-11 04:45:21.000000000","message":"Tempest tests are passing: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9c8/935371/29/check/watcher-prometheus-integration/9c888f4/testr_results.html","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"2eef1e02b3d8772ac77e403c9fb8000395d19033","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"c28f46bc_a55072f6","updated":"2024-12-11 07:42:45.000000000","message":"forgot to vote ;)","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7ef7b94c7b9877089a1ca91f5b168c4b308888f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"4b49682d_c12acc3b","updated":"2024-12-09 13:43:06.000000000","message":"im ok with moving forward with this as an inital version and following up to extend it.\n\nif you want to respin for chandans suggestion i can re review otherwise we can include that as part fo the other follwoups.","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"3492d650997c1ba09f2305156e675df45495d3ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"5ea346f4_2a447c85","updated":"2024-12-11 09:53:16.000000000","message":"lgtm, the job is passing, it\u0027s a solid foundation","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"c0560291f79e08c5e491ba9dfe81bb1c66c4022e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"8e642402_16e1a13f","updated":"2024-12-08 22:59:03.000000000","message":"recheck","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e6316b9308a45b8060f21929b94c237ddb5be895","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"cdf3aab4_854ecb67","updated":"2024-12-11 07:42:26.000000000","message":"very nice @rlandy@redhat.com and @smooney@redhat.com ++\n\ndoes anyone know if we collect/where the sg-core logs are? I could find \u0027sg-core\u0027 and \u0027prometheus\u0027 mentioned in services https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9c8/935371/29/check/watcher-prometheus-integration/9c888f4/controller/logs/services.txt  \n\nbut i expected to find a top-level screen-sg-core like we do for other plugins/services in /logs","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9b9be69a09ee2e21f4d333419254efb2825e24f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"41d21922_c57ec4d7","in_reply_to":"cdf3aab4_854ecb67","updated":"2024-12-13 15:46:19.000000000","message":"I wasn\u0027t able to find it too\nI saw sg-core service references, and the prometheus container being pulled in devstack logs","commit_id":"67584db193ce0a2d3587f7ab77ddd658c464643f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"290a02734df681e2a5a8ae53e768a5b18f53ce9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":31,"id":"abab9d46_5151a3bb","updated":"2025-01-10 13:43:33.000000000","message":"ill check back after the ci reports but we can proceed with this test only change provide this works as expected. thanks for updating to the new repo location,","commit_id":"2668cdceaee98494b77f83eb0f9bef5427f3f9d2"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7878388ab833b0daffc6241a8be65c357a6e9ac6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"f390c9b3_ffd5c59b","updated":"2025-01-10 15:12:47.000000000","message":"recheck the watcher-tempest-strategies failure is not related.\nit hit https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2090854\n\n025-01-10 14:42:29,563 101002 INFO     [tempest.lib.common.rest_client] Request (TestExecuteStrategies:test_execute_zone_migration_live_migration_strategy): 409 POST http://200.225.47.23/metric/v1/resource/instance 0.021s\n2025-01-10 14:42:29,563 101002 DEBUG    [tempest.lib.common.rest_client] Request - Headers: {\u0027Content-Type\u0027: \u0027application/json\u0027, \u0027Accept\u0027: \u0027application/json\u0027, \u0027X-Auth-Token\u0027: \u0027\u003comitted\u003e\u0027}\n        Body: {\"metrics\": {\"cpu_util\": {\"archive_policy_name\": \"low\"}}, \"host\": \"np0039534426\", \"display_name\": \"instance-0000000d\", \"image_ref\": \"d72bb1d2-f1a3-435a-985d-05a4b91a4b59\", \"flavor_id\": \"42\", \"flavor_name\": \"m1.nano\", \"id\": \"b5ce9dee-959d-4301-b6ce-53c1b311218f\"}\n    Response - Headers: {\u0027date\u0027: \u0027Fri, 10 Jan 2025 14:42:29 GMT\u0027, \u0027server\u0027: \u0027Apache/2.4.58 (Ubuntu)\u0027, \u0027content-length\u0027: \u0027113\u0027, \u0027content-type\u0027: \u0027application/json\u0027, \u0027connection\u0027: \u0027close\u0027, \u0027status\u0027: \u0027409\u0027, \u0027content-location\u0027: \u0027http://200.225.47.23/metric/v1/resource/instance\u0027}\n        Body: b\u0027{\"code\": 409, \"title\": \"Conflict\", \"description\": \"Resource b5ce9dee-959d-4301-b6ce-53c1b311218f already exists\"}\u0027","commit_id":"56b8c1211af05aad796daab1e8b79ca25b0b3c3c"}]}