Update Makefile to check all Dockerfiles (RonaldHensbergen#119)

* Update Makefile to check all Dockerfiles

* Update Dockerfile search pattern in CI workflow

* Fix/dagster round 2 (RonaldHensbergen#121) (RonaldHensbergen#122)

* Fixing Dagster user-code being unhealthy

* Test-case T1 succeeds!

* Updating the test checklist

* Updated test with correct Dockerfile

* Linting issues (why a file I didn't change?)

* Updating Dockerfile

* Adding healthcheck to docker-daemon image

* Added test for dagster daemon running when dagster is part of the profile

* Fix/dagster round 2 (RonaldHensbergen#117) (RonaldHensbergen#120)

* Fixing Dagster user-code being unhealthy

* Test-case T1 succeeds!

* Updating the test checklist

* Updated test with correct Dockerfile

* Linting issues (why a file I didn't change?)

* user-code image build check is good again

Author: RonaldHensbergen

.github/workflows/ci.yml

@@ -73,8 +73,12 @@ jobs:
 
       - name: Build all Dockerfiles
         run: |
-          for dockerfile in $(find . -name "Dockerfile" -type f); do
+          for dockerfile in $(find . -name "Dockerfile*" -type f); do
             dir=$(dirname "$dockerfile")
             echo "Building $dockerfile in directory $dir..."
-            docker build -f "$dockerfile" "$dir" || exit 1
+            context="$dir"
+            if [ "$dockerfile" = "./images/dagster/Dockerfile.user-code" ]; then
+              context="."
+            fi
+            docker build -f "$dockerfile" "$context" || exit 1
           done

Makefile

@@ -29,9 +29,13 @@ package:
 
 docker-build:
 	@echo "Building all Dockerfiles..."
-	@for dockerfile in $$(find . -name "Dockerfile" -type f); do \
+	@for dockerfile in $$(find . -name "Dockerfile*" -type f); do \
 		dir=$$(dirname "$$dockerfile"); \
 		echo "Building $$dockerfile in directory $$dir..."; \
-		docker build -f "$$dockerfile" "$$dir" || exit 1; \
+		context="$$dir"; \
+		if [ "$$dockerfile" = "./images/dagster/Dockerfile.user-code" ]; then \
+			context="."; \
+		fi; \
+		docker build -f "$$dockerfile" "$$context" || exit 1; \
 	done
 	@echo "All Dockerfiles built successfully!"