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

[feature] show uploaded files size in conan upload #9134

Closed
1 task done
leha-bot opened this issue Jun 21, 2021 · 3 comments · Fixed by #16103
Closed
1 task done

[feature] show uploaded files size in conan upload #9134

leha-bot opened this issue Jun 21, 2021 · 3 comments · Fixed by #16103

Comments

@leha-bot
Copy link

leha-bot commented Jun 21, 2021

It would be very convenient to provide the size of files which will be upload to specific remote via conan upload ... in diagnostic purposes, e.g. sometimes artifacts could be very large and CI could refuse its uploading, see e.g. this StackOverflow question as example of related CI error. With this change CI administrators could adjust the file limit at CI without trial and error changing of CI configs.

@memsharded
Copy link
Member

It is relatively straightforward to just check the files in the Conan cache in case the upload failed, so not a high priority. We might add it to the output in Conan 2.0, depends on how cluttered the output looks, might be added or not, we will see.

@mkmkme
Copy link
Contributor

mkmkme commented Jul 16, 2023

@memsharded was the decision to add it or to not? Should this ticket be closed?

@memsharded memsharded added this to the 2.X milestone Jul 16, 2023
@memsharded
Copy link
Member

Neither, a decision was not taken. This kind of small ux details haven't been considered yet. I am fine with adding the info before/during upload of big files only (all files would be very noisy)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants