-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update fbos deps to v2024.05.20.00 #9968
Conversation
✅ Deploy Preview for meta-velox canceled.
|
@majetideepak FYI. |
702382f
to
1df7a17
Compare
1df7a17
to
99394f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
43f7011
to
86328ef
Compare
@kgpai Can you please help commit this? |
@majetideepak @assignUser please add the "ready-to-merge" whenever you want this merged :) |
@pedroerp yes, the label has been working well! We are waiting on a rebase so CI will be green :) |
@czentgr setup-centos9.sh needs an update as well. |
86328ef
to
c94ba03
Compare
@kagamiori has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@kagamiori merged this pull request in 9558dcb. |
Conbench analyzed the 1 benchmark run on commit There was 1 benchmark result indicating a performance regression:
The full Conbench report has more details. |
The main fix is for CentOS9 to find libdwarf in the correct system install path (found in folder
libdwarf-0
). This was added at the end of April here facebook/folly@ebd05bfI decided not to use the latest due to an error with MacOS building fbthrift