Skip to content

Commit 5369dee

Browse files
authored
Merge pull request #8 from SimCubeLtd/dev
Fix Helm Rendering of Values
2 parents 9669832 + 9457b6b commit 5369dee

File tree

5 files changed

+16
-20
lines changed

5 files changed

+16
-20
lines changed

.github/workflows/pull-requests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ jobs:
1717
with:
1818
fetch-depth: 0
1919

20-
- uses: actions/setup-dotnet@v2
20+
- uses: actions/setup-dotnet@v3
2121
with:
2222
dotnet-version: '6.0.x'
2323
include-prerelease: false

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jobs:
2121
with:
2222
fetch-depth: 0
2323

24-
- uses: actions/setup-dotnet@v2
24+
- uses: actions/setup-dotnet@v3
2525
with:
2626
dotnet-version: '6.0.x'
2727
include-prerelease: false

src/SimCube.PulumiDeployments/GlobalUsings.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,4 +20,6 @@
2020
global using SimCube.PulumiDeployments.Helpers;
2121
global using SimCube.PulumiDeployments.Literals;
2222
global using SimCube.PulumiDeployments.Resources.Helm;
23-
global using SimCube.PulumiDeployments.Resources.Kubernetes;
23+
global using SimCube.PulumiDeployments.Resources.Kubernetes;
24+
global using CliWrap;
25+
global using CliWrap.Buffered;

src/SimCube.PulumiDeployments/Resources/Helm/BaseHelmChartResource.cs

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -28,23 +28,16 @@ protected BaseHelmChartResource(
2828

2929
protected static string RenderCommandName(string chartName) => $"render-values-{chartName}";
3030

31-
protected Command RenderYamlValues(string commandName, string helmFile, InputMap<string> environment)
32-
{
33-
var createCommand = new Command(
34-
commandName,
35-
new()
36-
{
37-
Create = RenderCreateCommand(helmFile),
38-
Environment = environment,
39-
},
40-
CustomResourceOptions);
41-
42-
CustomResourceOptions.DependsOn.Add(createCommand);
43-
44-
return createCommand;
45-
}
31+
protected static void RenderYamlValues(string helmFile, Dictionary<string, string?> environment) =>
32+
Cli.Wrap(EnvSubstitute)
33+
.WithArguments($"< {helmFile} > {helmFile}.new && mv {helmFile}.new {helmFile}")
34+
.WithEnvironmentVariables(environment)
35+
.WithValidation(CommandResultValidation.ZeroExitCode)
36+
.ExecuteAsync()
37+
.GetAwaiter()
38+
.GetResult();
4639

4740
protected string GetHelmValuesFilePath() => Path.Combine(AppContext.BaseDirectory, HelmValuesFolder, HelmValuesFile);
4841

49-
private static string RenderCreateCommand(string helmValuePath) => $"{EnvSubstitute} < {helmValuePath} > {helmValuePath}.new && mv {helmValuePath}.new {helmValuePath}";
50-
}
42+
private static string RenderCreateCommand(string helmValuePath) => $"{EnvSubstitute} ";
43+
}

src/SimCube.PulumiDeployments/SimCube.PulumiDeployments.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515

1616
<ItemGroup>
1717
<PackageReference Include="Ardalis.GuardClauses" Version="4.0.1" />
18+
<PackageReference Include="CliWrap" Version="3.5.0" />
1819
<PackageReference Include="FluentValidation" Version="11.2.2" />
1920
<PackageReference Include="Pulumi.Kubernetes" Version="3.21.4" />
2021
<PackageReference Include="Pulumi.Command" Version="4.5.0" />

0 commit comments

Comments
 (0)