-
Notifications
You must be signed in to change notification settings - Fork 85
ci(benchmark): switch to new conbench instance #593
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
Conversation
|
@zeroshade Please don't merge yet as I might restart conbench db one more time and results would be lost. |
|
I don't believe we did, is that something you can help us set up? |
|
@zeroshade sure, I need to reset alerts for apache/arrow so adding them for apache/arrow-go shouldn't be much overhead. |
|
@rok anything more needed to be done here? is it ready for review yet? |
|
I was waiting for the backfill to get some confidence in the process I think we can enable (merge) this and monitor how it goes. I think PR is good as is. |
|
@zeroshade so I think we can merge. |
zeroshade
left a comment
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 @rok!!
|
First benchmark has appeared on 'https://conbench.arrow-dev.org/'! |
Rationale for this change
conbench.ursa.dev is no longer online.
What changes are included in this PR?
Switch posting benchmarking result to conbench.arrow-dev.org in corresponiding workflow.
Are these changes tested?
No.
Are there any user-facing changes?
Conbench view is on a new url.
Closes #592