spreed icon indicating copy to clipboard operation
spreed copied to clipboard

Show user statuses in Nextcloud Talk channels with more than 1000 users

Open vladimirdulov opened this issue 3 years ago • 3 comments

NextCloud Talk shows user statuses in a channel only when there are participants less than 100. The are no settings to change it as It's hardcoded in php and js code.

https://github.com/nextcloud/spreed/blob/70335c2290bad8196a94232e3c02e81422d8672e/src/components/RightSidebar/Participants/ParticipantsList/ParticipantsList.vue#L72

https://github.com/nextcloud/spreed/blob/70335c2290bad8196a94232e3c02e81422d8672e/lib/Controller/RoomController.php#L936

Can it be amended to have an Admin setting input added to NextCloud Talk settings page?

Currently we are patching the above files but it's not an upgrade safe.

Nextcloud version 24.0.2

vladimirdulov avatar Aug 01 '22 05:08 vladimirdulov

There should be a negative performance impact, especially when everyone joins the room at a similar time e.g. for a scheduled meeting. Do you not observe something like this? We might be able to raise the value, but not sure it is a good idea to allow customizing it.

nickvergessen avatar Aug 01 '22 08:08 nickvergessen

We increased the value up to 200 and haven't noticed any performance impacts.

We run Nextcloud on 32gb host with the following apache mpm_prefork settings, all quite enough fast.

<IfModule mpm_prefork_module>
        StartServers 4
        MinSpareServers 3
        MaxSpareServers 40
        MaxRequestWorkers 200
        MaxConnectionsPerChild 10000
</IfModule>

vladimirdulov avatar Aug 01 '22 10:08 vladimirdulov