Skip to content

Commit befc434

Browse files
refactor: more layering fixes (#34)
1 parent f885229 commit befc434

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

hypertrace-gradle-docker-java-application-plugin/src/main/java/org/hypertrace/gradle/docker/HypertraceDockerJavaApplicationPlugin.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -112,12 +112,6 @@ void createDockerfileTask(Project project, HypertraceDockerJavaApplication javaA
112112
dockerfile.setGroup(DockerPlugin.TASK_GROUP);
113113
dockerfile.setDescription("Creates a Dockerfile for the java application");
114114
dockerfile.from(javaApplication.baseImage.map(From::new));
115-
dockerfile.arg(COMMIT_SHA_BUILD_ARG + "=unknown");
116-
String commitShaArgReference = "${" + COMMIT_SHA_BUILD_ARG + "}";
117-
dockerfile.label(singletonMap("commit_sha", commitShaArgReference));
118-
dockerfile.environmentVariable("COMMIT_SHA", commitShaArgReference);
119-
dockerfile.label(javaApplication.maintainer.map(maintainer -> singletonMap("maintainer", maintainer)));
120-
dockerfile.workingDir("/app");
121115
dockerfile.copyFile(relativeScriptPath.map(relativePath -> new CopyFile(relativePath, "run")));
122116
dockerfile.copyFile(provideIfDirectoryExists(dockerfile.getDestDir()
123117
.map(dir -> dir.dir(DOCKER_BUILD_CONTEXT_EXTERNAL_LIBS_DIR)))
@@ -132,6 +126,12 @@ void createDockerfileTask(Project project, HypertraceDockerJavaApplication javaA
132126
.map(dir -> dir.dir(DOCKER_BUILD_CONTEXT_RESOURCES_DIR)))
133127
.map(unused -> new CopyFile(DOCKER_BUILD_CONTEXT_RESOURCES_DIR, DOCKER_BUILD_CONTEXT_RESOURCES_DIR)));
134128
dockerfile.copyFile(new CopyFile(DOCKER_BUILD_CONTEXT_CLASSES_DIR, DOCKER_BUILD_CONTEXT_CLASSES_DIR));
129+
dockerfile.arg(COMMIT_SHA_BUILD_ARG + "=unknown");
130+
String commitShaArgReference = "${" + COMMIT_SHA_BUILD_ARG + "}";
131+
dockerfile.label(singletonMap("commit_sha", commitShaArgReference));
132+
dockerfile.environmentVariable("COMMIT_SHA", commitShaArgReference);
133+
dockerfile.label(javaApplication.maintainer.map(maintainer -> singletonMap("maintainer", maintainer)));
134+
dockerfile.workingDir("/app");
135135
dockerfile.instruction(javaApplication.healthCheck);
136136
dockerfile.environmentVariable(javaApplication.envVars);
137137
dockerfile.entryPoint("./run");

0 commit comments

Comments
 (0)