)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"32ba31db01fa47b284e4f50c04e8f3add6b95161","unresolved":true,"context_lines":[{"line_number":11,"context_line":"better control over which tests run in different CI environments."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Changes:"},{"line_number":14,"context_line":"- Add run_zone_migration_extra_tests flag check to skip"},{"line_number":15,"context_line":"  zone migration for instances without dst_node tests when disabled"},{"line_number":16,"context_line":"- Add run_zone_migration_storage_tests flag check to skip"},{"line_number":17,"context_line":"  storage-related zone migration tests when disabled"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"8ef46da9_275f8961","line":15,"range":{"start_line":14,"start_character":2,"end_line":15,"end_character":67},"updated":"2025-12-05 08:21:09.000000000","message":"Each of the zone migration tests contains \u0027zone_migration\u0027 tag. We can pass tag as regex via `--regex` and `--exclude-regex` option provided by tempest run cli. This way we can include it.\n\nRecently we have added similar flag for continuous and date model tests. I am not sure we want to add more such config in watcher tempest.conf.\n\nWe will end up with multiple such config flags to control different strategies.\n\nLet\u0027s see what other reviewers think about this.","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"4d0e690f9ef1ae52ec9c2d14511d387fb1ad2308","unresolved":true,"context_lines":[{"line_number":11,"context_line":"better control over which tests run in different CI environments."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Changes:"},{"line_number":14,"context_line":"- Add run_zone_migration_extra_tests flag check to skip"},{"line_number":15,"context_line":"  zone migration for instances without dst_node tests when disabled"},{"line_number":16,"context_line":"- Add run_zone_migration_storage_tests flag check to skip"},{"line_number":17,"context_line":"  storage-related zone migration tests when disabled"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"be24a790_374c9cef","line":15,"range":{"start_line":14,"start_character":2,"end_line":15,"end_character":67},"in_reply_to":"8ef46da9_275f8961","updated":"2025-12-08 17:47:54.000000000","message":"We could go with the exclude_regex or these configs, the idea is that we remove these configs once we merge the backport. The continuous audit I think that may be removed with some backports too. The only exception would be extended_data_model tests which is a new feature.","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"12fe7666fb91a3a08b6076e32c7e6ff785e59f88","unresolved":true,"context_lines":[{"line_number":11,"context_line":"better control over which tests run in different CI environments."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Changes:"},{"line_number":14,"context_line":"- Add run_zone_migration_extra_tests flag check to skip"},{"line_number":15,"context_line":"  zone migration for instances without dst_node tests when disabled"},{"line_number":16,"context_line":"- Add run_zone_migration_storage_tests flag check to skip"},{"line_number":17,"context_line":"  storage-related zone migration tests when disabled"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"93ac9281_24b4eeb1","line":15,"range":{"start_line":14,"start_character":2,"end_line":15,"end_character":67},"in_reply_to":"be24a790_374c9cef","updated":"2025-12-08 22:41:16.000000000","message":"that not really how that works.\n\nin general we shoudl not be backporting change the result in api changes that tempest can detect.\n\n\nwe woudl remove thsee when the last reelase that did not have the new behvior goes unmaintained.\n\nwe do not backport feature and upstream the about to not specify a destions is arguable a feature.","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"ded02fd8663f874982c06e03f85e5c172f974318","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2d6ba3be_b195bc5e","updated":"2025-12-02 15:31:42.000000000","message":"recheck unrelated failure","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"}],"watcher_tempest_plugin/config.py":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"2d166bc7e6b887972e92f56ed33b0cc97ccc5076","unresolved":false,"context_lines":[{"line_number":145,"context_line":"             \"less than 300 seconds.\""},{"line_number":146,"context_line":"    ),"},{"line_number":147,"context_line":"    cfg.BoolOpt("},{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests. \""},{"line_number":151,"context_line":"    ),"}],"source_content_type":"text/x-python","patch_set":2,"id":"0fbff7ae_102a2562","line":148,"in_reply_to":"616ab2fd_57d3b438","updated":"2025-12-09 07:23:07.000000000","message":"Done","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"12fe7666fb91a3a08b6076e32c7e6ff785e59f88","unresolved":false,"context_lines":[{"line_number":145,"context_line":"             \"less than 300 seconds.\""},{"line_number":146,"context_line":"    ),"},{"line_number":147,"context_line":"    cfg.BoolOpt("},{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests. \""},{"line_number":151,"context_line":"    ),"}],"source_content_type":"text/x-python","patch_set":2,"id":"2f2c35ed_441a1048","line":148,"in_reply_to":"908cb331_3e83d2cd","updated":"2025-12-08 22:41:16.000000000","message":"we generate the docs form the code","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"12fe7666fb91a3a08b6076e32c7e6ff785e59f88","unresolved":true,"context_lines":[{"line_number":145,"context_line":"             \"less than 300 seconds.\""},{"line_number":146,"context_line":"    ),"},{"line_number":147,"context_line":"    cfg.BoolOpt("},{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests. \""},{"line_number":151,"context_line":"    ),"}],"source_content_type":"text/x-python","patch_set":2,"id":"616ab2fd_57d3b438","line":148,"in_reply_to":"e6a40875_fb977ba7","updated":"2025-12-08 22:41:16.000000000","message":"\u003e Consider using more descriptive help text for configuration options\n\u003e \n\u003e **Severity**: SUGGESTION | **Confidence**: 0.7\n\u003e \n\u003e **Benefit**: Better documentation for users configuring the test runner\n\u003e \n\u003e **Recommendation**:\n\u003e Expand help text to explain what types of tests are controlled, e.g., \"Whether or not to run extra zone migration tests that test migration without specified destination nodes.\"\n\nya that valid i guess its a better description of that it does.","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"12fe7666fb91a3a08b6076e32c7e6ff785e59f88","unresolved":true,"context_lines":[{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests. \""},{"line_number":151,"context_line":"    ),"},{"line_number":152,"context_line":"    cfg.BoolOpt("},{"line_number":153,"context_line":"        \"run_zone_migration_storage_tests\","},{"line_number":154,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"4dbd7f01_aa8726dd","line":151,"in_reply_to":"361efd49_a3083a7c","updated":"2025-12-08 22:41:16.000000000","message":"this is saying you shoudl not have trailing whitespace in the help text\n\n\n```\n        help\u003d\"Whether or not to run extra zone migration tests.\"\n```","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"2d166bc7e6b887972e92f56ed33b0cc97ccc5076","unresolved":false,"context_lines":[{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests. \""},{"line_number":151,"context_line":"    ),"},{"line_number":152,"context_line":"    cfg.BoolOpt("},{"line_number":153,"context_line":"        \"run_zone_migration_storage_tests\","},{"line_number":154,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"30c2447b_6f068d21","line":151,"in_reply_to":"4dbd7f01_aa8726dd","updated":"2025-12-09 07:23:07.000000000","message":"Done","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0728ef7efcb1d82935aa9bde56773eb7062f44e3","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# -*- encoding: utf-8 -*-"},{"line_number":2,"context_line":"# Copyright (c) 2016 b\u003c\u003ecom"},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"}],"source_content_type":"text/x-python","patch_set":4,"id":"b6d63fb9_44fa0f36","line":1,"in_reply_to":"d0edf973_34d3db0c","updated":"2025-12-09 08:48:13.000000000","message":"\u003e Encoding declaration in config.py uses non-standard format\n\u003e \n\u003e **Severity**: WARNING | **Confidence**: 0.8\n\u003e \n\u003e **Impact**: OpenStack projects typically use standard Python encoding declarations\n\u003e \n\u003e **Suggestion**:\n\u003e Consider changing to standard format: `-*- coding: utf-8 -*-`\n\n\nhuh today i learned \nhttps://docs.python.org/3/reference/lexical_analysis.html#encoding-declarations\n\nit should be codeing not encoding\n\nhowever we shoudl also remove these entrily and not add them in new code\n\nthis was requried in python 2 but its not required in python 3  as utf-8 is the enecodeing unless you overrided it.\n\nwe can ignore this for now but we shoudl just remove all (en)coding marker seperately","commit_id":"0b81f2b911cc274acd5ba39b66e0a718b65180e5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0728ef7efcb1d82935aa9bde56773eb7062f44e3","unresolved":false,"context_lines":[{"line_number":146,"context_line":"    ),"},{"line_number":147,"context_line":"    cfg.BoolOpt("},{"line_number":148,"context_line":"        \"run_zone_migration_extra_tests\","},{"line_number":149,"context_line":"        default\u003dFalse,"},{"line_number":150,"context_line":"        help\u003d\"Whether or not to run extra zone migration tests that test\""},{"line_number":151,"context_line":"             \"migration without specified destination nodes.\""},{"line_number":152,"context_line":"    ),"}],"source_content_type":"text/x-python","patch_set":4,"id":"668410a3_a9a730e2","line":149,"in_reply_to":"8203bfb8_feabd784","updated":"2025-12-09 08:48:13.000000000","message":"this is enough","commit_id":"0b81f2b911cc274acd5ba39b66e0a718b65180e5"}]}