-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Push nightly image with profiling profile #15206
Conversation
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.
Can we add one for op-reth profiling as well?
sure, can add |
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.
just some nits, otherwise lgtm
Co-authored-by: Dan Cline <[email protected]>
Co-authored-by: Dan Cline <[email protected]>
Co-authored-by: Dan Cline <[email protected]>
Co-authored-by: Dan Cline <[email protected]>
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.
oops, sorry just one more thing
Co-authored-by: Dan Cline <[email protected]>
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.
lgtm
Co-authored-by: Dan Cline <[email protected]>
Description
For running automated benchmarking we will need a docker image built with profiling profile. This PR adds this to our nightly build workflow.
Note: Thought about whether to make the profiling profile default in nightly tag, but decided against it since some teams use the
nightly
tag (built withmaxperf
) and just pushing a separate image with new tag callednightly-profiling
.