[bazel][ci-minor-increase] Make new CI workflow: faster and aligned with Bazel ideas#39

Merged

HyperWinX merged 7 commits intomasterfrom

Aug 25, 2025

Contributor

AshFungor commented Aug 23, 2025 •

edited

Loading

This started out as support for distros musl-libc compiled binaries, but turned out to be a full workflow refactor.

Key changes:

remove conan. Quite useless, since we mostly consume google libs - they ship with bazel. Except for eternal, which is header-only so no trouble here.
use single CI docker image, then build and cross-compile everywhere. Much easier to support, much easier to setup.
binaries are packed into tars and deployed for testing (arch, debian, fedora)
formatting is mandatory
linting is optional, but it is recommended to search their outputs

There is still work to document new dev workflows, I'll work on that a couple of weeks later.

mentatbot bot commented Aug 23, 2025

You are out of Mentat credits. You can view and refresh your credits here.

_{To disable this message on pr open, disable Always Review on my settings page}

github-advanced-security bot found potential problems

View reviewed changes

.github/workflows/mainline-compile.yml Fixed Show fixed Hide fixed

.github/workflows/mainline-prepare.yml Dismissed Show dismissed Hide dismissed

.github/workflows/mainline-prepare.yml Dismissed Show dismissed Hide dismissed

.github/workflows/mainline-prepare.yml Dismissed Show dismissed Hide dismissed

.github/workflows/mainline.yml Fixed Show fixed Hide fixed

.github/workflows/mainline.yml Fixed Show fixed Hide fixed

.github/workflows/version-increment.yml Dismissed Show dismissed Hide dismissed

.github/workflows/version-increment.yml Dismissed Show dismissed Hide dismissed

github-advanced-security bot found potential problems

View reviewed changes

.github/workflows/mainline.yml Dismissed Show dismissed Hide dismissed

.github/workflows/mainline.yml Fixed Show fixed Hide fixed

github-advanced-security bot found potential problems

View reviewed changes

.github/workflows/mainline.yml

Comment on lines +15 to +18

+                  needs: prepare
+                  uses: ./.github/workflows/mainline-compile.yml
+                format:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

.github/workflows/mainline.yml

Comment on lines +19 to +22

+                  needs: prepare
+                  uses: ./.github/workflows/mainline-format.yml
+                lint:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

.github/workflows/mainline.yml Fixed Show fixed Hide fixed

github-advanced-security bot found potential problems

View reviewed changes

.github/workflows/mainline-test.yml

Comment on lines 12 to 40

+                  runs-on: ubuntu-latest
+                  strategy:
+                    fail-fast: false
+                    matrix:
+                      include:
+                        - name: Debian
+                          image: debian:stable
+                        - name: Fedora
+                          image: fedora:latest
+                        - name: Arch Linux
+                          image: archlinux:base-devel
+                  container:
+                    image: ${{ matrix.image }}
+                  steps:
+                    - name: Download tarball artifacts
+                      uses: actions/download-artifact@v4
+                      with:
+                        name: ${{ inputs.artifact_name }}
+                        path: artifacts
+                    - name: Extract and run tarballs
+                      run: |
+                        for archive in artifacts/*.tar.gz; do
+                          tar -xzf "$archive" -C / --no-same-owner --no-same-permissions
+                          for test in /opt/hcpu/tests/*; do ./$test; done
+                        done

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

.github/workflows/mainline.yml

Comment on lines +23 to +26

+                  needs: prepare
+                  uses: ./.github/workflows/mainline-lint.yml
+                tests:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

.github/workflows/mainline.yml

Comment on lines +27 to +28

		needs: package
		uses: ./.github/workflows/mainline-test.yml No newline at end of file

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

AshFungor requested a review from HyperWinX

August 24, 2025 18:16

AshFungor self-assigned this


          repo: rebased and resolved conflicts

8d0365c

AshFungor force-pushed the support-musl branch from 6febbc4 to 8d0365c Compare

August 24, 2025 18:24

AshFungor added 6 commits

August 24, 2025 21:35


          cpp: format

818cf56


          testing: fix binary invokation

6e9abc6


          pkg: ensure right perms

c99c098


          test: use find & execute technique

b8d66de


          test: switch back to glob

560ee84


          test: no overrite flag

90ee17c

AshFungor changed the title ~~[bazel] Make new CI workflow: faster and aligned with Bazel ideas~~ [bazel][ci-minor-increase] Make new CI workflow: faster and aligned with Bazel ideas

HyperWinX approved these changes

View reviewed changes

Member

HyperWinX left a comment

Perfect.

HyperWinX merged commit 1380e07 into master

11 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet