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

Turn off download progress #83

Closed
ferventcoder opened this issue Jun 2, 2017 · 9 comments
Closed

Turn off download progress #83

ferventcoder opened this issue Jun 2, 2017 · 9 comments

Comments

@ferventcoder
Copy link
Member

ferventcoder commented Jun 2, 2017

In 0.10.4, Chocolatey started to show download progress for packages as well as any downloaded resources. When something else is using Chocolatey and capturing output for logging purposes, it can overwhelm the log.

This is the case in PowerShell DSC. You receive a cryptic error similar to The WS-Management service cannot process the request. The computed response packet size (3388031) exceeds the maximum envelope size that is allowed (512000).

When 0.10.4+ is detected, add --no-progress (chocolatey/choco#917) automatically when managing packages with cChoco.

@jrdnr
Copy link
Contributor

jrdnr commented Nov 14, 2017

@ferventcoder Should this be Up for grabs or are you working on it? I have working code to patch this that I'm using internally, but its a little fragile because I just pasted the same check into the install/uninstall/ and update functions. The right way to fix this might be to refactor the install/uninstall/update functions to have more consistent code and naming across them, and actually break running choco $params into its own function so that anytime there is a a change to the core choco application output it only has to be updated one place.

@ferventcoder
Copy link
Member Author

@jrdnr up for grabs.

@ferventcoder
Copy link
Member Author

Go for the simple changes first and we can look for the more invasive changes later.

@jessejacob
Copy link

jessejacob commented Dec 8, 2017

Just wanted to say thanks y'all for implementing this. HUGE reduction in Packer log sizes where I was using lots of choco operations!

@jrdnr
Copy link
Contributor

jrdnr commented Aug 17, 2018

@ferventcoder Any chance of getting this PR merged?

@pauby pauby modified the milestones: 2.4.0, 2.5.0 Nov 9, 2018
@TTimo
Copy link

TTimo commented Dec 7, 2018

I would like to see this addressed as well.

@jrdnr
Copy link
Contributor

jrdnr commented Dec 7, 2018

@TTimo The PR to fix was submitted almost a year ago, but I haven't been able to get a response from the maintainers.

@FinickyCode
Copy link

A possible workaround while waiting for this PR to be merged is to disable progress globally.

choco feature disable -n=showDownloadProgress

@pauby
Copy link
Member

pauby commented Mar 6, 2019

The PR is working it's way through. We have been very slow to pick this up. Changes have been requested and would hope to get this ready for the next release.

pauby added a commit that referenced this issue Jul 28, 2019
@pauby pauby added the 4 - Done label Jul 28, 2019
@pauby pauby closed this as completed Jul 28, 2019
@pauby pauby modified the milestones: 2.5.0, 2.4.1 Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants