)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"bb7d4b85ed2bf98b7961f7baa710aae31c3a1f05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dc7c1989_148840b9","updated":"2026-09-15 17:37:04.000000000","message":"recheck\n\nlets get teim-ci run again","commit_id":"d1d774b55f06dbf0a7484af127e1d3bed857e519"}],"doc/source/admin/apache-mod-wsgi.rst":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"bb7d4b85ed2bf98b7961f7baa710aae31c3a1f05","unresolved":true,"context_lines":[{"line_number":30,"context_line":"   * Modify the ``WSGIDaemonProcess`` directive to set the ``user`` and"},{"line_number":31,"context_line":"     ``group`` values to appropriate user on your server."},{"line_number":32,"context_line":"   * Modify the ``WSGIScriptAlias`` directive to point to the"},{"line_number":33,"context_line":"     watcher/api/app.wsgi script."},{"line_number":34,"context_line":"   * Modify the ``Directory`` directive to set the path to the Watcher API"},{"line_number":35,"context_line":"     code."},{"line_number":36,"context_line":"   * Modify the ``ErrorLog and CustomLog`` to redirect the logs to the right"}],"source_content_type":"text/x-rst","patch_set":2,"id":"f8380c32_23a74f1e","side":"PARENT","line":33,"range":{"start_line":33,"start_character":5,"end_line":33,"end_character":25},"updated":"2026-09-15 17:37:04.000000000","message":"I think that this should also be deleted","commit_id":"91ddc626edeb5fd6589eab8cfcae4882453afa7f"}],"doc/source/user/ways-to-install.rst":[{"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":"b8c89b9f481ab4b93e198c3f4b5a9815d3f97b74","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":".. code-block:: bash"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    $ watcher-decision-engine"},{"line_number":88,"context_line":"    $ watcher-applier"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"The Watcher API is a WSGI application and must be served by a WSGI server,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b2e68aab_f21eb5e1","line":87,"updated":"2026-09-15 17:53:31.000000000","message":"The change removes the watcher-api console script and updates the run-the-services section of doc/source/user/ways-to-install.rst, but the same file still tells readers that watcher-api is a command available on the PATH and still ships a Docker example that installs and runs the removed binary. The file is now internally inconsistent: following its own instructions yields a command-not-found error or a broken container entrypoint.\n\n**Severity**: WARNING | **Confidence**: 0.85\n\n**Impact**: A user installing from source following this guide will look for or attempt to run a watcher-api command that no longer exists, and the documented Docker container fails to start because /usr/bin/watcher-api is no longer shipped. Both outcomes directly contradict the WSGI guidance the same change added to this file.\n\n**Suggestion**:\nRemove the watcher-api bullet from the command list at line 64 and update or drop the Debian experimental Docker example (lines 127-135) so it either serves the API via watcher.wsgi.api:application under a WSGI server or is removed with a pointer to the admin WSGI guide (:doc:`../admin/wsgi`).","commit_id":"d1d774b55f06dbf0a7484af127e1d3bed857e519"}],"watcher/conf/api.py":[{"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":"b8c89b9f481ab4b93e198c3f4b5a9815d3f97b74","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    cfg.PortOpt("},{"line_number":34,"context_line":"        \u0027port\u0027,"},{"line_number":35,"context_line":"        default\u003d9322,"},{"line_number":36,"context_line":"        deprecated_for_removal\u003dTrue,"},{"line_number":37,"context_line":"        deprecated_since\u003d\u00272027.1\u0027,"},{"line_number":38,"context_line":"        deprecated_reason\u003d\u0027The standalone Watcher API server has been \u0027"},{"line_number":39,"context_line":"        \u0027removed. The API is now deployed as a WSGI application, so the \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"462e18ce_4f3b36ef","line":36,"updated":"2026-09-15 17:53:31.000000000","message":"The new deprecation metadata marks the four [api] options as deprecated since 2027.1 and the new WSGI doc uses a 2027.1 versionchanged directive, but this change is part of the 2026.2 (Watcher 17.0.0) cycle: the 2026.2 prelude release note was committed on 2026-09-03 and the 17.0.0.0rc1 tag was created on 2026-09-08, one week before this review. oslo_config will emit deprecation warnings naming 2027.1 in a 2026.2 release, misstating when the options were deprecated and when they will be removed.\n\n**Severity**: WARNING | **Confidence**: 0.85\n\n**Impact**: Operators reading deprecation warnings, the generated configuration reference, or the new WSGI doc will believe the [api] options were deprecated in (and the watcher-api script removed in) 2027.1 rather than 2026.2. The six-month discrepancy misleads upgrade planning for deployments that still set these options and cannot be reconciled once the 2026.2 notes ship.\n\n**Suggestion**:\nChange the four deprecated_since values in watcher/conf/api.py (lines 37, 47, 63, 76) and the versionchanged directive in doc/source/admin/wsgi.rst:5 from 2027.1 to 2026.2, unless maintainers deliberately hold this change for the 2027.1 release, in which case it must not merge into the 2026.2 cycle.","commit_id":"d1d774b55f06dbf0a7484af127e1d3bed857e519"}]}