ci: run jobs in parallel, simplify dockerignore [skip tests]

kalisio · Mar 15, 2024 · 6b7aee5 · 6b7aee5
1 parent 7fef5c2
commit 6b7aee5
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 167 deletions.
diff --git a/.github/workflows/additional_tests.yaml b/.github/workflows/additional_tests.yaml
diff --git a/.github/workflows/build_app.yaml b/.github/workflows/build_app.yaml
diff --git a/.github/workflows/build_docs.yaml b/.github/workflows/build_docs.yaml
diff --git a/.github/workflows/main.yaml b/.github/workflows/main.yaml
@@ -56,7 +56,6 @@ jobs:
   build_app:
     name: Build app
     if: ${{ !contains(github.event.head_commit.message, 'skip app') }}
-    needs: [ run_tests ]
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout repo

diff --git a/.github/workflows/run_tests.yaml b/.github/workflows/run_tests.yaml
diff --git a/app.Dockerfile.dockerignore b/app.Dockerfile.dockerignore
@@ -1,16 +1,10 @@
-# Ignore verything
-*
+# This is the dockerignore used by app.Dockerfile, it filters out
+# everything we don't want in the 'COPY . /opt/kalisio' step where
+# '.' is the workspace folder.
+# Since we expect the workspace to be clean, we don't allowlist
+# specific folders, we copy everyhting but the following :
 
-# Except these
-!kli.js
-!kli/*
-!feathers-s3/*
-!feathers-import-export/*
-!feathers-webpush/*
-!kdk/*
-!kapp/*
-
-# But ignore these anyway
+# Ignore all these
 **/node_modules
 **/npm-debug.log
 **/*Dockerfile

diff --git a/scripts/init_runner.sh b/scripts/init_runner.sh
@@ -24,10 +24,6 @@ init_github_build_docs() {
     install_reqs age sops nvm node18
 }
 
-init_github_main_ci() {
-    install_reqs age sops nvm node16 node18 mongo4
-}
-
 init_github_additional_tests() {
     install_reqs age sops nvm node16 node18 node20 mongo4 mongo5 mongo6
 }