Skip to content

DLPX-91653 Use default inotify.max_user_watches #490

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

Conversation

david-mendez1
Copy link
Contributor

@david-mendez1 david-mendez1 commented Jul 16, 2024

Problem

Customers still occasionally hit the inotify.max_user_watches limit even at the max setting of 32768. Use default inotify.max_user_watches now that the following Linux change has increased the max_user_watches based on the amount of addressable memory available.

inotify: Increase default inotify.max_user_watches limit to 1048576 · torvalds/linux@9289012

Solution

Remove hard coded max value for fs.inotify.max_user_watches in 50-override.conf.

Testing Done

ab-pre-push
http://selfservice.jenkins.delphix.com/job/appliance-build-orchestrator-pre-push/8954/

Test different instance sizes:

david.mendez@dlpxdc:~$ dc list -o size sync-vm-8a5cd044-6619-0
SIZE
T3_LARGE
delphix@ip-10-110-194-224:~$ cat /proc/sys/fs/inotify/max_user_watches 
56783

david.mendez@dlpxdc:~$ dc list -o size,source_group dm-test-aws
SIZE                 SOURCE_GROUP
DE_CERT_R5N_8XLARGE  temp-sync-AWS-image
elphix@ip-10-110-217-97:~$ cat /proc/sys/fs/inotify/max_user_watches 
1048576

@david-mendez1 david-mendez1 force-pushed the dlpx/pr/david-mendez1/57dd1aba-b602-42b2-9dca-78a659c186f2 branch from 91f4e8a to 5624efb Compare July 16, 2024 21:25
@david-mendez1 david-mendez1 marked this pull request as ready for review July 16, 2024 22:26
Copy link
Contributor

@palash-gandhi palash-gandhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, we may want to give the scale & perf folks a heads up to keep an eye out for unusual memory trends.

@david-mendez1 david-mendez1 enabled auto-merge July 17, 2024 23:15
@david-mendez1 david-mendez1 merged commit ade7e87 into develop Jul 17, 2024
15 checks passed
@david-mendez1 david-mendez1 deleted the dlpx/pr/david-mendez1/57dd1aba-b602-42b2-9dca-78a659c186f2 branch July 17, 2024 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants