Skip to content

Switch the onder-riscv64 builder to Debian, enable dtrace - #802

Merged
StanFromIreland merged 4 commits into
python:mainfrom
furkanonder:onder-riscv64-debian-dtrace
Sep 27, 2026
Merged

StanFromIreland merged 4 commits into
python:mainfrom
furkanonder:onder-riscv64-debian-dtrace

Conversation

@furkanonder

Copy link
Copy Markdown
Contributor

Since @StanFromIreland moved RISC-V to tier 3 in #791 we've got four riscv64 builders, and they were all Ubuntu.
I switched my board to Debian to increase distro diversity across the fleet.
While I was there I turned on --with-dtrace, which no other riscv64 worker has.

Comment thread master/custom/builders.py Outdated

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We might have to include it in d74e3cb, I'm not sure if the tests will pass on older branches?

Comment thread master/custom/builders.py Outdated
Comment thread master/custom/builders.py Outdated
@furkanonder

furkanonder commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor Author

We might have to include it in d74e3cb, I'm not sure if the tests will pass on older branches?

I tested it on the board. I built main with --with-dtrace and test_dtrace passes.

I don't think we need the d74e3cb skip. We only pass --with-dtrace on a PR or on 3.15+ (workers.py:76), so on 3.11 through 3.14 the worker builds without it and the < (3, 13) condition never applies.

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

I'm not sure if you can merge here (?), if you can't send me a message on Discord when you're ready and I'm happy to land this.

@StanFromIreland
StanFromIreland merged commit 66d588e into python:main Sep 27, 2026
1 check passed
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