)]}'
{"doc/source/install/installation.rst":[{"robot_id":"zuul","robot_run_id":"cf2dcd425ea94e16afd3c2910f3ead6c","url":"https://zuul.teim.app/t/main/buildset/cf2dcd425ea94e16afd3c2910f3ead6c","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":"3f4a13dc4024e12a34aba63d16232066b2313bbc","patch_set":1,"id":"d4d7463d_cf27dcae","line":72,"updated":"2026-02-28 12:35:29.000000000","message":"Installation documentation has \u0027Integration testing\u0027 section (lines 72-93) referencing deleted horizon.conf and non-working commands.\n\n**Severity**: CRITICAL | **Confidence**: 1.0\n\n**Risk**: Documentation leads users to errors - outdated instructions reference deleted files\n\n**Priority**: Before merge\n**Why This Matters**: Users following integration testing instructions encounter errors, wasting effort and undermining documentation trust.\n\n**Recommendation**:\nRemove entire \u0027Integration testing\u0027 section (lines 72-93) or replace with note explaining removal and directing to unit tests.","commit_id":"635f762da055a17fc76055ca544e27e1631fd5c2"},{"robot_id":"zuul","robot_run_id":"131e7255acd64104b7986ca1589143c2","url":"https://zuul.teim.app/t/main/buildset/131e7255acd64104b7986ca1589143c2","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":"39088d2fe65fdbee2df11bd4419987747f1c9c5a","patch_set":3,"id":"38e893ba_5a583eaf","line":82,"updated":"2026-03-30 15:06:49.000000000","message":"installation.rst lines 82-83 reference the deleted file watcher_dashboard/test/integration_tests/horizon.conf, leaving a stale reference in the docs.\n\n**Severity**: HIGH | **Confidence**: 1.0\n\n**Risk**: Users following docs encounter a dead reference to a deleted config file, causing confusion when setting up integration tests.\n\n**Priority**: Before merge\n**Why This Matters**: The doc section describing integration test config references a file deleted in this patch, making the documentation inaccurate.\n\n**Recommendation**:\nRemove or rewrite the horizon.conf references in installation.rst. Consider removing the integration testing subsection entirely since tests are gone.","commit_id":"1667a35fd0d3cb358a102c219aae0ff09321a6a2"},{"robot_id":"zuul","robot_run_id":"e11c6c7bc83241eb89c2532ae1ed67f6","url":"https://zuul.teim.app/t/main/buildset/e11c6c7bc83241eb89c2532ae1ed67f6","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":"2808ef573fb83f83e1346f81d2d1d8d43c31b788","patch_set":4,"id":"d2251704_9d014215","line":68,"updated":"2026-04-04 01:25:09.000000000","message":"The blank line before the \u0027.. note::\u0027 directive in installation.rst now has two blank lines preceding it where the integration testing section was removed, creating slightly excessive whitespace in the RST source.\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Cleaning up extra whitespace improves document readability and ensures consistent RST formatting.\n\n**Recommendation**:\nRemove one of the blank lines before \u0027.. note::\u0027 so there is exactly one blank line between the end of the preceding paragraph and the directive, consistent with standard RST conventions.","commit_id":"fda67923d3bf463212c470060803ebeda521a094"}],"run_tests.sh":[{"robot_id":"zuul","robot_run_id":"cf2dcd425ea94e16afd3c2910f3ead6c","url":"https://zuul.teim.app/t/main/buildset/cf2dcd425ea94e16afd3c2910f3ead6c","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":"3f4a13dc4024e12a34aba63d16232066b2313bbc","patch_set":1,"id":"5a7a860a_4422e5fb","line":390,"updated":"2026-02-28 12:35:29.000000000","message":"run_integration_tests() function references deleted integration_tests path. Running ./run_tests.sh --integration fails with \u0027No such file or directory\u0027.\n\n**Severity**: CRITICAL | **Confidence**: 0.9\n\n**Risk**: User-facing runtime error - CI script option produces cryptic failure\n\n**Priority**: Before merge\n**Why This Matters**: Users attempting integration tests get confusing error instead of helpful message, creating poor experience and support burden.\n\n**Recommendation**:\nRemove run_integration_tests() function entirely and update --integration option to display clear message that integration tests were removed.","commit_id":"635f762da055a17fc76055ca544e27e1631fd5c2"},{"robot_id":"zuul","robot_run_id":"131e7255acd64104b7986ca1589143c2","url":"https://zuul.teim.app/t/main/buildset/131e7255acd64104b7986ca1589143c2","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":"39088d2fe65fdbee2df11bd4419987747f1c9c5a","patch_set":3,"id":"9dd4f15a_5605418f","line":33,"updated":"2026-03-30 15:06:49.000000000","message":"The --integration flag, integration\u003d0 variable, run_integration_tests() function, and help text remain in run_tests.sh despite integration tests being deleted.\n\n**Severity**: SUGGESTION | **Confidence**: 0.9\n\n**Benefit**: Completes cleanup by removing all dead integration test code, reducing maintenance burden and avoiding confusion.\n\n**Recommendation**:\nRemove the integration variable (line 77), --integration case (line 115), run_integration_tests function (lines 385-395), dispatch block (lines 566-569), and help text (lines 33-34).","commit_id":"1667a35fd0d3cb358a102c219aae0ff09321a6a2"},{"robot_id":"zuul","robot_run_id":"131e7255acd64104b7986ca1589143c2","url":"https://zuul.teim.app/t/main/buildset/131e7255acd64104b7986ca1589143c2","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":"39088d2fe65fdbee2df11bd4419987747f1c9c5a","patch_set":3,"id":"a3ecf5d9_f83d6951","line":390,"updated":"2026-03-30 15:06:49.000000000","message":"run_integration_tests() in run_tests.sh line 390 references the deleted watcher_dashboard/test/integration_tests/tests directory. The --integration flag will fail at runtime with a nosetests error.\n\n**Severity**: CRITICAL | **Confidence**: 1.0\n\n**Risk**: Running ./run_tests.sh --integration fails because the target directory no longer exists, producing a confusing nosetests error.\n\n**Priority**: Before merge\n**Why This Matters**: The --integration flag is still callable (lines 115, 566-569). Users will encounter a broken entry point rather than a clear deprecation message.\n\n**Recommendation**:\nRemove the entire run_integration_tests() function and --integration flag handling since integration tests no longer exist.","commit_id":"1667a35fd0d3cb358a102c219aae0ff09321a6a2"},{"robot_id":"zuul","robot_run_id":"e11c6c7bc83241eb89c2532ae1ed67f6","url":"https://zuul.teim.app/t/main/buildset/e11c6c7bc83241eb89c2532ae1ed67f6","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":"2808ef573fb83f83e1346f81d2d1d8d43c31b788","patch_set":4,"id":"376f5fe9_a2c390b4","line":331,"updated":"2026-04-04 01:25:09.000000000","message":"The run_tests.sh file still references nosetests in run_tests_all and run_tests_subset for test execution, but nose is deprecated and the project has switched to Django\u0027s default test runner.\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: Removing stale nosetests references would reduce confusion about the test runner and align the script with the project\u0027s actual test infrastructure.\n\n**Recommendation**:\nIn a follow-up patch, consider updating run_tests.sh to use python manage.py test directly or verifying that all test runner references are consistent with the Django default test runner.","commit_id":"fda67923d3bf463212c470060803ebeda521a094"},{"robot_id":"zuul","robot_run_id":"6a1b55d0187e40bb949a5f0d70e3df2c","url":"https://zuul.teim.app/t/main/buildset/6a1b55d0187e40bb949a5f0d70e3df2c","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":"44776886addeed66aad1846e158921087de0850e","patch_set":5,"id":"814d2227_219e85f7","line":174,"updated":"2026-04-30 17:59:01.000000000","message":"run_tests.sh references watcher_dashboard.test.settings (singular) but the actual package directory is watcher_dashboard/tests (plural). This will cause ImportError and break all test execution via run_tests.sh.\n\n**Severity**: CRITICAL | **Confidence**: 1.0\n\n**Risk**: Every invocation of run_tests.sh that calls run_pep8, run_pep8_changed, run_sphinx, or run_tests_all will fail with ModuleNotFoundError: No module named \u0027watcher_dashboard.test\u0027.\n\n**Priority**: Before merge\n**Why This Matters**: The dependent change 978257 renamed test/ to tests/ (plural). This change (978304) then updated run_tests.sh from the correct watcher_dashboard.tests.settings to the incorrect watcher_dashboard.test.settings, creating a path mismatch that breaks the script entirely.\n\n**Recommendation**:\nRevert all four occurrences of watcher_dashboard.test.settings back to watcher_dashboard.tests.settings in run_tests.sh (lines 174, 190, 196, 353 in the current file). The directory is named tests/ (plural) and tox.ini correctly uses the plural form.","commit_id":"cc795235172303fb03f89d00a1c0d8d4c0140e1c"}]}