Skip to content

Don't set affinity if XDP is not in use#6618

Merged
alessandrod merged 1 commit intoanza-xyz:masterfrom
alessandrod:macos-affinity
Jun 17, 2025
Merged

Don't set affinity if XDP is not in use#6618
alessandrod merged 1 commit intoanza-xyz:masterfrom
alessandrod:macos-affinity

Conversation

@alessandrod
Copy link
Copy Markdown

This fixes the validator not starting on macos, oops

This fixes the validator not starting on macos, oops
@alessandrod alessandrod requested a review from brooksprumo June 17, 2025 16:51
Copy link
Copy Markdown

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

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

:shipit:

Built this branch on my macbook and confirmed I could run the validator successfully. Thanks!

@alessandrod alessandrod added automerge automerge Merge this Pull Request automatically once CI passes v2.3 labels Jun 17, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 82.8%. Comparing base (cbd3eb5) to head (7d719d0).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6618   +/-   ##
=======================================
  Coverage    82.8%    82.8%           
=======================================
  Files         849      849           
  Lines      379357   379358    +1     
=======================================
+ Hits       314270   314306   +36     
+ Misses      65087    65052   -35     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 17, 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.

@alessandrod alessandrod merged commit 7510f5d into anza-xyz:master Jun 17, 2025
30 checks passed
mergify Bot pushed a commit that referenced this pull request Jun 17, 2025
This fixes the validator not starting on macos, oops

(cherry picked from commit 7510f5d)
steviez pushed a commit that referenced this pull request Jun 18, 2025
)

Don't set affinity if XDP is not in use (#6618)

This fixes the validator not starting on macos, oops

(cherry picked from commit 7510f5d)

Co-authored-by: Alessandro Decina <alessandro.d@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants