Add hadolint to CI (home-assistant#34758)

* Add hadolint to CI * Fix lint & name * Update azure-pipelines-ci.yml Co-Authored-By: Franck Nijhof <[email protected]> Co-authored-by: Franck Nijhof <[email protected]>
peribeir · Apr 27, 2020 · be57d45 · be57d45
1 parent e459053
commit be57d45
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 2 deletions.
diff --git a/.hadolint.yaml b/.hadolint.yaml
@@ -0,0 +1,5 @@
+ignored:
+  - DL3006
+  - DL3008
+  - DL3013
+  - DL3018
diff --git a/Dockerfile.dev b/Dockerfile.dev
@@ -18,8 +18,7 @@ WORKDIR /usr/src
 
 # Setup hass-release
 RUN git clone --depth 1 https://github.com/home-assistant/hass-release \
-    && cd hass-release \
-    && pip3 install -e .
+    && pip3 install -e hass-release/
 
 WORKDIR /workspaces
 

diff --git a/azure-pipelines-ci.yml b/azure-pipelines-ci.yml
@@ -26,6 +26,8 @@ resources:
 variables:
   - name: PythonMain
     value: "37"
+  - name: versionHadolint
+    value: "v1.17.6"
 
 stages:
   - stage: "Overview"
@@ -121,6 +123,22 @@ stages:
               . venv/bin/activate
               pre-commit run black --all-files --show-diff-on-failure
             displayName: "Check Black formatting"
+      - job: "Docker"
+        pool:
+          vmImage: "ubuntu-latest"
+        steps:
+          - script: sudo docker pull hadolint/hadolint:$(versionHadolint)
+            displayName: "Install Hadolint"
+          - script: |
+              set -e
+              for dockerfile in Dockerfile Dockerfile.dev
+              do
+                echo "Linting: $dockerfile"
+                docker run --rm -i \
+                  -v "$(pwd)/.hadolint.yaml:/.hadolint.yaml:ro" \
+                  hadolint/hadolint:$(versionHadolint) < "$dockerfile"
+              done
+            displayName: "Run Hadolint"
 
   - stage: "Tests"
     dependsOn: