)]}'
{"id":"openstack%2Fwatcher~1002694","triplet_id":"openstack%2Fwatcher~master~Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3","project":"openstack/watcher","branch":"master","topic":"bp/audit-pipeline","attention_set":{"30002":{"account":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"last_update":"2026-09-02 23:28:40.000000000","reason":"Someone else replied on the change"}},"removed_from_attention_set":{},"hashtags":[],"change_id":"Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3","subject":"Consolidate strategy metric-query params behind get_ methods","status":"NEW","created":"2026-08-27 21:00:11.000000000","updated":"2026-09-04 17:20:22.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":false,"submittable":false,"total_comment_count":6,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"4369498a42912d9e2ce14865970d5c8afa67f2e9","_number":1002694,"virtual_id_number":1002694,"owner":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"tag":"autogenerated:zuul:check","value":1,"date":"2026-09-03 02:48:36.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"disliked":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"all":[{"value":2,"date":"2026-09-04 17:20:22.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},{"value":-1,"date":"2026-09-02 23:38:39.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Standard Change","+1":"Important Change","+2":"High Priority Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"}],"CC":[{"_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"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-08-27 21:00:44.000000000","updated_by":{"_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"},"reviewer":{"_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"},"state":"CC"},{"updated":"2026-08-27 21:11:17.000000000","updated_by":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"reviewer":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"state":"REVIEWER"},{"updated":"2026-08-27 22:36:02.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2026-09-04 17:20:22.000000000","updated_by":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"reviewer":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"state":"REVIEWER"}],"messages":[{"id":"dda079095a705e02ff6c1c180a04fd32c26b9460","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-08-27 21:00:11.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"f331daa0efc49730a0683920054941a80550eb83","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-27 21:00:44.000000000","message":"Patch Set 1:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":1},{"id":"bbb9aefb36674d6e6d7d241a87ad55ee73beec06","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-08-27 21:11:17.000000000","message":"Patch Set 1: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/ed6b51b814b24bcca15ec6c09993db14\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/dd3b4e6a2bbf44539d57d5b912314059 : FAILURE in 9m 19s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/8d4f152b298d43448bea9a7a3c55282d : SKIPPED Skipped due to failed job openstack-meta-content-provider-master\n- s2i-openstack-container-content-provider https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/b576d50093e24235b1cffc72caefb195 : SUCCESS in 4m 51s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"bc8a983dae23b73d86f18e1967b1e321345d01ac","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-08-27 21:46:19.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n","accounts_in_message":[],"_revision_number":2},{"id":"fe09967cb86099faa16c83b3d056f7013c0076b3","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-27 21:46:53.000000000","message":"Patch Set 2:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":2},{"id":"558488e3a43f2b9f241025bd08726a6167a80149","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-08-27 21:56:12.000000000","message":"Patch Set 2: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/0ee7963c5a7d411399023346b15bcff4\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/e0e52f64e90e4a4492d7b754f383112a : FAILURE in 7m 46s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/d87ec47fce0b45489c0b18edb7cf11d3 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master\n- s2i-openstack-container-content-provider https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/f88f32439bd34340964b2f9a5da04db9 : SUCCESS in 4m 57s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"5882575e31fa4fe8585c29db68a4b083da871d6d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-27 22:36:02.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/b6411e6f03004d16b9c748b19b3ec951\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/d0fc1dc7e6534337a47f7e3668f73aaa : FAILURE in 4m 43s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/23182d6639424d46846eccc3f9821272 : SUCCESS in 4m 12s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/8ac988abfaa64d3b8d4a6005f4dabe09 : FAILURE in 5m 46s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/48d2c7a536594a14852350705312b438 : FAILURE in 5m 44s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/22d3bbb8d3d74e2d9a9be534047db856 : FAILURE in 6m 56s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/cefe2b93ab7446589b6d34459136f348 : SUCCESS in 8m 24s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/4ea8026e2b874118878c566cf2daff23 : SUCCESS in 4m 19s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/d0df3854464a417998115c6de742b5f7 : FAILURE in 4m 32s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/cd5a0ea5eff9446db11c64826a16bd57 : FAILURE in 22m 52s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/e9e7571ed2244d199627dce49f1fec64 : FAILURE in 19m 46s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/d7d66a26f3624214ac43186fb19c33d3 : FAILURE in 33m 49s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/18c9a7cc712d4a53a3a55df95dde0176 : FAILURE in 35m 48s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/48c0786b86794817872bf99945103120 : FAILURE in 17m 23s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/a62808ca2bee4d6c9aeaa62c60d08b7a : FAILURE in 27m 06s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/453dda1082be496fb87cfb23b7a576cb : FAILURE in 37m 46s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/f3a06e50a3c74a89a30160e78746e235 : SUCCESS in 4m 49s","accounts_in_message":[],"_revision_number":2},{"id":"a2d7252283a1cb8cc1257a38857c8f0c7cf683b7","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-27 22:57:10.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/3e4929182926497b994da14c0d0a26b8\n\n- teim-code-review https://zuul.teim.app/t/main/build/aa8b7e1c655043dea15c9fecb593a6b3 : SUCCESS in 8m 43s\n\nWarning:\n  The commit message body reads \"fold their logic into get_period()/get_granularity()/get_aggregate(), androute all internal reads through the get_ interface.\" The words \"and\" and \"route\" are run together as \"androute\", a clear typo in the permanent change history. The intended meaning (\"and route all internal reads through the get_ interface\") remains obvious.\n\n**Severity**: SUGGESTION | **Confidence**: 0.95\n\n**Impact**: Minor readability defect in the permanent git history; no code impact.\n\n**Recommendation**:\nAmend the commit message to read \"... get_aggregate(), and route all internal reads through the get_ interface.\"","accounts_in_message":[],"_revision_number":2},{"id":"b029f547d4a2eeced85aa926932d027de147183d","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-08-28 13:42:37.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n\nOutdated Votes:\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":3},{"id":"84e57cd66bc2fc433daa9ef650c907e2ddff3ae3","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-28 13:43:10.000000000","message":"Patch Set 3:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":3},{"id":"5bc3f3991418ee3b1d0dd47946b78a3b4522c25d","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-08-28 13:52:56.000000000","message":"Patch Set 3: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/9319e06c107c4957b5a92d775499e240\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/b7dcd82212fa4fe2a37438095f1396db : FAILURE in 8m 27s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/627eb1c68f764eb78f7fd92713b22499 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master\n- s2i-openstack-container-content-provider https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/770be035e43f498fb1bef4c199a87db7 : SUCCESS in 4m 46s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"eddc79504922fcc4c891f47b03ea9c433b621f0b","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-28 14:52:19.000000000","message":"Patch Set 3:\n\n(1 comment)\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/491c93195c6340b6ab10f0ee026a14a5\n\n- teim-code-review https://zuul.teim.app/t/main/build/827f96a5f14f42e09e43e11aef9601e2 : SUCCESS in 8m 33s\n\nWarning:\n  The commit message body reads \"...fold their logic into get_period()/get_granularity()/get_aggregate(), androute all internal reads through the get_ interface.\" The intended wording is \"and route\". This is a clear typographical defect in the permanent change history that will not be caught by any code linter.\n\n**Severity**: SUGGESTION | **Confidence**: 0.95\n\n**Impact**: Minor reduction in readability of the permanent Git history; the intended meaning (\"and route all internal reads\") remains obvious to a reader, so there is no functional or review-blocking impact.\n\n**Recommendation**:\nAmend the commit message to insert the missing space: \"...get_aggregate(), and route all internal reads through the get_ interface.\" No code change is required.","accounts_in_message":[],"_revision_number":3},{"id":"197aaaf0bf29b602c43aa5025be091df1eab5f39","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-28 15:34:02.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/afd29f09b57843e9a02f4f6bb5862472\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/217e154b13024cbd9cd6779c5f787f41 : SUCCESS in 6m 49s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/be3a478545f747c29f5834e38cda8d7d : SUCCESS in 5m 08s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/3d7f4f482f724ad7b931672925f6250e : SUCCESS in 5m 48s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/5d147060ece744e0a22aa91b752e88b0 : FAILURE in 6m 31s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/e92ab4f57e90488a88d99be3e3b8257b : SUCCESS in 7m 33s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/458c3d33bd214937ac4d6fda08ce964e : SUCCESS in 9m 34s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/afd5b672c80e42e2a6717fa1136b57fd : SUCCESS in 5m 42s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/5e4f08edd8ee45f8affb0a45aaa6f4f8 : SUCCESS in 5m 15s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/a0d689b1fcd541b89ad219e9f3afaef2 : SUCCESS in 53m 28s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/64c11213429b4a1796a525e110f0f2f9 : SUCCESS in 28m 32s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/e0aec1c6bf0246599532ab8c73184a98 : SUCCESS in 35m 50s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/65a0aad5fd2f4bd0ada06b74fc82bf12 : SUCCESS in 1h 16m 48s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/596e0ce6b4ce4b86954365e631668356 : SUCCESS in 1h 44m 22s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/1a685b17d3a1418797e395204898b2fd : SUCCESS in 1h 40m 43s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/4cccb3713c3f4b02b8efc4ef1eada2e5 : SUCCESS in 52m 16s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/3964a7d9fa59487789f75a693532239d : SUCCESS in 2m 48s","accounts_in_message":[],"_revision_number":3},{"id":"e474a06b52ff9eb4f8cdad263678b99d08c1d92e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-08-28 18:35:12.000000000","message":"Uploaded patch set 4: Patch Set 3 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n\nOutdated Votes:\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":4},{"id":"c8f96484102c5e827f229e1bdf6eec65cef65ba7","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-28 18:35:37.000000000","message":"Patch Set 4:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":4},{"id":"3d94174246661035d68deabc7ddf32d86264f53b","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-08-28 18:44:47.000000000","message":"Patch Set 4: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/626e5536a45c47698f2f34547dcde273\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/8442d8c1b79e4cf48a34a1ca2c298afa : FAILURE in 7m 54s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/c48dee43a1634a3f82c4d4d70e218070 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master\n- s2i-openstack-container-content-provider https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/4ebe3ba4e6354fbebb891462ac354be1 : SUCCESS in 4m 57s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"bdc009bf064d33e15a38aef0ba1ca7a238c320ce","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-08-28 19:22:01.000000000","message":"Patch Set 4:\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/f25033bc48604bfbba86c6b326eed2bb\n\n- teim-code-review https://zuul.teim.app/t/main/build/066c5eb56e7f49ab911bcd435d5889e7 : SUCCESS in 7m 48s\n\nWarning:\n  The head commit message body contains \u0027androute all internal reads through the get_ interface\u0027 - a missing space that slightly garbles the description of the change\u0027s main mechanism. Impact: Cosmetic only; no runtime effect, but the text is the permanent record of why the migration was done. Recommendation: Fix the typo when amending the patch (e.g., together with the CF-002 message correction).","accounts_in_message":[],"_revision_number":4},{"id":"0c4e064cef89cb389ac3da12fd7986a538eea0a8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-08-28 20:17:35.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/6498da2b67e444459274e251a86bfca8\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/777830c9d46249ec944793d80a48802c : SUCCESS in 8m 29s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b21ea2885c884f2b985b8b16575c4171 : SUCCESS in 3m 59s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/03d123385eef47a288bb6933ca5f17c6 : SUCCESS in 5m 45s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/14d67b0b49664e57bc4416e027079e55 : SUCCESS in 7m 52s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/5085d8f3279149d88be76c1a799e8c04 : SUCCESS in 8m 51s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a8908d420b5f4ff8bf9331de7022c1eb : SUCCESS in 9m 09s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/c6766e8f4a69486c8f583c6566e5e114 : SUCCESS in 4m 23s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/571281f254eb440d895ab7e10288a7a0 : SUCCESS in 3m 23s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/388494f093d54b1fb2b241f8ac95f7ba : SUCCESS in 52m 42s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/fd3a4cb0a9194e22b32f9093f7b7878f : SUCCESS in 53m 44s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/950f3fd976a549ca9e929829866cf79d : SUCCESS in 34m 13s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/866554b21b5b4277a74e974807c053e3 : SUCCESS in 43m 11s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/08f3265710b4472897881aea1c773d6a : SUCCESS in 1h 34m 21s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/94100e6833364d96bb147cde0cf9ed7a : SUCCESS in 50m 01s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/3e8496007115469da75c08ffdd5efc4e : SUCCESS in 51m 04s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/03a9a794d9754743a36655c853153308 : SUCCESS in 4m 11s","accounts_in_message":[],"_revision_number":4},{"id":"0792d240e69e2c4cac48678f7cc81cf6312cd3ba","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-09-01 21:59:16.000000000","message":"Uploaded patch set 5: Patch Set 4 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n\nOutdated Votes:\n* Verified+1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":5},{"id":"6930b027bcf20a17e3b4ce00b26807b44526951a","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-01 21:59:43.000000000","message":"Patch Set 5:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":5},{"id":"f8de267bf64084610b74f2fc64d303f6977f30d6","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-09-01 22:15:32.000000000","message":"Patch Set 5: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/4b8a79ce158e41aab1b9c6ace80dcd8b\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/ffb5f06ad6d14f8ead79e2d4ea1c9ae1 : FAILURE in 14m 36s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/82f91bb8329e4be79d0db59d7db46567 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master\n- s2i-openstack-container-content-provider https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/32e59bd8422546a7a9e94d01d2bc62db : SUCCESS in 5m 59s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"66c8daab9f95d3d0554c7c95cdef5011a42ffaab","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-01 23:13:47.000000000","message":"Patch Set 5:\n\n(1 comment)\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/4383c6df911a4a0184e81b041a229940\n\n- teim-code-review https://zuul.teim.app/t/main/build/4dfd7a9e9644446c8fe7e6b824e8e2fb : SUCCESS in 8m 50s\n\nWarning:\n  The commit message paragraph ends \u0027...fold their logic into get_period()/get_granularity()/get_aggregate(), androute all internal reads through the get_ interface.\u0027 The words \u0027and\u0027 and \u0027route\u0027 are run together as \u0027androute\u0027, a clear spelling defect in the permanent change history that a code linter will never catch.\n\n**Severity**: SUGGESTION | **Confidence**: 0.95\n\n**Impact**: Minor reduction in the readability and quality of the permanent git history; the intended meaning remains obvious.\n\n**Recommendation**:\nAmend the commit message so the sentence reads \u0027... and route all internal reads through the get_ interface.\u0027","accounts_in_message":[],"_revision_number":5},{"id":"e3b5c02b490e5f6d65b535ee00a7d81150fc1b2b","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-09-01 23:48:46.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/3b7aacd807b44a37837f9d5c1046e459\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1ebc421bc510449197b2b9dba5440f29 : SUCCESS in 7m 34s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/a3c1f19d804540bfbd0aaa35e11ed133 : SUCCESS in 4m 08s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/202be0154a884fea8b7ec1c21bddab43 : SUCCESS in 3m 50s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/25b19b4e6d5f4158b34e741fef4d9687 : SUCCESS in 4m 52s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/abfaaaf5a7cc4fc190baccc8f99a3bd4 : SUCCESS in 10m 23s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ab98302e615f40dfaa41c31f1e6ecee2 : SUCCESS in 7m 17s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/153dbe59136b4365b459fe0597840175 : SUCCESS in 4m 42s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/b7520e81d9c045749d59fc8ed1704a11 : SUCCESS in 5m 18s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/cd9766225982449da32665d4051cd622 : SUCCESS in 55m 43s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/46f8dca08edb493087fcc47895c0d3b5 : SUCCESS in 29m 14s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/a637e695af6d4a06a5e033af005e5a99 : SUCCESS in 21m 21s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/aaf8ef450e8e4a048c3b0064ced6ad63 : SUCCESS in 1h 37m 54s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/987c3d0ef2f74a9e97969ac8005f5bc8 : SUCCESS in 1h 39m 56s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/a30ac82f870e4d23bfe94d90679d230a : SUCCESS in 1h 18m 05s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/f7e04a74507442b19688d1efeb07046b : SUCCESS in 1h 34m 14s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/35ec58a49fca4ea9a89d3c31dc53c1e6 : SUCCESS in 4m 36s","accounts_in_message":[],"_revision_number":5},{"id":"a0f45c571e43509cb61fe44541dab9d8dd950bfb","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-09-02 15:49:06.000000000","message":"Uploaded patch set 6: Patch Set 5 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n\nOutdated Votes:\n* Verified+1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":6},{"id":"19fa06b509beaabf0e7d942bdad72c73ee1d1d5f","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-02 15:49:37.000000000","message":"Patch Set 6:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":6},{"id":"00556b6247c40972b7ced744df176dd7bd8a1714","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-02 17:02:43.000000000","message":"Patch Set 6:\n\n(1 comment)\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/e54edfdeffc14fb295da9c717f6c4e8d\n\n- teim-code-review https://zuul.teim.app/t/main/build/dbc2addf904c429d8f0fcaf4fa5f3fa2 : SUCCESS in 11m 40s\n\nWarning:\n  The commit message body reads \u0027get_period()/get_granularity()/get_aggregate(), androute all internal reads through the get_ interface.\u0027 The run-on \u0027androute\u0027 is a typo for \u0027and route\u0027, a spelling defect in the permanent change history that is not caught by code linters.\n\n**Severity**: SUGGESTION | **Confidence**: 0.97\n\n**Impact**: Minor readability defect in the permanent git history; meaning is still recoverable from context, but the sentence requires the reader to mentally correct the typo.\n\n**Recommendation**:\nAmend the commit message to \u0027... and route all internal reads through the get_ interface.\u0027 before merge.\n  This change completes removing the period/granularity/aggregation_method attribute accessors and routes all metric queries through the get_period()/get_granularity()/get_aggregate() interface, but the contributor documentation that teaches third-party strategy authors how to query metrics still demonstrates the legacy direct-attribute pattern (self.periods[\u0027instance\u0027], self.granularity, aggregation\u003dself.aggregation_method[\u0027instance\u0027]). New plugin authors following the official tutorial will bypass the consolidated get_ interface this migration establishes. Impact: Future out-of-tree strategy plugin authors copy the documented pattern and define their own period/granularity/aggregation attributes instead of overriding the get_ methods, producing plugins that bypass the consolidated interface (e.g. the cascade context\u0027s _strategy_metric_param in cascade.py:313-323 reads only the get_ methods, so such plugins would get no period/granularity/aggregate when simulated in cascade audits). Recommendation: Update the statistic_aggregation example in doc/source/contributor/plugin/strategy-plugin.rst (lines ~299-305) to use self.get_period(\u0027instance\u0027), self.get_aggregate(\u0027instance\u0027), and self.get_granularity(\u0027instance\u0027), matching the interface now defined in watcher/decision_engine/strategy/strategies/base.py.","accounts_in_message":[],"_revision_number":6},{"id":"366787d4899709466fd243dc3eac3d2ea39c3b9a","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-09-02 17:53:21.000000000","message":"Patch Set 6: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/ad1eb524a13e4ca0a95f8283e9cba744\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/9c0673342383435aac7ba1026aec23c3 : FAILURE in 11m 16s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/9cc90a3cac3a472e87095451ffbdfbc8 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master","accounts_in_message":[],"_revision_number":6},{"id":"f98b7700c665cbd78b429ae1bc4ea5041abb8613","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-09-02 19:19:03.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/5c328c2b8d664d958548b0682d7a089b\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/11aca891147d48fc908f5e6007f6aa23 : SUCCESS in 7m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ae0a365610574aa08cc77288950ed9e3 : SUCCESS in 7m 30s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/992bda8e3a624c5abb44bba7765b585a : SUCCESS in 6m 10s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/24209dff0d8f412287e1fd10eff355a8 : SUCCESS in 5m 55s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/19263126c08f45b492dc03e0c5ed5a41 : SUCCESS in 8m 47s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/1b57450d99ba467297a647f279c2d7dc : SUCCESS in 9m 35s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/1424dd8b8dd64678b7dfecd0f924bb73 : SUCCESS in 7m 06s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/dbc823b24a2d483ea59e2aa39c6171a8 : SUCCESS in 5m 17s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/0448166f50094462a596223b13c2a91c : SUCCESS in 1h 00m 46s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/4c42bdebd5b84a9bac07e0f4529e1992 : SUCCESS in 51m 26s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/4d7895456cb9400d9153b35a6e9c139a : ERROR Failed to update project openstack/glance in 15s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/4a73e330aac3439dba3bfb5b30eb2b15 : SUCCESS in 1h 32m 33s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/4bf8ff4620ac458a86ee08091a08ac63 : SUCCESS in 1h 38m 23s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/216355329eb147bcbf3b0020b65eead2 : SUCCESS in 1h 48m 36s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/6847f6f808db42b68d338cab9c67d488 : ERROR Failed to update project openstack/glance in 8s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/bdfedbcbb6ac45fca43b1a0bb127d6b9 : SUCCESS in 5m 22s","accounts_in_message":[],"_revision_number":6},{"id":"673c7bbd3499cdb65819b6e9ee23c19dda81d4a7","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2026-09-02 23:28:07.000000000","message":"Uploaded patch set 7: Patch Set 6 was rebased.\n\nCopied Votes:\n* Code-Review-1 (copy condition: \"**changekind:TRIVIAL_REBASE** OR is:MIN\")\n\nOutdated Votes:\n* Verified-1 (copy condition: \"NEVER\")\n","accounts_in_message":[],"_revision_number":7},{"id":"73c99822d54120425c1f1d09b41a0f8a87c45284","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-02 23:28:40.000000000","message":"Patch Set 7:\n\nStarting automatic-ci jobs.","accounts_in_message":[],"_revision_number":7},{"id":"faadb57eecc8872cb8fff6e1c0714e1ec823889a","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2026-09-02 23:38:39.000000000","message":"Patch Set 7: Code-Review-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\nhttps://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/7bd50fc9f41a453dba6b3bce0d1b0655\n\n- openstack-meta-content-provider-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/1ed00596bb7b421f9a6505eeb1e78943 : FAILURE in 9m 16s\n- watcher-operator-validation-master https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/build/43a7a0e1b2664174a0105b2449023e43 : SKIPPED Skipped due to failed job openstack-meta-content-provider-master","accounts_in_message":[],"_revision_number":7},{"id":"6c4f6b7171b7294e21158f7f129b443b0c4ecfe2","tag":"autogenerated:zuul:automatic-ci","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"},"date":"2026-09-03 00:36:22.000000000","message":"Patch Set 7:\n\n(1 comment)\n\nBuild succeeded (automatic-ci pipeline).\nhttps://zuul.teim.app/t/main/buildset/81ac87d0406841788d3cb2a9bc15bd45\n\n- teim-code-review https://zuul.teim.app/t/main/build/251f805a0a524392a7a08dec6c1df370 : SUCCESS in 9m 18s\n\nWarning:\n  The commit message body reads \"fold their logic into get_period()/get_granularity()/get_aggregate(), androute all internal reads through the get_ interface.\" \"androute\" should be \"and route\". The intended meaning stays obvious, but the wording defect is preserved in the permanent change history and Gerrit change page.\n\n**Severity**: SUGGESTION | **Confidence**: 0.95\n\n**Impact**: Minor readability blemish in permanent git history and the Gerrit change record; no code impact.\n\n**Recommendation**:\nAmend the patchset to fix \"androute\" to \"and route\" (and optionally \"Complete the migration started in previous change\" to \"...in the previous change\") before merging; no code change needed.","accounts_in_message":[],"_revision_number":7},{"id":"051ec0d95772c9507cf575d698c00a28735d50b9","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-09-03 02:48:36.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/2a0d469be5c24abdb29bef5f9879c279\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/5de3c936416b418cb93145ca9eee5791 : SUCCESS in 7m 29s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e926608ed43b43abaad14b1b509e848d : SUCCESS in 5m 52s\n- openstack-tox-py311 https://zuul.opendev.org/t/openstack/build/32dd76c3852247d088fb9e64c10a8981 : SUCCESS in 8m 53s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/f95291d7c1494dd8a783a02e69e0efc0 : SUCCESS in 5m 37s\n- openstack-tox-py314 https://zuul.opendev.org/t/openstack/build/2d15c623bcbb47e7b012a1952b755358 : SUCCESS in 10m 46s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c29ec80e57c048c5aaa4847eda89b14b : SUCCESS in 9m 05s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/f35d171b735945ceb5f9ff8e1e498154 : SUCCESS in 6m 58s\n- openstack-tox-py313-eventlet https://zuul.opendev.org/t/openstack/build/0ddd930310584d429bd7a4d18fc48b24 : SUCCESS in 5m 40s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/cd99aeb4eda449ef82afc649ba059565 : SUCCESS in 54m 37s\n- watcher-grenade-skip-level-always https://zuul.opendev.org/t/openstack/build/e145f9f7c7ca4794b6acceacea69eda0 : SUCCESS in 49m 41s\n- watcher-tempest-api-ipv6-only https://zuul.opendev.org/t/openstack/build/0e59bc54d63b4f43a25f5b2ead2aa4f6 : SUCCESS in 36m 15s\n- watcher-tempest-gnocchi https://zuul.opendev.org/t/openstack/build/c486307e75e24ed9a7fd7846c43fb525 : SUCCESS in 1h 42m 59s\n- watcher-tempest-prometheus https://zuul.opendev.org/t/openstack/build/402f77cff89244e5bef20040f23e2401 : SUCCESS in 57m 50s\n- watcher-tempest-prometheus-eventlet https://zuul.opendev.org/t/openstack/build/da64d470fddc4f16a77101f570888509 : SUCCESS in 1h 37m 18s\n- watcher-tempest-aetos https://zuul.opendev.org/t/openstack/build/798d7c41601844c3b24a5d5f37449e54 : SUCCESS in 1h 19m 08s\n- openstack-tox-functional-py313 https://zuul.opendev.org/t/openstack/build/1294580aa725463ba09e3c92484c1b6c : SUCCESS in 4m 58s","accounts_in_message":[],"_revision_number":7},{"id":"4369498a42912d9e2ce14865970d5c8afa67f2e9","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2026-09-04 17:20:22.000000000","message":"Patch Set 7: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":7}],"current_revision_number":7,"current_revision":"1fdcecb1374d1658ddfda0f44ce6b246b7eb0681","revisions":{"3524a77ce7e627cb6831dcb4894cfcdfb11ba297":{"kind":"REWORK","_number":1,"created":"2026-08-27 21:00:11.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/1"}}},"commit":{"parents":[{"commit":"07c3b091dcf890deaad3d18d63f62db8ff501b7f","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/07c3b091dcf890deaad3d18d63f62db8ff501b7f"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 20:58:45.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/3524a77ce7e627cb6831dcb4894cfcdfb11ba297"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/3524a77ce7e627cb6831dcb4894cfcdfb11ba297"}]},"branch":"refs/heads/master"},"6ee90dde41b97f548645e56480c70941c5fa8778":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2026-08-27 21:46:19.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/2"}}},"commit":{"parents":[{"commit":"f068af143732a4d10126d84b0c348db94bc9f3de","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/f068af143732a4d10126d84b0c348db94bc9f3de"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 21:42:35.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6ee90dde41b97f548645e56480c70941c5fa8778"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/6ee90dde41b97f548645e56480c70941c5fa8778"}]},"branch":"refs/heads/master"},"8d0400e3377f4f5af6db8604e2a309c7f0aa050f":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2026-08-28 13:42:37.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/3"}}},"commit":{"parents":[{"commit":"364f7e1db8377517f5373b1bd605b8e9c583eb7f","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/364f7e1db8377517f5373b1bd605b8e9c583eb7f"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-28 13:01:44.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8d0400e3377f4f5af6db8604e2a309c7f0aa050f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8d0400e3377f4f5af6db8604e2a309c7f0aa050f"}]},"branch":"refs/heads/master"},"67d4d4eae6aa382231d8b0ea7a6c74ef8f7253a4":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2026-08-28 18:35:12.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/4"}}},"commit":{"parents":[{"commit":"d1f84c4d5f5ca77b1d03b21839b053713991f94b","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/d1f84c4d5f5ca77b1d03b21839b053713991f94b"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-28 18:23:18.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/67d4d4eae6aa382231d8b0ea7a6c74ef8f7253a4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/67d4d4eae6aa382231d8b0ea7a6c74ef8f7253a4"}]},"branch":"refs/heads/master"},"7aad3c5861fe82ffd34a2061d31530e8852f44eb":{"kind":"TRIVIAL_REBASE","_number":5,"created":"2026-09-01 21:59:16.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/5"}}},"commit":{"parents":[{"commit":"63c3b754551f0d366df8c9ace5393a361578a04a","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/63c3b754551f0d366df8c9ace5393a361578a04a"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-09-01 21:46:32.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/7aad3c5861fe82ffd34a2061d31530e8852f44eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/7aad3c5861fe82ffd34a2061d31530e8852f44eb"}]},"branch":"refs/heads/master"},"90aedc022486d9aa95d4413b16d29fb47e9c1a09":{"kind":"TRIVIAL_REBASE","_number":6,"created":"2026-09-02 15:49:06.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/6"}}},"commit":{"parents":[{"commit":"8e0700506ae2b3da81d966d14aca0f0faef8e19d","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8e0700506ae2b3da81d966d14aca0f0faef8e19d"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-09-02 15:46:52.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/90aedc022486d9aa95d4413b16d29fb47e9c1a09"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/90aedc022486d9aa95d4413b16d29fb47e9c1a09"}]},"branch":"refs/heads/master"},"1fdcecb1374d1658ddfda0f44ce6b246b7eb0681":{"kind":"TRIVIAL_REBASE","_number":7,"created":"2026-09-02 23:28:07.000000000","uploader":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"ref":"refs/changes/94/1002694/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/94/1002694/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/94/1002694/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/94/1002694/7"}}},"commit":{"parents":[{"commit":"3658497679dff7a694a494bb0d67538a0ce7d2f8","subject":"Add Audit Pipeline REST API endpoints","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/3658497679dff7a694a494bb0d67538a0ce7d2f8"}]}],"author":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-08-27 17:56:19.000000000","tz":-180},"committer":{"name":"Douglas Viroel","email":"viroel@gmail.com","date":"2026-09-02 23:26:13.000000000","tz":-180},"subject":"Consolidate strategy metric-query params behind get_ methods","message":"Consolidate strategy metric-query params behind get_ methods\n\nComplete the migration started in previous change by removing the\nredundant period, granularity, and aggregation_method @property\naccessors from the strategies, fold their logic into\nget_period()/get_granularity()/get_aggregate(), androute all internal\nreads through the get_ interface.\nAlso rename get_datasource_metrics() to get_datasource_metrics_in_use()\nsince datasource metrics can be retrieved from DATASOURCE_METRICS but\nsome strategies allow users to select a subset of metrics when creating\ntheir audits. get_datasource_metrics_in_use() will provide the in_use\nmetrics based on the strategy\u0027s input_parameters. No behavior change;\ntests updated accordingly.\n\nAssisted-By: Claude Code (Opus 4.8)\nPartially-implements: bp audit-pipeline\nChange-Id: Idf00bcb6f3fd7e5c92bc966c69a9deaa88661fb3\nSigned-off-by: Douglas Viroel \u003cviroel@gmail.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/1fdcecb1374d1658ddfda0f44ce6b246b7eb0681"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/1fdcecb1374d1658ddfda0f44ce6b246b7eb0681"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"}},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}
