Skip to content

Add nightly builds for out-of-tree python extension#18239

Merged
Mytherin merged 5 commits intoduckdb:v1.3-ossivalisfrom
evertlammerts:nightly_python_builds
Jul 15, 2025
Merged

Add nightly builds for out-of-tree python extension#18239
Mytherin merged 5 commits intoduckdb:v1.3-ossivalisfrom
evertlammerts:nightly_python_builds

Conversation

@evertlammerts
Copy link
Contributor

@evertlammerts evertlammerts commented Jul 14, 2025

See https://github.com/duckdblabs/duckdb-internal/issues/5309

This also adds the changes from 0053604 (since we need the version override on both the bugfix and main branches)

@evertlammerts evertlammerts marked this pull request as ready for review July 14, 2025 12:26
@evertlammerts evertlammerts reopened this Jul 14, 2025
Copy link
Contributor

@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.

This looks great, I would consider:

"publish_to_pypi": ${{ inputs.should-publish }}

to be set to false to start with, so basically the duckdb-python repo will be notified but will not publish, and once building is tested to work, sending a PR removing Python.yml AND switching this as relevant.

Or directly in this PR skip publishing for Python.yml, but doing so in lockstep, so to avoid both compete for pypi, it should be a matter of avoiding job twine-upload at https://github.com/duckdb/duckdb/blob/main/.github/workflows/Python.yml#L423

@duckdb-draftbot duckdb-draftbot marked this pull request as draft July 14, 2025 14:30
@evertlammerts evertlammerts marked this pull request as ready for review July 14, 2025 14:41
@evertlammerts
Copy link
Contributor Author

Thanks for the review!

This looks great, I would consider:

"publish_to_pypi": ${{ inputs.should-publish }}

to be set to false to start with, so basically the duckdb-python repo will be notified but will not publish, and once building is tested to work, sending a PR removing Python.yml AND switching this as relevant.

Or directly in this PR skip publishing for Python.yml, but doing so in lockstep, so to avoid both compete for pypi, it should be a matter of avoiding job twine-upload at https://github.com/duckdb/duckdb/blob/main/.github/workflows/Python.yml#L423

Right now the workflow in duckdb-python won't publish to pypi, it will just upload to the s3 staging bucket.

@Mytherin Mytherin merged commit e926f28 into duckdb:v1.3-ossivalis Jul 15, 2025
2 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Jul 15, 2025
Backport DB invalidation flag to ossivalis (duckdb/duckdb#18244)
Add nightly builds for out-of-tree python extension (duckdb/duckdb#18239)
Remove require block size from a batch of tests (duckdb/duckdb#18242)
github-actions bot added a commit to duckdb/duckdb-r that referenced this pull request Jul 15, 2025
Backport DB invalidation flag to ossivalis (duckdb/duckdb#18244)
Add nightly builds for out-of-tree python extension (duckdb/duckdb#18239)
Remove require block size from a batch of tests (duckdb/duckdb#18242)

Co-authored-by: krlmlr <krlmlr@users.noreply.github.com>
@evertlammerts evertlammerts deleted the nightly_python_builds branch July 22, 2025 07:45
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.

3 participants