Skip to content

Commit

Permalink
Revert "add /tmp emptyDir volume to connector pods (#10761)" (#11053)
Browse files Browse the repository at this point in the history
This reverts commit 921f4a1.
  • Loading branch information
pmossman committed Mar 11, 2022
1 parent 99338c8 commit eea5156
Showing 1 changed file with 2 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,6 @@ public class KubePodProcess extends Process implements KubePod {
private static final String STDOUT_PIPE_FILE = PIPES_DIR + "/stdout";
private static final String STDERR_PIPE_FILE = PIPES_DIR + "/stderr";
public static final String CONFIG_DIR = "/config";
public static final String TMP_DIR = "/tmp";
private static final String TERMINATION_DIR = "/termination";
private static final String TERMINATION_FILE_MAIN = TERMINATION_DIR + "/main";
private static final String TERMINATION_FILE_CHECK = TERMINATION_DIR + "/check";
Expand Down Expand Up @@ -427,24 +426,13 @@ public KubePodProcess(final boolean isOrchestrator,
.withMountPath(TERMINATION_DIR)
.build();

final Volume tmpVolume = new VolumeBuilder()
.withName("tmp")
.withNewEmptyDir()
.endEmptyDir()
.build();

final VolumeMount tmpVolumeMount = new VolumeMountBuilder()
.withName("tmp")
.withMountPath(TMP_DIR)
.build();

final Container init = getInit(usesStdin, List.of(pipeVolumeMount, configVolumeMount), busyboxImage);
final Container main = getMain(
image,
imagePullPolicy,
usesStdin,
entrypointOverride,
List.of(pipeVolumeMount, configVolumeMount, terminationVolumeMount, tmpVolumeMount),
List.of(pipeVolumeMount, configVolumeMount, terminationVolumeMount),
resourceRequirements,
internalToExternalPorts,
envMap,
Expand Down Expand Up @@ -512,7 +500,7 @@ public KubePodProcess(final boolean isOrchestrator,
.withRestartPolicy("Never")
.withInitContainers(init)
.withContainers(containers)
.withVolumes(pipeVolume, configVolume, terminationVolume, tmpVolume)
.withVolumes(pipeVolume, configVolume, terminationVolume)
.endSpec()
.build();

Expand Down

0 comments on commit eea5156

Please sign in to comment.