Merge pull request #425 from keboola/odin-container-id-fix

fix: container id/name generation
keboola · May 11, 2019 · 38700b1 · 38700b1
2 parents 3f241bf + fd72a9c
commit 38700b1
Showing 1 changed file with 13 additions and 10 deletions.
diff --git a/src/Docker/Runner.php b/src/Docker/Runner.php
@@ -472,16 +472,19 @@ private function runImages($jobId, $configId, $rowId, Component $component, Usag
             ];
             $configFile->createConfigFile($image->getConfigData(), $outputFilter);
 
-            $containerIdParts[] = $jobId;
-            $containerIdParts[] = $this->storageClient->getRunId() ?: 'norunid';
-            $containerIdParts[] = $rowId;
-            $containerIdParts[] = $priority;
-            $containerIdParts[] = $image->getSourceComponent()->getSanitizedComponentId();
-
-            $containerNameParts[] = $image->getSourceComponent()->getSanitizedComponentId();
-            $containerNameParts[] = $jobId;
-            $containerNameParts[] = $this->storageClient->getRunId() ?: 'norunid';
-            $containerNameParts[] = $priority;
+            $containerIdParts = [
+                $jobId,
+                $this->storageClient->getRunId() ?: 'norunid',
+                $rowId,
+                $priority,
+                $image->getSourceComponent()->getSanitizedComponentId()
+            ];
+            $containerNameParts = [
+                $image->getSourceComponent()->getSanitizedComponentId(),
+                $jobId,
+                $this->storageClient->getRunId() ?: 'norunid',
+                $priority,
+            ];
 
             $container = $this->createContainerFromImage(
                 $image,