Skip to content

Enable dynamic matrix jobs based on github event path#324

Merged
carlopi merged 2 commits intoduckdb:mainfrom
smvv:enable-dynamic-matrix
Feb 20, 2026
Merged

Enable dynamic matrix jobs based on github event path#324
carlopi merged 2 commits intoduckdb:mainfrom
smvv:enable-dynamic-matrix

Conversation

@smvv
Copy link
Contributor

@smvv smvv commented Feb 19, 2026

  • Use the extbuild matrix output instead of the Python script.

@smvv smvv requested a review from carlopi February 19, 2026 15:47
Copy link
Collaborator

@carlopi carlopi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment, but this it's blocked on the comments at #323 (review), to be discussed what to be implemented.

Comment on lines -226 to -231
- name: Compare extbuild vs python matrices
continue-on-error: true
run: |
mkdir -p build/python_ref build/extbuild_ref

python3 extension-ci-tools/scripts/modify_distribution_matrix.py --input extension-ci-tools/config/distribution_matrix.json --select_os linux --output build/python_ref/linux_matrix.json --exclude "${{ inputs.exclude_archs }}" --opt_in "${{ inputs.opt_in_archs }}" --pretty --reduced_ci_mode ${{ inputs.reduced_ci_mode }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, more as a note, but I am not sure if we are ready for removing this part yet, to be discussed.

@smvv smvv marked this pull request as ready for review February 19, 2026 16:17
@carlopi carlopi merged commit 2c4c5eb into duckdb:main Feb 20, 2026
8 checks passed
@carlopi
Copy link
Collaborator

carlopi commented Feb 20, 2026

Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants