Skip to content

Fix profiler build by downgrading nbbuild to 11 - #9717

Merged
ebarboni merged 1 commit into
apache:masterfrom
mbien:fix-profiler-build_delivery
Oct 15, 2026
Merged

ebarboni merged 1 commit into
apache:masterfrom
mbien:fix-profiler-build_delivery

Conversation

@mbien

@mbien mbien commented Oct 15, 2026

Copy link
Copy Markdown
Member

@mbien mbien added do not merge Don't merge this PR, it is not ready or just demonstration purposes. profiler build labels Oct 15, 2026
@mbien mbien added this to the NB32 milestone Oct 15, 2026
@mbien mbien removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Oct 15, 2026
@mbien

mbien commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

note: usually we wouldn't do that but we will merge this before CI ran since the queue has unknown length. Some jobs waited 7h yesterday. Ran some checks locally.

this is so that master/delivery are unblocked since they would be failing

@mbien
mbien requested a review from ebarboni October 15, 2026 13:50
@ebarboni
ebarboni changed the base branch from delivery to master October 15, 2026 13:52

@ebarboni ebarboni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

agree, wil redo branching

@mbien mbien changed the title try to fix profiler build by downgrading nbbuild to 11 Fix profiler build by downgrading nbbuild to 11 Oct 15, 2026
@ebarboni
ebarboni merged commit 0104eeb into apache:master Oct 15, 2026
9 checks passed
@neilcsmith-net

Copy link
Copy Markdown
Member

I didn't notice when porting this workflow originally that the setup doesn't work properly, and it just reverts to running on the default JDK on the runner. Workflow needs revision going forward as the ubuntu 22-04 runners have now been deprecated, and will cease to exist soon.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants