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

Salt reports success on pecl failure #8750

Closed
jeteokeeffe opened this issue Nov 22, 2013 · 1 comment
Closed

Salt reports success on pecl failure #8750

jeteokeeffe opened this issue Nov 22, 2013 · 1 comment

Comments

@jeteokeeffe
Copy link
Contributor

Pecl state is reporting a successful installation of a pecl module but the module did not install.

CentOS 6, Salt 17.1 (this case should error since libssh2-devel is a requirement for ssh2 pecl module)

libssh2-devel:
pkg:
- removed

ssh2:
pecl:
- installed
- version: 0.12
- defaults: True
- require:
- pkg: libssh2-devel

@cachedout
Copy link
Contributor

Thanks for reporting this, @jeteokeeffe. Take a look at the above patch and see if that resolves it for you. Because it's working for me in testing, I'm going to close this for now but if it's not working for you as well feel free to leave a comment and we'll re-open this. Thanks!

thatch45 added a commit that referenced this issue Nov 25, 2013
Don't depend on the pecl return code because it's broken.
cachedout pushed a commit that referenced this issue Dec 5, 2013
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

No branches or pull requests

2 participants