[PR] Only list workers of readable topologies on the supervisor page (storm)

["rzo1 (via GitHub)" <gi...@apache.org>]

rzo1 opened a new pull request, #9095:
URL: https://github.com/apache/storm/pull/9095

   `getSupervisorPageInfo` returned a worker summary for every topology assigned to the supervisor. For topologies the caller cannot read, it only left out `componentNumTasks`. Worker placement is otherwise only returned by `getTopologyInfo` and `getTopologyPageInfo`, which require read access to the topology. The supervisor page now matches them and omits the workers of topologies the caller is not allowed to read. It uses the existing `filterAuthorized("getTopology", ...)` check, and topologies that fail it are no longer loaded at all.
   
   The supervisor summary on the same page still reports the node's aggregate slot, memory and CPU usage, so the capacity view is unchanged. Admins and users with read access see the same page as before.
   
   `docs/STORM-UI-REST-API.md` now says the `workers` list is limited to readable topologies.
   
   Extends `NimbusClojurePortTest`. The existing supervisor page test now also asserts the returned worker, and a new test covers a caller without read access.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Only list workers of readable topologies on the supervisor page (storm)

["rzo1 (via GitHub)" <gi...@apache.org>]

rzo1 merged PR #9095:
URL: https://github.com/apache/storm/pull/9095


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@storm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org