Merge pull request #126 from xdev-software/v2

V2

Author: AB-xdev

CHANGELOG.md

@@ -1,3 +1,29 @@
+# 2.0.0
+* Changed ignore backend to utilize [JGit](https://github.com/eclipse-jgit/jgit) 
+    * This should now behave exactly like a ``.gitignore``
+    * Overall performance should be a lot faster
+* Make it possible to modify transferred files
+* Provide an option to emulate [``COPY --parents``](https://docs.docker.com/reference/dockerfile/#copy---parents) (which is currently not supported by Docker out of the box)
+    * This option is required to utilize Docker's cache properly
+    ```docker
+    # ...
+    
+    # Copy & Cache wrapper
+    COPY --parents mvnw .mvn/** ./
+    RUN ./mvnw --version
+
+    # Copy & Cache poms/dependencies
+    COPY --parents **/pom.xml ./
+    # Resolve jars so that they can be cached and don't need to be downloaded when a Java file changes
+    RUN ./mvnw -B dependency:go-offline -pl app -am -DincludeScope=runtime -T2C
+
+    # Copying all other files
+    COPY . ./
+    # Run the actual build
+    RUN ./mvnw -B clean package -pl app -am -T2C -Dmaven.test.skip
+    ```
+* At ton of minor optimizations and improvements
+
 # 1.2.0
 * Provide an option to always transfer specific paths
 * Always transfer Dockerfile - as it is required for building - by default

README.md

@@ -8,10 +8,12 @@ A re-implementation of [Testcontainers Image-Builder](https://java.testcontainer
 * Allows passing a custom logger to the image build - [testcontainers-java#3093](https://github.com/testcontainers/testcontainers-java/issues/3093)
 * Allows using ``ARG``s for ``FROM`` - [testcontainers-java#3238](https://github.com/testcontainers/testcontainers-java/issues/3238)
 * Brings a custom [build-context](https://docs.docker.com/build/building/context/) processor
-  * Works more efficient and reliable than the default implementation (however likely still not perfect)
+  * Works more efficient and reliable than the default implementation (utilizes [JGit](https://github.com/eclipse-jgit/jgit))
   * uses ``.gitignore`` if available
   * Allows adding custom ignores
     * This way the build-context can be fine tuned in a way that the build cache works very efficiently (e.g. only re-built when actual code that matters changes)
+  * Makes it possible to modify files that are transferred
+* Provide a compatibility layer to emulate [``COPY --parents``](https://docs.docker.com/reference/dockerfile/#copy---parents) (which is currently not supported by Docker out of the box)
 * Do not pull images that are declared inside the Dockerfile
 * Makes logger non generic and therefore controllable
 * Did some general code cleanup

pom.xml

@@ -6,7 +6,7 @@
 
 	<groupId>software.xdev</groupId>
 	<artifactId>testcontainers-advanced-imagebuilder-root</artifactId>
-	<version>1.2.1-SNAPSHOT</version>
+	<version>2.0.0-SNAPSHOT</version>
 	<packaging>pom</packaging>
 
 	<organization>

testcontainers-advanced-imagebuilder-demo/Dockerfile

@@ -1,34 +1,36 @@
+# syntax=docker/dockerfile:1-labs
 # Stage 1: Build the dummy app
 ARG JAVA_VERSION=21
 FROM eclipse-temurin:$JAVA_VERSION-jdk-alpine AS build-env
 
 RUN apk add --no-cache git bash
 
-# Create non root user
-ARG userName=limitedbuild
-ARG groupName=limitedbuild
-ARG userId=1000
-
-RUN addgroup --system ${groupName} \
-	&& adduser --uid ${userId} --system --disabled-password --shell /bin/bash ${userName} \
-	&& adduser ${userName} ${groupName}
-
-# Create build dir
-RUN mkdir /build \
-    && chown ${userName}:${groupName} /build
 WORKDIR /build
 
-USER ${userName}
-
-# Copying context is prepared by Testcontainers
-COPY --chown=${userName}:${groupName} . ./
-
-# RUN chmod +x ./mvnw
-ARG mvncmd='clean package -pl "testcontainers-advanced-imagebuilder-dummy-app" -am -T2C -Dmaven.test.skip'
-
-RUN echo "Executing '$mvncmd'"
-RUN chmod +x ./mvnw \
-  && ./mvnw -B ${mvncmd}
+# Copy & Cache wrapper
+COPY --parents mvnw .mvn/** ./
+RUN ./mvnw --version
+
+# Copy & Cache poms/dependencies
+COPY --parents **/pom.xml ./
+# Resolve jars so that they can be cached and don't need to be downloaded when a Java file changes
+ARG MAVEN_GO_OFFLINE_COMMAND='./mvnw -B dependency:go-offline -pl "testcontainers-advanced-imagebuilder-dummy-app" -am -DincludeScope=runtime -T2C'
+RUN echo "Executing '$MAVEN_GO_OFFLINE_COMMAND'"
+RUN ${MAVEN_GO_OFFLINE_COMMAND}
+
+# Copying all other files
+COPY . ./
+
+# A valid Git repo is required for the build
+RUN git config --global user.email "[email protected]" \
+  && git config --global user.name "Dynamic Build" \
+  && git init --initial-branch=dynamically-built-tcst \
+  && git add . \
+  && git commit -m "Init commit"
+
+ARG MAVEN_BUILD_COMMAND='./mvnw -B clean package -pl "testcontainers-advanced-imagebuilder-dummy-app" -am -T2C -Dmaven.test.skip'
+RUN echo "Executing '$MAVEN_BUILD_COMMAND'"
+RUN ${MAVEN_BUILD_COMMAND}
 
 # Stage 2: Build the executable image
 FROM eclipse-temurin:21-jre-alpine

testcontainers-advanced-imagebuilder-demo/pom.xml

@@ -7,11 +7,11 @@
 	<parent>
 		<groupId>software.xdev</groupId>
 		<artifactId>testcontainers-advanced-imagebuilder-root</artifactId>
-		<version>1.2.1-SNAPSHOT</version>
+		<version>2.0.0-SNAPSHOT</version>
 	</parent>
 
 	<artifactId>testcontainers-advanced-imagebuilder-demo</artifactId>
-	<version>1.2.1-SNAPSHOT</version>
+	<version>2.0.0-SNAPSHOT</version>
 	<packaging>jar</packaging>
 
 	<organization>

testcontainers-advanced-imagebuilder-demo/src/main/java/software/xdev/Application.java

@@ -5,6 +5,7 @@
 import org.slf4j.LoggerFactory;
 
 import software.xdev.testcontainers.imagebuilder.AdvancedImageFromDockerFile;
+import software.xdev.testcontainers.imagebuilder.compat.DockerfileCOPYParentsEmulator;
 
 
 public final class Application
@@ -14,7 +15,7 @@ public static void main(final String[] args)
 	{
 		final AdvancedImageFromDockerFile builder = new AdvancedImageFromDockerFile("dynamically-built")
 			.withLoggerForBuild(LoggerFactory.getLogger("container.build"))
-			.withAdditionalIgnoreLines(
+			.withPostGitIgnoreLines(
 				// Ignore files that aren't related to the built code
 				".git/**",
 				".config/**",
@@ -26,11 +27,11 @@ public static void main(final String[] args)
 				"/renovate.json5",
 				// We need to keep the pom.xml as maven can't resolve the modules otherwise
 				"testcontainers-advanced-imagebuilder/src/**",
-				"testcontainers-advanced-imagebuilder/test/**",
 				"testcontainers-advanced-imagebuilder-demo/src/**"
 			)
 			.withDockerFilePath(Paths.get("../testcontainers-advanced-imagebuilder-demo/Dockerfile"))
-			.withBaseDir(Paths.get("../"));
+			.withBaseDir(Paths.get("../"))
+			.withDockerFileLinesModifier(new DockerfileCOPYParentsEmulator());
 
 		final String imageName = builder.get();
 

testcontainers-advanced-imagebuilder-dummy-app/pom.xml

@@ -6,7 +6,7 @@
 
 	<groupId>software.xdev</groupId>
 	<artifactId>testcontainers-advanced-imagebuilder-dummy-app</artifactId>
-	<version>1.2.1-SNAPSHOT</version>
+	<version>2.0.0-SNAPSHOT</version>
 	<packaging>jar</packaging>
 
 	<organization>

testcontainers-advanced-imagebuilder/pom.xml

@@ -6,7 +6,7 @@
 
 	<groupId>software.xdev</groupId>
 	<artifactId>testcontainers-advanced-imagebuilder</artifactId>
-	<version>1.2.1-SNAPSHOT</version>
+	<version>2.0.0-SNAPSHOT</version>
 	<packaging>jar</packaging>
 
 	<name>testcontainers-advanced-imagebuilder</name>