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

svr4pkg module: Doesn't properly report failures. #7645

Closed
yeroc opened this issue Jun 2, 2014 · 1 comment · Fixed by #7699
Closed

svr4pkg module: Doesn't properly report failures. #7645

yeroc opened this issue Jun 2, 2014 · 1 comment · Fixed by #7699
Labels
bug This issue/PR relates to a bug. solaris Solaris community

Comments

@yeroc
Copy link

yeroc commented Jun 2, 2014

Issue Type:

Bug Report

Ansible Version:

ansible 1.7 (devel 6bc056e) last updated 2014/06/02 14:42:33 (GMT -600)

Environment:

Solaris 10/11

Summary:

pkgadd failures aren't properly reported as errors but instead the status returned is "changed".

Steps To Reproduce:

Given a trivial task with invalid parameters:

- name: Install Non-Existant Package
  svr4pkg: name="somename" src="" state=present
Expected Results:

The task should report a "failed" status.

Actual Results:

Actually the task reports a "changed" status:

changed: [dbload-solaris] => {"changed": true, "item": "", "name": "somename", "state": "present", "stderr": "pkgadd: ERROR: the path <> is invalid!\n"}
@jimi-c
Copy link
Member

jimi-c commented Jun 9, 2014

The above PR was merged in, and should correct this. Please let us know if you continue to see any problems related to this issue. Thanks!

jimi-c added a commit that referenced this issue Jun 9, 2014
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 6, 2018
@dagwieers dagwieers added the solaris Solaris community label Jan 9, 2019
@ansible ansible locked and limited conversation to collaborators Apr 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug. solaris Solaris community
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants