Skip to content
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

Save sizes of built dynamic libraries and sys images #186

Closed
wants to merge 1 commit into from

Conversation

Zentrik
Copy link

@Zentrik Zentrik commented Mar 25, 2024

I just picked some big files that seemed related to Julia or LLVM.
The results are not shown in the report currently, I don't expect these sizes to change much frequently so it didn't seem necessary. I plan on adding these to https://tealquaternion.camdvr.org/.
I've tried to make sure even if fetching/ saving the sizes fails it doesn't affect anything else.

I wasn't sure how to test this locally.

I just picked some big files that seemed related to Julia or LLVM.
This is not shown in the report currently, I don't expect these sizes to change much so it didn't seem necessary.
@maleadt
Copy link
Member

maleadt commented Mar 27, 2024

LGTM, but @vtjnash would have to deploy this to the benchmark cluster (i.e. #125).

@vtjnash
Copy link
Member

vtjnash commented Mar 30, 2024

I should try to see if anyone else has an MIT account who can also update the server, as I don't have too much time to deal with updates often. Otherwise though, this seems like a good idea. I had sometimes been scraping the buildkite logs for this same purpose, but I don't know if that runs the make build-stats steps anymore which would be a necessary prerequisite for it

@Zentrik
Copy link
Author

Zentrik commented Mar 30, 2024

I think once JuliaLang/julia#53902 is merged, JuliaCI/julia-buildkite#224 can be merged to reenable build-stats at which point this pr isn't particularly useful.

@Zentrik Zentrik closed this Jun 18, 2024
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.

3 participants