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

Chocolatey 0.9.10 and exit codes - Chocolatey Salt module is now reporting successful package installs as failures #34099

Closed
ferventcoder opened this issue Jun 17, 2016 · 3 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior Execution-Module P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around State-Module Windows
Milestone

Comments

@ferventcoder
Copy link

Description of Issue

I mentioned this before in April - chocolatey/choco#512. 0.9.10 just released today and folks are going to start reporting breakages even though we noted it in our release notes and tried to give plenty of notice to folks that it was coming (2 months notice in this case).

Here is the relevant portion of the ChangeLog (under breaking changes):

  • Exit with the same exit code as the software being installed - see #512

There are more exit codes from Chocolatey now that indicate success -0, 1605, 1614, 1641, and 3010. You may need to adjust anything you were using that would only check for 0 and nonzero.
If you need the previous behavior, be sure to disable the feature usePackageExitCodes or use the --ignore-package-exit-codes switch in your choco commands.

Steps to Reproduce Issue

Use Chocolatey 0.9.10 with the Chocolatey Salt module and install a package that would indicate a reboot is necessary.

@ferventcoder
Copy link
Author

cc @jfindlay @rallytime

@Ch3LL
Copy link
Contributor

Ch3LL commented Jun 20, 2016

@ferventcoder thanks for the heads up. Looks like we need to add some more logic for those return codes.

@Ch3LL Ch3LL added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around P3 Priority 3 Execution-Module State-Module Platform Relates to OS, containers, platform-based utilities like FS, system based apps Windows labels Jun 20, 2016
@Ch3LL Ch3LL added this to the Approved milestone Jun 20, 2016
@Ch3LL
Copy link
Contributor

Ch3LL commented Jun 20, 2016

ping @twangboy just so you are aware as well

@twangboy twangboy self-assigned this Jun 20, 2016
@twangboy twangboy modified the milestones: C 7, Approved Jun 20, 2016
@meggiebot meggiebot modified the milestones: Approved, C 7 Jun 27, 2016
@meggiebot meggiebot modified the milestones: C 4, Approved Aug 24, 2016
@meggiebot meggiebot modified the milestones: C 3, C 4 Sep 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior Execution-Module P3 Priority 3 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around State-Module Windows
Projects
None yet
Development

No branches or pull requests

4 participants