Bug 1308640 - hammer task progress accepts NIL value for id parameter
Summary: hammer task progress accepts NIL value for id parameter
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Tasks Plugin
Version: 6.1.6
Hardware: Unspecified
OS: Unspecified
unspecified
medium vote
Target Milestone: Unspecified
Assignee: Amit Karsale
QA Contact: Roman Plevka
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-02-15 16:36 UTC by Roman Plevka
Modified: 2017-08-04 13:19 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-08-04 13:19:06 UTC
Target Upstream Version:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 17358 0 None None None 2016-11-16 11:50:51 UTC

Description Roman Plevka 2016-02-15 16:36:39 UTC 
Description of problem:
hammer displays a fake progress bar (always 0%) for task with null id

Version-Release number of selected component (if applicable):
rubygem-hammer_cli_katello-0.0.7.21-1.el6.noarch

How reproducible:
always

Steps to Reproduce:
1. # hammer -v -u admin -p changeme  task progress --id=
or
# hammer -v -u admin -p changeme  task progress --id=''


Actual results:
# hammer -v -u admin -p changeme  task progress --id=
[                       ] [0%]

Expected results:
Error: missing param 'id' in parameters
or
Error: invalid 'id' parameter specified

Additional info:
Comment 1 Bryan Kearney 2016-07-26 19:04:54 UTC 
Moving 6.2 bugs out to sat-backlog.
Comment 2 Adam Ruzicka 2016-11-16 11:50:50 UTC 
Created redmine issue http://projects.theforeman.org/issues/17358 from this bug
Comment 3 Satellite Program 2017-01-05 16:07:30 UTC 
Upstream bug assigned to akarsale
Comment 4 Amit Karsale 2017-01-09 07:02:31 UTC 
PR submitted for the mentioned issue, can we have it verified and merged by if appropriate. 

https://github.com/theforeman/hammer-cli-foreman-tasks/pull/31
Comment 5 Bryan Kearney 2017-08-04 13:19:06 UTC 
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.
Note You need to log in before you can comment on or make changes to this bug.