Skip to content

Commit

Permalink
fix(runner): main container only has dependency if there is some inpu…
Browse files Browse the repository at this point in the history
…tFiles
  • Loading branch information
brian-mulier-p committed Mar 27, 2024
1 parent afcc5f7 commit 65db8b0
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -326,12 +326,15 @@ public RunnerResult run(RunContext runContext, ScriptCommands commands, List<Str
.options(Map.of("awslogs-stream-prefix", jobName))
.build()
)
.dependsOn(TaskContainerDependency.builder().containerName(inputFilesContainerName).condition("SUCCESS").build())
.essential(!hasFilesToDownload),
Integer.parseInt(resources.getRequest().getMemory()) - sideContainersMemoryAllocations,
Float.parseFloat(resources.getRequest().getCpu()) - sideContainersCpuAllocations
);

if (hasFilesToUpload) {
mainContainerBuilder.dependsOn(TaskContainerDependency.builder().containerName(inputFilesContainerName).condition("SUCCESS").build());
}

if (commands.getEnv() != null) {
mainContainerBuilder
.environment(
Expand Down

0 comments on commit 65db8b0

Please sign in to comment.