GitHub user shwstppr added a comment to the discussion: Provide the option of 
disabling/Enabling Shared/Isolated/L2 networks,

@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

GitHub link: 
https://github.com/apache/cloudstack/discussions/9162#discussioncomment-9643964

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to