Fix ImageChange triggers not being set in DeploymentConfig when resou…

…rce fragments are used.

Related StackOverlow post: https://stackoverflow.com/questions/60375063

This seems to be a regression since we introduced sidecar containers support. For applying
ImageChange triggers we have to check which containers are main application containers and
which are sidecars. Well, we do that but when we don't set any `defaultApplicationContainerName`
when sidecar option is disabled. It seemed like a bug introduced my me during implementation.

CHANGELOG.md

@@ -26,6 +26,7 @@ Usage:
 * Fix #1789: script to extract changelog information for notifications
 * Fix #1770: Fixed my mistake, remove constructors from OpenshiftBuildConfig which were causing PluginConfigurationException
 * Fix #1791: Fix NullPointerException in FileDataSecretEnricher (due to recent change in Kubernetes Client Builder null to LinkedHashMap conversion)
+* Fix: ImageChangeTags not being applied in DeploymentConfig generated by plugin in case of resource fragments. (Issue found in [this](https://stackoverflow.com/questions/60375063) StackOverflow post.)
 
 ### 4.4.0 (2020-02-13)
 * Fix #1572: Support maven --batch-mode option

core/src/main/java/io/fabric8/maven/core/util/kubernetes/KubernetesResourceUtil.java

@@ -723,6 +723,8 @@ public static String mergePodSpec(PodSpecBuilder builder, PodSpec defaultPodSpec
  }
 
  public static String mergePodSpec(PodSpecBuilder builder, PodSpec defaultPodSpec, String defaultName, boolean sidecarEnabled) {
+ // The default application container name is needed by plugin in order
+ // to add ImageChange triggers in case of DeploymentConfig
  String defaultApplicationContainerName = null;
  List<Container> containers = builder.buildContainers();
  List<Container> defaultContainers = defaultPodSpec.getContainers();
@@ -753,6 +755,11 @@ public static String mergePodSpec(PodSpecBuilder builder, PodSpec defaultPodSpec
  container = new Container();
  containers.add(container);
  }
+ // If default container name is not set, add first found
+ // container as default application container.
+ if (defaultApplicationContainerName == null) {
+ defaultApplicationContainerName = container.getName();
+ }
  }
 
  mergeSimpleFields(container, defaultContainer);