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

Process for host instance notification shouldn't rely on file watchers #10861

Open
mattchenderson opened this issue Feb 19, 2025 · 0 comments
Open

Comments

@mattchenderson
Copy link
Contributor

This item is to track investigation into alternative methods for coordination across host instances, such as a queue based model, etc.

Today, file-based sentinels are used, but this has a few issues:

  1. This can be missed in situations where the assumption of a shared file system isn't upheld. That can manifest as a silent failure for the scenario, though the impact would be lessened by other coordinated recycle events such as configuration changes to the resource.
  2. Using file-based also creates a requirement on a mutable filesystem being present, which limits hosting environment options.
  3. File watchers have a material perf impact. We should strive to move away from them.

An example scenario can be seen with the default secrets repository:

string filePath = GetSecretsSentinelFilePath(type, functionName);
await FileUtility.WriteAsync(filePath, DateTime.UtcNow.ToString());

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

No branches or pull requests

1 participant