)]}'
{"watcher_tempest_plugin/config.py":[{"robot_id":"zuul","robot_run_id":"27f6055200ae4b6bb3453f4b890a59af","url":"https://zuul.teim.app/t/main/buildset/27f6055200ae4b6bb3453f4b890a59af","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":"512c709045fa67fab61e20d51f308fb65442dd03","patch_set":2,"id":"908cb331_3e83d2cd","line":148,"updated":"2025-12-02 15:50:40.000000000","message":"Consider adding the new configuration options to the documentation\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Users will discover these options through documentation rather than code inspection\n\n**Recommendation**:\nUpdate README.rst or add documentation about the new zone migration test configuration options","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"robot_id":"zuul","robot_run_id":"27f6055200ae4b6bb3453f4b890a59af","url":"https://zuul.teim.app/t/main/buildset/27f6055200ae4b6bb3453f4b890a59af","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":"512c709045fa67fab61e20d51f308fb65442dd03","patch_set":2,"id":"e6a40875_fb977ba7","line":148,"updated":"2025-12-02 15:50:40.000000000","message":"Consider using more descriptive help text for configuration options\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Better documentation for users configuring the test runner\n\n**Recommendation**:\nExpand 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.\"","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"robot_id":"zuul","robot_run_id":"27f6055200ae4b6bb3453f4b890a59af","url":"https://zuul.teim.app/t/main/buildset/27f6055200ae4b6bb3453f4b890a59af","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":"512c709045fa67fab61e20d51f308fb65442dd03","patch_set":2,"id":"361efd49_a3083a7c","line":151,"updated":"2025-12-02 15:50:40.000000000","message":"Help text for configuration options should end with a period\n\n**Severity**: WARNING | **Confidence**: 0.8\n\n**Impact**: Inconsistent documentation format that may affect config help display\n\n**Suggestion**:\nAdd a period at the end of the help text: \"Whether or not to run extra zone migration tests.\"","commit_id":"aa5db42a0d6039096db126a95ed43e7c5afa13fd"},{"robot_id":"zuul","robot_run_id":"3ce76c6ab8cd4c50851253a83b4d40dd","url":"https://zuul.teim.app/t/main/buildset/3ce76c6ab8cd4c50851253a83b4d40dd","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":"a273fe65b7bd8f87457700661159eba23824a41f","patch_set":3,"id":"79724bc7_7b3e2662","line":149,"updated":"2025-12-09 07:35:15.000000000","message":"Configuration help text could be more descriptive about when to enable these tests\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Better documentation helps users understand when and why to enable these specific test categories\n\n**Recommendation**:\nConsider expanding the help text to explain the specific requirements or use cases for these test categories, such as mentioning they require specific infrastructure setup.","commit_id":"15eef7f387b61429f7ffed00be022a11384cd794"},{"robot_id":"zuul","robot_run_id":"3ce76c6ab8cd4c50851253a83b4d40dd","url":"https://zuul.teim.app/t/main/buildset/3ce76c6ab8cd4c50851253a83b4d40dd","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":"a273fe65b7bd8f87457700661159eba23824a41f","patch_set":3,"id":"36c3b87b_37508ff2","line":150,"updated":"2025-12-09 07:35:15.000000000","message":"Typo in help text: \u0027destionation\u0027 should be \u0027destination\u0027\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: Minor documentation issue that affects readability\n\n**Suggestion**:\nFix the typo by changing \u0027destionation\u0027 to \u0027destination\u0027 in the help text for run_zone_migration_extra_tests","commit_id":"15eef7f387b61429f7ffed00be022a11384cd794"},{"robot_id":"zuul","robot_run_id":"f583e279f31c44fb90f63f041b4b1221","url":"https://zuul.teim.app/t/main/buildset/f583e279f31c44fb90f63f041b4b1221","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":"ae429b11c3a82a4494f0c269cc8d93c31d4bdd02","patch_set":4,"id":"d0edf973_34d3db0c","line":1,"updated":"2025-12-09 08:39:25.000000000","message":"Encoding declaration in config.py uses non-standard format\n\n**Severity**: WARNING | **Confidence**: 0.8\n\n**Impact**: OpenStack projects typically use standard Python encoding declarations\n\n**Suggestion**:\nConsider changing to standard format: -*- coding: utf-8 -*-","commit_id":"0b81f2b911cc274acd5ba39b66e0a718b65180e5"},{"robot_id":"zuul","robot_run_id":"f583e279f31c44fb90f63f041b4b1221","url":"https://zuul.teim.app/t/main/buildset/f583e279f31c44fb90f63f041b4b1221","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":"ae429b11c3a82a4494f0c269cc8d93c31d4bdd02","patch_set":4,"id":"8203bfb8_feabd784","line":149,"updated":"2025-12-09 08:39:25.000000000","message":"Configuration flag help strings could be more descriptive\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Better documentation for users configuring the test environment\n\n**Recommendation**:\nAdd more context about when these tests should be enabled and what infrastructure they require","commit_id":"0b81f2b911cc274acd5ba39b66e0a718b65180e5"}],"watcher_tempest_plugin/tests/scenario/test_execute_zone_migration.py":[{"robot_id":"zuul","robot_run_id":"4a8d049691dd4f3fb93a2b9b3209a489","url":"https://zuul.teim.app/t/main/buildset/4a8d049691dd4f3fb93a2b9b3209a489","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":"109eaf7a4c5b7ca4e4a2daf4635ae1e85baafbf3","patch_set":1,"id":"a7e174b2_5b37e24e","line":84,"updated":"2025-12-02 13:59:03.000000000","message":"Configuration options used without being defined in config.py\n\n**Severity**: CRITICAL | **Confidence**: 0.9\n\n**Risk**: Runtime failure due to undefined configuration attributes\n\n**Priority**: Immediate\n**Why This Matters**: The code references CONF.optimize.run_zone_migration_extra_tests and CONF.optimize.run_zone_migration_storage_tests which are not defined in the configuration, causing AttributeError when these tests run.\n\n**Recommendation**:\nAdd the missing configuration options to watcher_tempest_plugin/config.py in the OptimizationGroup list. Add cfg.BoolOpt(\u0027run_zone_migration_extra_tests\u0027, default\u003dTrue, help\u003d\u0027Whether or not to run extra zone migration tests without destination host\u0027) and cfg.BoolOpt(\u0027run_zone_migration_storage_tests\u0027, default\u003dTrue, help\u003d\u0027Whether or not to run storage-related zone migration tests\u0027)","commit_id":"7ff9518b42559c9fb4cdbec7eaa65c69f1fe3a44"}]}