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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
