Maven release v0.9.13

jib-core/src/main/java/com/google/cloud/tools/jib/frontend/JavaDockerContextGenerator.java

@@ -133,8 +133,8 @@ private static String mapToDockerfileString(Map<String, String> map, String comm
   @Nullable private String baseImage;
   @Nullable private List<String> entrypoint;
   @Nullable private List<String> programArguments;
+  @Nullable private Map<String, String> environment;
   @Nullable private String user;
-  private Map<String, String> environment = Collections.emptyMap();
   private List<String> exposedPorts = Collections.emptyList();
   private Map<String, String> labels = Collections.emptyMap();
 
@@ -219,7 +219,7 @@ public JavaDockerContextGenerator setProgramArguments(@Nullable List<String> pro
    * @param environment map from the environment variable name to value
    * @return this
    */
-  public JavaDockerContextGenerator setEnvironment(Map<String, String> environment) {
+  public JavaDockerContextGenerator setEnvironment(@Nullable Map<String, String> environment) {
     this.environment = environment;
     return this;
   }
@@ -336,7 +336,9 @@ String makeDockerfile() throws JsonProcessingException {
       dockerfile.append("\nEXPOSE ").append(port);
     }
 
-    dockerfile.append(mapToDockerfileString(environment, "ENV"));
+    if (environment != null) {
+      dockerfile.append(mapToDockerfileString(environment, "ENV"));
+    }
     dockerfile.append(mapToDockerfileString(labels, "LABEL"));
     if (entrypoint != null) {
       dockerfile.append("\nENTRYPOINT ").append(objectMapper.writeValueAsString(entrypoint));

jib-maven-plugin/CHANGELOG.md

@@ -18,6 +18,12 @@ All notable changes to this project will be documented in this file.
 
 ### Fixed
 
+## 0.9.13
+
+### Fixed
+
+- Adds environment variable configuration to Docker context generator ([#890 (comment)](https://github.com/GoogleContainerTools/jib/issues/890#issuecomment-430227555))
+
 ## 0.9.11
 
 ### Added

jib-maven-plugin/pom.xml

@@ -3,7 +3,7 @@
 
   <groupId>com.google.cloud.tools</groupId>
   <artifactId>jib-maven-plugin</artifactId>
-  <version>0.9.12-SNAPSHOT</version>
+  <version>0.9.14-SNAPSHOT</version>
   <packaging>maven-plugin</packaging>
 
   <name>Jib</name>

jib-maven-plugin/src/main/java/com/google/cloud/tools/jib/maven/DockerContextMojo.java

@@ -88,6 +88,7 @@ public void execute() throws MojoExecutionException {
           .setEntrypoint(entrypoint)
           .setProgramArguments(getArgs())
           .setExposedPorts(getExposedPorts())
+          .setEnvironment(getEnvironment())
           .setLabels(getLabels())
           .setUser(getUser())
           .generate(Paths.get(targetDir));

jib-maven-plugin/src/test/java/com/google/cloud/tools/jib/maven/DockerContextMojoTest.java

@@ -17,11 +17,13 @@
 package com.google.cloud.tools.jib.maven;
 
 import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableMap;
 import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.List;
+import java.util.Map;
 import javax.annotation.Nullable;
 import org.apache.maven.model.Build;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -119,6 +121,11 @@ public void testGenerateDockerContext_errorOnWindowsAppRootWithDriveLetter() {
   }
 
   @Test
+  public void testGeneratedDockerContext_env() throws MojoExecutionException, IOException {
+    mojo.execute();
+    Assert.assertEquals("ENV envKey=\"envVal\"", getDockerfileLine("ENV"));
+  }
+
   public void testBaseImage_nonWarPackaging() throws MojoExecutionException, IOException {
     mojo.execute();
 
@@ -234,6 +241,11 @@ String getMainClass() {
     String getAppRoot() {
       return appRoot;
     }
+
+    @Override
+    Map<String, String> getEnvironment() {
+      return ImmutableMap.of("envKey", "envVal");
+    }
   }
 
   @Nullable