Skip to content

Commit 80fa39f

Browse files
authored
fix: ensure image build process uses specified base directory for context (#178)
1 parent d8481d3 commit 80fa39f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/main/java/com/epam/aidial/deployment/manager/service/pipeline/specification/ImageBuildFromGitJobSpecification.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ private void configureBuilderContainerArgs(MappingChain<Container> builderContai
151151
: WORKSPACE_PATH;
152152
var args = new ArrayList<>(List.of(
153153
"--local",
154-
"context=%s".formatted(WORKSPACE_PATH),
154+
"context=%s".formatted(dockerfile),
155155
"--local",
156156
"dockerfile=%s".formatted(dockerfile)
157157
));

src/test/java/com/epam/aidial/deployment/manager/service/pipeline/specification/ImageBuildFromGitJobSpecificationTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ void getJob_shouldReturnJobWithCorrectBuildkitRootlessConfigurationWhenBaseDirec
221221
List<String> args = buildContainer.getArgs();
222222
assertNotNull(args);
223223
assertTrue(args.stream().anyMatch(arg -> arg.equals("dockerfile=/workspace/src/app")));
224-
assertTrue(args.stream().anyMatch(arg -> arg.equals("context=/workspace")));
224+
assertTrue(args.stream().anyMatch(arg -> arg.equals("context=/workspace/src/app")));
225225
assertFalse(args.stream().anyMatch(arg -> arg.equals("--no-push")));
226226
}
227227

@@ -249,7 +249,7 @@ void getJob_shouldUseCorrectContextWhenBaseDirectoryIsProvided() {
249249
Container buildContainer = getContainerByName(job, BUILDER_CONTAINER_NAME);
250250
List<String> args = buildContainer.getArgs();
251251

252-
assertThat(args).contains("context=/workspace");
252+
assertThat(args).contains("context=/workspace/src/app");
253253
assertThat(args).contains("dockerfile=/workspace/src/app");
254254
}
255255

0 commit comments

Comments
 (0)