Skip to content

v2.3: Don't set affinity if XDP is not in use (backport of #6618)#6620

Merged
steviez merged 1 commit intov2.3from
mergify/bp/v2.3/pr-6618
Jun 18, 2025
Merged

v2.3: Don't set affinity if XDP is not in use (backport of #6618)#6620
steviez merged 1 commit intov2.3from
mergify/bp/v2.3/pr-6618

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Jun 17, 2025

This fixes the validator not starting on macos, oops


This is an automatic backport of pull request #6618 done by Mergify.

This fixes the validator not starting on macos, oops

(cherry picked from commit 7510f5d)
@mergify mergify Bot requested a review from a team as a code owner June 17, 2025 18:17
Copy link
Copy Markdown

@bw-solana bw-solana left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify
Copy link
Copy Markdown
Author

mergify Bot commented Jun 18, 2025

Backports to the beta branch are to be avoided unless absolutely necessary for fixing bugs, security issues, and perf regressions. Changes intended for backport should be structured such that a minimum effective diff can be committed separately from any refactoring, plumbing, cleanup, etc that are not strictly necessary to achieve the goal. Any of the latter should go only into master and ride the normal stabilization schedule. Exceptions include CI/metrics changes, CLI improvements and documentation updates on a case by case basis.

Copy link
Copy Markdown

@steviez steviez left a comment

Choose a reason for hiding this comment

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

LGTM; this fixes the default behavior while still erroring if someone tries to use this config on unsupported platform

@steviez steviez merged commit 6c40b14 into v2.3 Jun 18, 2025
47 checks passed
@steviez steviez deleted the mergify/bp/v2.3/pr-6618 branch June 18, 2025 16:15
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.

4 participants