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

Better to poll on alpine or in containers to prevent issues inotify exceptions #61

Open
wants to merge 1 commit into
base: master
from

Conversation

3 participants
@shanselman

shanselman commented Aug 29, 2018

Better to poll on alpine or in containers to prevent issues like "inotify exceptions user limit (8192) on the number of inotify instances has been reached." with the .NET file watcher

better to poll on alpine or in containers to prevent issues with 'The…
… configured user limit (8192) on the number of inotify instances has been reached.'
@now

This comment has been minimized.

Show comment
Hide comment
@now

now bot Aug 29, 2018

This pull request is automatically deployed with Now.

To access deployments, click Details below or on the icon next to each push.

now bot commented Aug 29, 2018

This pull request is automatically deployed with Now.

To access deployments, click Details below or on the icon next to each push.

@rauchg

This comment has been minimized.

Show comment
Hide comment
@rauchg

rauchg Sep 1, 2018

Contributor

@TooTallNate what's the status of "inotify-gate"?

Contributor

rauchg commented Sep 1, 2018

@TooTallNate what's the status of "inotify-gate"?

@TooTallNate

This comment has been minimized.

Show comment
Hide comment
@TooTallNate

TooTallNate Sep 12, 2018

Member

Status is fixed by my testing. @shanselman if you could confirm that would be great 🙏

Member

TooTallNate commented Sep 12, 2018

Status is fixed by my testing. @shanselman if you could confirm that would be great 🙏

@shanselman

This comment has been minimized.

Show comment
Hide comment
@shanselman

shanselman Sep 14, 2018

Looking at this, but I can't get the deployment to succeed. https://superzeit-cpycxbeqpq.now.sh/ I'll try again in the morning.

09/14 01:10 AM (7m) Deploying image
09/14 01:10 AM (6m) Container started
09/14 01:10 AM (6m) Deploying image
09/14 01:10 AM (6m) Container started
09/14 01:11 AM (6m) Deploying image
09/14 01:11 AM (6m) Container started
09/14 01:11 AM (5m) Deploying image
09/14 01:12 AM (5m) Container started
09/14 01:12 AM (4m) Deploying image
09/14 01:12 AM (4m) Container started
09/14 01:14 AM (3m) Deploying image
09/14 01:14 AM (3m) Container started
09/14 01:17 AM (6s) Deploying image

shanselman commented Sep 14, 2018

Looking at this, but I can't get the deployment to succeed. https://superzeit-cpycxbeqpq.now.sh/ I'll try again in the morning.

09/14 01:10 AM (7m) Deploying image
09/14 01:10 AM (6m) Container started
09/14 01:10 AM (6m) Deploying image
09/14 01:10 AM (6m) Container started
09/14 01:11 AM (6m) Deploying image
09/14 01:11 AM (6m) Container started
09/14 01:11 AM (5m) Deploying image
09/14 01:12 AM (5m) Container started
09/14 01:12 AM (4m) Deploying image
09/14 01:12 AM (4m) Container started
09/14 01:14 AM (3m) Deploying image
09/14 01:14 AM (3m) Container started
09/14 01:17 AM (6s) Deploying image
@shanselman

This comment has been minimized.

Show comment
Hide comment
@shanselman

shanselman Sep 14, 2018

Got it! Works! Thanks @TooTallNate @jamo

You can merge this PR now if you want.

shanselman commented Sep 14, 2018

Got it! Works! Thanks @TooTallNate @jamo

You can merge this PR now if you want.

@jamo jamo requested a review from TooTallNate Sep 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment