shwstppr commented on issue #8442:
URL: https://github.com/apache/cloudstack/issues/8442#issuecomment-2102152699
@kiranchavala and others If I'm understanding it right I guess we are mixing
listall functionality with displaynetwork functionality.
`displaynetwork` param description says `an optional field, whether to the
display the network to the end user or not.`
It is an admin only parameter to allow making network not viewable to the
end user.
Later admin can list such networks using API, passing the same network:
```
> list networks displaynetwork=false filter=name,displaynetwork,
{
"count": 1,
"network": [
{
"displaynetwork": false,
"name": "hidden-nw"
}
]
}
```
Also, Kiran there is no param `name` in listNetworks API.
IMO, list* apis should list all resources **for root admins** irrespective
of the value of displaynetwork, displayvm, displayvolume fields but we have
prevented that for all types of accounts for a long,
https://github.com/apache/cloudstack/commit/bbf5a912c61c42587f4403ea6f64301d421d3204
--
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]
