Merge pull request socallinuxexpo#783 from socallinuxexpo/rh/issue-781

[READY] - gha: limit nixos builds to master and manual
ruebenramirez · Sep 27, 2024 · ef74408 · ef74408
2 parents 25abe86 + ce6a9bc
commit ef74408
Show file tree

Hide file tree

Showing 2 changed files with 40 additions and 30 deletions.
diff --git a/.github/workflows/ci-nixos.yml b/.github/workflows/ci-nixos.yml
@@ -0,0 +1,40 @@
+name: ci-nixos
+on:
+  push: # This is only run when PRs are merged into master
+    branches:
+      - master
+  workflow_dispatch:
+# These jobs can take a long time and potentially cause us to bump up against CI
+# rate limits. Instead we've opted to run these only on merges to master. We might
+# have to limit these further if the rate limiting remains
+jobs:
+  nixos_configs_get:
+    name: nixos configurations get name
+    runs-on: ubuntu-latest
+    outputs:
+      nixosconfignames: ${{ steps.nixosconfigsget.outputs.nixosconfignames }}
+    steps:
+      - uses: actions/checkout@v3
+      - uses: DeterminateSystems/nix-installer-action@main
+      - uses: DeterminateSystems/magic-nix-cache-action@main
+      - name: Get all NixOS configurations
+        id: nixosconfigsget
+        run: |
+          nixosconfignames=$(nix eval .\#nixosConfigurations --apply builtins.attrNames --json)
+          echo "$nixosconfignames"
+          echo "nixosconfignames=$nixosconfignames" >> $GITHUB_OUTPUT
+  nixos_configs_build:
+    name: build ${{ matrix.configName }} nixos configuration
+    needs: nixos_configs_get
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        configName: ${{ fromJSON(needs.nixos_configs_get.outputs.nixosconfignames) }}
+    steps:
+      - uses: actions/checkout@v3
+      - uses: DeterminateSystems/nix-installer-action@main
+      - uses: DeterminateSystems/magic-nix-cache-action@main
+      - name: Build ${{ matrix.configName }} nixos configuration
+        run: |
+          nix build -L .#nixosConfigurations.${{ matrix.configName }}.config.system.build.toplevel
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -28,33 +28,3 @@ jobs:
       - run: nix build -L .#massflash
       - run: nix build -L .#scaleInventory
       - run: nix build -L .#serverspec
-  nixos_configs_get:
-    name: nixos configurations get name
-    runs-on: ubuntu-latest
-    outputs:
-      nixosconfignames: ${{ steps.nixosconfigsget.outputs.nixosconfignames }}
-    steps:
-      - uses: actions/checkout@v3
-      - uses: DeterminateSystems/nix-installer-action@main
-      - uses: DeterminateSystems/magic-nix-cache-action@main
-      - name: Get all NixOS configurations
-        id: nixosconfigsget
-        run: |
-          nixosconfignames=$(nix eval .\#nixosConfigurations --apply builtins.attrNames --json)
-          echo "$nixosconfignames"
-          echo "nixosconfignames=$nixosconfignames" >> $GITHUB_OUTPUT
-  nixos_configs_build:
-    name: build ${{ matrix.configName }} nixos configuration
-    needs: nixos_configs_get
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false
-      matrix:
-        configName: ${{ fromJSON(needs.nixos_configs_get.outputs.nixosconfignames) }}
-    steps:
-      - uses: actions/checkout@v3
-      - uses: DeterminateSystems/nix-installer-action@main
-      - uses: DeterminateSystems/magic-nix-cache-action@main
-      - name: Build ${{ matrix.configName }} nixos configuration
-        run: |
-          nix build -L .#nixosConfigurations.${{ matrix.configName }}.config.system.build.toplevel