)]}'
{"doc/source/datasources/migrate-prometheus-to-aetos.rst":[{"robot_id":"zuul","robot_run_id":"80f5492183744bbfb1166bea401716db","url":"https://zuul.teim.app/t/main/buildset/80f5492183744bbfb1166bea401716db","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"30fc556a1481f91954bcf9d345e3494dc8772491","patch_set":5,"id":"f3f96215_b2e590ae","line":51,"updated":"2026-02-26 12:17:26.000000000","message":"Minor typo in migration guide documentation: \u0027This is step is not strictly required\u0027 should be \u0027This step is not strictly required\u0027\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: Minor documentation typo that does not affect functionality but should be corrected for professional quality\n\n**Suggestion**:\nChange \u0027This is step is not strictly required\u0027 to \u0027This step is not strictly required\u0027 on line 51 of the migration guide","commit_id":"7c0a1f84eacfe55be74931783c2b54e6f8fe2208"}],"releasenotes/notes/deprecate-prometheus-datasource-f8a3c2e1d9b4f5a7.yaml":[{"robot_id":"zuul","robot_run_id":"9fd16bc378194d76bf0c09870a9f7dbf","url":"https://zuul.teim.app/t/main/buildset/9fd16bc378194d76bf0c09870a9f7dbf","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"8018842bab3b47dddd30f69b24ae6485b210c4d6","patch_set":2,"id":"af53c53a_2a3ffc41","line":39,"updated":"2025-11-11 13:47:58.000000000","message":"Missing newline at end of release notes file - (Severity: critical, Confidence: 1.0) | Risk: Style/Quality | Priority: Fix Now | Why This Matters: Git diff shows trailing whitespace issue that violates formatting standards | Recommendation: Add a newline at the end of the YAML file","commit_id":"583c6b9689dcd5f0b3c514db83dfb7b5c0df67bc"}],"watcher/conf/datasources.py":[{"robot_id":"zuul","robot_run_id":"dcbce64bc7f447279b74b6b783029dd7","url":"https://zuul.teim.app/t/main/buildset/dcbce64bc7f447279b74b6b783029dd7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"5f46681e2146d505b944e725a54e58304e2c2b83","patch_set":3,"id":"45e7285e_7edf5f35","line":29,"updated":"2025-11-11 15:38:40.000000000","message":"Line length violation in datasources.py comment - (Severity: warning, Confidence: 0.9)","commit_id":"e46a45e2506ee8bf2934af66469dc4a9ff31e4c7"},{"robot_id":"zuul","robot_run_id":"9042ac22682d45e493c6b2b1815e6378","url":"https://zuul.teim.app/t/main/buildset/9042ac22682d45e493c6b2b1815e6378","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"75d60e71975784f047846e062ba0b37dd300ea44","patch_set":4,"id":"e37fd324_9b759914","line":19,"updated":"2025-11-11 16:03:07.000000000","message":"Import order inconsistency in datasources.py - (Severity: suggestion, Confidence: 0.8) | Recommendation: Move prometheus import before manager import to maintain stdlib → third-party → local project order","commit_id":"2de20c8a221accf5894b11150563fea5639bd9f4"},{"robot_id":"zuul","robot_run_id":"9042ac22682d45e493c6b2b1815e6378","url":"https://zuul.teim.app/t/main/buildset/9042ac22682d45e493c6b2b1815e6378","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"75d60e71975784f047846e062ba0b37dd300ea44","patch_set":4,"id":"cec8d136_5012c0b6","line":34,"updated":"2025-11-11 16:03:07.000000000","message":"Line 34 in watcher/conf/datasources.py - Potential exception risk when prometheus is not in default datasources - (Severity: warning, Confidence: 0.6)","commit_id":"2de20c8a221accf5894b11150563fea5639bd9f4"},{"robot_id":"zuul","robot_run_id":"d66abc595bb14284a0ebf5413eccc03b","url":"https://zuul.teim.app/t/main/buildset/d66abc595bb14284a0ebf5413eccc03b","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"16ebec9f07721edea831e458cc168cce67fc152a","patch_set":6,"id":"492e269f_e41356d3","line":20,"updated":"2026-02-26 14:50:50.000000000","message":"The import reordering in datasources.py could be simplified by keeping both imports together\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Minor code readability improvement. The import order change from aetos to prometheus is functional but could be more intuitive with a comment explaining the change.\n\n**Recommendation**:\nConsider adding a comment explaining why prometheus import is needed for the default_datasources removal logic, or keep both imports together with an ordering comment.","commit_id":"9132e604b6ef0986fc8f217b9a6c0a21a6f5e6a1"}],"watcher/conf/prometheus_client.py":[{"robot_id":"zuul","robot_run_id":"9fd16bc378194d76bf0c09870a9f7dbf","url":"https://zuul.teim.app/t/main/buildset/9fd16bc378194d76bf0c09870a9f7dbf","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"8018842bab3b47dddd30f69b24ae6485b210c4d6","patch_set":2,"id":"14773db5_12651d7f","line":21,"updated":"2025-11-11 13:47:58.000000000","message":"Long line in prometheus_client.py exceeds ruff line-length - (Severity: warning, Confidence: 0.8)","commit_id":"583c6b9689dcd5f0b3c514db83dfb7b5c0df67bc"},{"robot_id":"zuul","robot_run_id":"9fd16bc378194d76bf0c09870a9f7dbf","url":"https://zuul.teim.app/t/main/buildset/9fd16bc378194d76bf0c09870a9f7dbf","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"8018842bab3b47dddd30f69b24ae6485b210c4d6","patch_set":2,"id":"0719e83f_501feab3","line":33,"updated":"2025-11-11 13:47:58.000000000","message":"Configuration help strings contain redundant deprecation notices - (Severity: warning, Confidence: 0.7)","commit_id":"583c6b9689dcd5f0b3c514db83dfb7b5c0df67bc"},{"robot_id":"zuul","robot_run_id":"dcbce64bc7f447279b74b6b783029dd7","url":"https://zuul.teim.app/t/main/buildset/dcbce64bc7f447279b74b6b783029dd7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"5f46681e2146d505b944e725a54e58304e2c2b83","patch_set":3,"id":"bfe32192_defd558d","line":22,"updated":"2025-11-11 15:38:40.000000000","message":"URL in prometheus_client.py help text may wrap - (Severity: warning, Confidence: 0.8)","commit_id":"e46a45e2506ee8bf2934af66469dc4a9ff31e4c7"},{"robot_id":"zuul","robot_run_id":"d66abc595bb14284a0ebf5413eccc03b","url":"https://zuul.teim.app/t/main/buildset/d66abc595bb14284a0ebf5413eccc03b","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"16ebec9f07721edea831e458cc168cce67fc152a","patch_set":6,"id":"d8312f98_5ae24a80","line":30,"updated":"2026-02-26 14:50:50.000000000","message":"The deprecated_reason strings in prometheus_client.py are concise but could reference the migration guide URL\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Improved user experience by directly linking to migration documentation in deprecation warnings.\n\n**Recommendation**:\nConsider including the migration guide URL (https://docs.openstack.org/watcher/latest/datasources/migrate-prometheus-to-aetos.html) in the deprecated_reason strings for better user guidance.","commit_id":"9132e604b6ef0986fc8f217b9a6c0a21a6f5e6a1"}],"watcher/decision_engine/datasources/manager.py":[{"robot_id":"zuul","robot_run_id":"dcbce64bc7f447279b74b6b783029dd7","url":"https://zuul.teim.app/t/main/buildset/dcbce64bc7f447279b74b6b783029dd7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"5f46681e2146d505b944e725a54e58304e2c2b83","patch_set":3,"id":"56aca8a2_eafebfda","line":72,"updated":"2025-11-11 15:38:40.000000000","message":"Consider adding version number to deprecation warnings - (Severity: suggestion, Confidence: 0.7) | Recommendation: Include version information in warning messages to match the deprecation implementation (removed in 2027.1)","commit_id":"e46a45e2506ee8bf2934af66469dc4a9ff31e4c7"},{"robot_id":"zuul","robot_run_id":"dcbce64bc7f447279b74b6b783029dd7","url":"https://zuul.teim.app/t/main/buildset/dcbce64bc7f447279b74b6b783029dd7","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"5f46681e2146d505b944e725a54e58304e2c2b83","patch_set":3,"id":"f8a1be06_9440ae12","line":74,"updated":"2025-11-11 15:38:40.000000000","message":"Typo in warning message - (Severity: critical, Confidence: 1.0) | Risk: User experience | Priority: Fix Now | Why This Matters: The typo \"beremoved\" instead of \"be removed\" in a deprecation warning message looks unprofessional and reduces confidence in the code quality. | Recommendation: Change \u0027beremoved\u0027 to \u0027be removed\u0027 on line 74","commit_id":"e46a45e2506ee8bf2934af66469dc4a9ff31e4c7"},{"robot_id":"zuul","robot_run_id":"d66abc595bb14284a0ebf5413eccc03b","url":"https://zuul.teim.app/t/main/buildset/d66abc595bb14284a0ebf5413eccc03b","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"16ebec9f07721edea831e458cc168cce67fc152a","patch_set":6,"id":"8b404526_bfa96e5d","line":65,"updated":"2026-02-26 14:50:50.000000000","message":"The warning for prometheus datasource usage could include the migration guide URL\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Better user guidance when prometheus datasource is actively used.\n\n**Recommendation**:\nConsider enhancing the LOG.warning message to include the migration guide URL for users who see this warning in their logs.","commit_id":"9132e604b6ef0986fc8f217b9a6c0a21a6f5e6a1"}],"watcher/decision_engine/datasources/prometheus.py":[{"robot_id":"zuul","robot_run_id":"9fd16bc378194d76bf0c09870a9f7dbf","url":"https://zuul.teim.app/t/main/buildset/9fd16bc378194d76bf0c09870a9f7dbf","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"8018842bab3b47dddd30f69b24ae6485b210c4d6","patch_set":2,"id":"f610238d_80c4c66d","line":34,"updated":"2025-11-11 13:47:58.000000000","message":"Add deprecation warning log when prometheus datasource is instantiated - (Severity: suggestion, Confidence: 0.8) | Recommendation: Add a LOG.warning message in the PrometheusHelper init method","commit_id":"583c6b9689dcd5f0b3c514db83dfb7b5c0df67bc"},{"robot_id":"zuul","robot_run_id":"9042ac22682d45e493c6b2b1815e6378","url":"https://zuul.teim.app/t/main/buildset/9042ac22682d45e493c6b2b1815e6378","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"75d60e71975784f047846e062ba0b37dd300ea44","patch_set":4,"id":"46c11c78_a9fbbdd1","line":35,"updated":"2025-11-11 16:03:07.000000000","message":"Consider adding deprecation version to prometheus.py class docstring - (Severity: suggestion, Confidence: 0.6) | Recommendation: Include removal_version\u003d\"2027.1\" in the deprecation note for consistency with debtcollector decorator","commit_id":"2de20c8a221accf5894b11150563fea5639bd9f4"},{"robot_id":"zuul","robot_run_id":"80f5492183744bbfb1166bea401716db","url":"https://zuul.teim.app/t/main/buildset/80f5492183744bbfb1166bea401716db","author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"30fc556a1481f91954bcf9d345e3494dc8772491","patch_set":5,"id":"1a98adec_0b041f64","line":29,"updated":"2026-02-26 12:17:26.000000000","message":"Global warnings.simplefilter(\u0027always\u0027) call in prometheus.py may affect other parts of the application. The pattern should be \u0027once\u0027 like in clients.py to avoid duplicate warnings.\n\n**Severity**: WARNING | **Confidence**: 0.7\n\n**Impact**: Using \u0027always\u0027 may cause duplicate deprecation warnings to be displayed. The project standard (clients.py:47) uses \u0027once\u0027 for deprecation warnings.\n\n**Suggestion**:\nConsider changing warnings.simplefilter(\u0027always\u0027) to warnings.simplefilter(\u0027once\u0027) to match the pattern used in watcher/common/clients.py and avoid duplicate warnings. However, this may be intentional to ensure the deprecation warning is always visible during the transition period.","commit_id":"7c0a1f84eacfe55be74931783c2b54e6f8fe2208"}]}