Return code of the operation called by pulp-admin should be propagated to pulp-admin
The return code of pulp-admin is always 0 which is not correct. This is a broad issue so the example in this bug is one single example. This bug is to fix all return code usage.
Commands that were called by pulp-admin should propagate in a proper way their return codes to pulp-admin. In other words pulp-admin should have same return code as the command it called.
For the command:
pulp-admin -u admin -p admin rpm consumer package install run --name some_rpm --consumer-id c1
Produces the output:
Install task created with id [ 44307c9e-c721-4b56-8e87-1954c104e3de ] This command may be exited via ctrl+c without affecting the request. Refresh Repository Metadata Install Failed failure: repodata/repomd.xml from m2: [Errno 256] No more mirrors to try. https://example.com/pulp/repos/m2/repo data/repomd.xml: [Errno 14] HTTPS Error 404 - Not Found
The return code in this case is still 0 which is incorrect.
[root@ec2-54-220-72-88 ~]# echo $? 0
If the script would call pulp-admin it would have to grep for any failure(that would complicate significantly the real life scenarios) instead of just checking the return code.
Updated by mhrivnak over 8 years ago
The pulp-admin exit code should be non-zero in this case, but should not necessarily mirror the exit code encountered by a handler on a consumer. The handler may not necessarily run a command that returns an exit code at all.
+ This comment was cloned from Bugzilla #1096875 comment 1 +
Updated by bmbouter over 4 years ago
Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.