Skip to content
This repository was archived by the owner on Jul 30, 2024. It is now read-only.

Commit 5eaf82d

Browse files
authored
Merge pull request #691 from NuGet/ryuyu-filesystemwatcher-leak-fix
Fixing file system watcher leak for PackageLag monitor.
2 parents fb1ddfb + 62bbeff commit 5eaf82d

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/NuGet.Jobs.Common/JsonConfigurationJob.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ private IConfigurationRoot GetConfigurationRoot(string configurationFilename, ou
7676

7777
var builder = new ConfigurationBuilder()
7878
.SetBasePath(Environment.CurrentDirectory)
79-
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: true);
79+
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: false);
8080

8181
var uninjectedConfiguration = builder.Build();
8282

src/NuGet.Services.Validation.Orchestrator/Job.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ private IConfigurationRoot GetConfigurationRoot(string configurationFilename, bo
125125
Logger.LogInformation("Using the {ConfigurationFilename} configuration file", configurationFilename);
126126
var builder = new ConfigurationBuilder()
127127
.SetBasePath(Environment.CurrentDirectory)
128-
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: true);
128+
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: false);
129129

130130
var uninjectedConfiguration = builder.Build();
131131

src/PackageLagMonitor/Job.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ private IConfigurationRoot GetConfigurationRoot(string configurationFilename)
6060
Logger.LogInformation("Using the {ConfigurationFilename} configuration file", configurationFilename);
6161
var builder = new ConfigurationBuilder()
6262
.SetBasePath(Environment.CurrentDirectory)
63-
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: true);
63+
.AddJsonFile(configurationFilename, optional: false, reloadOnChange: false);
6464

6565
var uninjectedConfiguration = builder.Build();
6666

0 commit comments

Comments
 (0)