Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should the snap offer the option to use APCu for local cache? #2663

Open
pachulo opened this issue Jan 27, 2024 · 10 comments
Open

Should the snap offer the option to use APCu for local cache? #2663

pachulo opened this issue Jan 27, 2024 · 10 comments
Labels
feature New feature or request

Comments

@pachulo
Copy link
Member

pachulo commented Jan 27, 2024

After reading the Recommendations based on type of deployment on the Memory caching page of the nextcloud manual, I was wondering:

  • Should the snap come with the PHP APCu extension so that it can be used for memcache.local instead of Redis?

I haven't found any real benchmarks or anything, just that it should be faster but consume more RAM, but as it is also the official recommendation, I just wanted to understand better why we use Redis instead.

@scubamuc
Copy link
Member

@pachulo we have a large user base running "limited memory" devices (SBC's) so this should definitely be optional.

I haven't found any real benchmarks or anything, just that it should be faster but consume more RAM, but as it is also the official recommendation, I just wanted to understand better why we use Redis instead.

According to the recommendation, it could increase performance on "enhanced memory" machines. That would be desirable.

@kyrofa
Copy link
Member

kyrofa commented Jan 29, 2024

Yeah, it makes sense that we'd be using redis given our start on the Nextcloud box. However, while I agree with @scubamuc that continuing to support such users is important, I just checked the metrics this morning: literally 95% of our users are amd64. If this would enhance performance for those users, it might be worth investigating as long as we can figure out a way for it to not hurt the other 5%.

@pachulo
Copy link
Member Author

pachulo commented Jan 29, 2024

OK, I found more information:

Could we configure this based on available RAM?🤔

P.D.
Should we also look to move to NGINX based on this comment @kyrofa ? 😝

@kyrofa
Copy link
Member

kyrofa commented Jan 29, 2024

Should we also look to move to NGINX based on this comment @kyrofa ? 😝

No, only Apache is supported:

Please note that webservers other than Apache 2.x are not officially supported.

That's the reason we went this direction in the first place. NGINX is definitely lighter-weight. That's one of the reasons we've been so careful about not opening up Apache configs: we can switch to NGINX in the future with no one being the wiser if we feel it necessary/worthwhile.

@scubamuc
Copy link
Member

checked the metrics this morning: literally 95% of our users are amd64

Only 5% running "limited memory" devices (SBC's), interesting...

we have a large user base running "limited memory" devices (SBC's)

I would be remiss if I hadn't mentioned these users?

@pachulo
Copy link
Member Author

pachulo commented Feb 4, 2024

@pachulo we have a large user base running "limited memory" devices (SBC's) so this should definitely be optional.

I'm one of such users, so I can't agree more! 😄

Could we configure this based on available RAM?🤔

What do you think would make more sense @kyrofa :

  1. to make it an autoconfig, depending on the amount of RAM?;
  2. or to create a manual knob?

@alfi4000
Copy link

alfi4000 commented Feb 13, 2024

Um I am a bit confused I am totally unfamiliar with those stuff so is there a way to tune the server performance because I have 256gb ram 24 core but I had closely a crash on my device that was accessing the Nextcloud instance that is on the high performance server it were looking for me that the access device loads the hole stuff not the server where Nextcloud were running on especially when I uploaded 361 images at once my device from where I was uploading were stuck for 20sec! That was mind blowing for me ! :/
Is there something misconfigured on my instance or what is there going on any advice what I can do to improve the performance I have currently 400gb storage in Use and plan to add 1.5tb of data!

@alfi4000
Copy link

Any advice would be great! :)

@scubamuc
Copy link
Member

@alfi4000, you have a powerful machine by the sounds of it...

this PR is a development discussion and is not the right PR for your issue... I guess. @pachulo is considering some default memory cache changes for the Snap.

as far as your performance issues are concerned, its not as simple as tuning here or there... you'd have to study your system logs and performance monitors carefully to pinpoint the bottleneck. That's admin stuff and not necessarily a Nextcloud snap issue. there may be some knobs to turn if we knew more about your system. but we're trying to make Nextcloud snap work for everyone "out of the box". So we're more concerned with stability than performance... especially since we don't know what device is being used.

basically advice for system tuning your machine would be going way beyond what we're trying to do here... also this PR would be wrong for questions about your device. So I'd suggest you make a new PR with your performance issue.

@alfi4000
Copy link

Ok

@pachulo pachulo added the feature New feature or request label Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants