Merge pull request #556 from danjagnow/features/github-actions-set-cloud-variable-retry

AArnott · web-flow · commit 0637ff096837 · 2021-01-18T10:44:46.000-07:00
Added retries to SetCloudBuildVariable for GitHubActions
diff --git a/src/NerdBank.GitVersioning/CloudBuildServices/GitHubActions.cs b/src/NerdBank.GitVersioning/CloudBuildServices/GitHubActions.cs
@@ -28,7 +28,7 @@ public IReadOnlyDictionary<string, string> SetCloudBuildNumber(string buildNumbe
 
         public IReadOnlyDictionary<string, string> SetCloudBuildVariable(string name, string value, TextWriter stdout, TextWriter stderr)
         {
-            File.AppendAllText(EnvironmentFile, $"{Environment.NewLine}{name}={value}{Environment.NewLine}");
+            Utilities.FileOperationWithRetry(() => File.AppendAllText(EnvironmentFile, $"{Environment.NewLine}{name}={value}{Environment.NewLine}"));
             return GetDictionaryFor(name, value);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public IReadOnlyDictionary<string, string> SetCloudBuildNumber(string buildNumbe`
`28`	`28`
`29`	`29`	`public IReadOnlyDictionary<string, string> SetCloudBuildVariable(string name, string value, TextWriter stdout, TextWriter stderr)`
`30`	`30`	`{`
`31`		`- File.AppendAllText(EnvironmentFile, $"{Environment.NewLine}{name}={value}{Environment.NewLine}");`
	`31`	`+ Utilities.FileOperationWithRetry(() => File.AppendAllText(EnvironmentFile, $"{Environment.NewLine}{name}={value}{Environment.NewLine}"));`
`32`	`32`	`return GetDictionaryFor(name, value);`
`33`	`33`	`}`
`34`	`34`