ci: build node in manylinux docker container (#350)

Closes #359

TODO:
 * [x] test in a sample of Linux distro docker containers

.github/workflows/npm-publish.yml

@@ -67,48 +67,24 @@ jobs:
             node/dist/lancedb-vectordb-darwin*.tgz
 
   node-linux:
-    name: node-linux (${{ matrix.arch}}-unknown-linux-${{ matrix.libc }})
-    runs-on: ubuntu-latest
+    name: node-linux (${{ matrix.config.arch}}-unknown-linux-gnu
+    runs-on: ${{ matrix.config.runner }}
     # Only runs on tags that matches the make-release action
     if: startsWith(github.ref, 'refs/tags/v')
     strategy:
       fail-fast: false
       matrix:
-        libc:
-          - gnu
-          # TODO: re-enable musl once we have refactored to pre-built containers
-          # Right now we have to build node from source which is too expensive.
-          # - musl
-        arch:
-          - x86_64
-          # Building on aarch64 is too slow for now
-          # - aarch64
+        config:
+          - arch: x86_64
+            runner: ubuntu-latest
+          - arch: aarch64
+            runner: buildjet-4vcpu-ubuntu-2204-arm
     steps:
       - name: Checkout
         uses: actions/checkout@v3
-      - name: Change owner to root (for npm)
-        # The docker container is run as root, so we need the files to be owned by root
-        # Otherwise npm is a nightmare: https://github.com/npm/cli/issues/3773
-        run: sudo chown -R root:root .
-      - name: Set up QEMU
-        if: ${{ matrix.arch == 'aarch64' }}
-        uses: docker/setup-qemu-action@v2
-        with:
-          platforms: arm64
-      - name: Build Linux GNU native node modules
-        if: ${{ matrix.libc == 'gnu' }}
+      - name: Build Linux Artifacts
         run: |
-          docker run \
-            -v $(pwd):/io -w /io \
-            rust:1.70-bookworm \
-            bash ci/build_linux_artifacts.sh ${{ matrix.arch }}-unknown-linux-gnu
-      - name: Build musl Linux native node modules
-        if: ${{ matrix.libc == 'musl' }}
-        run: |
-          docker run --platform linux/arm64/v8 \
-            -v $(pwd):/io -w /io \
-            quay.io/pypa/musllinux_1_1_${{ matrix.arch }} \
-            bash ci/build_linux_artifacts.sh ${{ matrix.arch }}-unknown-linux-musl
+          bash ci/build_linux_artifacts.sh ${{ matrix.config.arch }}
       - name: Upload Linux Artifacts
         uses: actions/upload-artifact@v3
         with:
@@ -116,33 +92,6 @@ jobs:
           path: |
             node/dist/lancedb-vectordb-linux*.tgz
 
-  node-linux-arm:
-    name: node-linux (aarch64-unknown-linux-gnu)
-    runs-on: buildjet-4vcpu-ubuntu-2204-arm
-    # Only runs on tags that matches the make-release action
-    if: startsWith(github.ref, 'refs/tags/v')
-    strategy:
-      fail-fast: false
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-      - name: Change owner to root (for npm)
-        # The docker container is run as root, so we need the files to be owned by root
-        # Otherwise npm is a nightmare: https://github.com/npm/cli/issues/3773
-        run: sudo chown -R root:root .
-      - name: Build Linux GNU native node modules
-        run: |
-          docker run \
-            -v $(pwd):/io -w /io \
-            rust:1.70-bookworm \
-            bash ci/build_linux_artifacts.sh aarch64-unknown-linux-gnu
-      - name: Upload Linux Artifacts
-        uses: actions/upload-artifact@v3
-        with:
-          name: native-linux-arm
-          path: |
-            node/dist/lancedb-vectordb-linux*.tgz
-
   node-windows:
     runs-on: windows-2022
     # Only runs on tags that matches the make-release action
@@ -177,7 +126,7 @@ jobs:
             node/dist/lancedb-vectordb-win32*.tgz
 
   release:
-    needs: [node, node-macos, node-linux, node-linux-arm, node-windows]
+    needs: [node, node-macos, node-linux, node-windows]
     runs-on: ubuntu-latest
     # Only runs on tags that matches the make-release action
     if: startsWith(github.ref, 'refs/tags/v')