Bug 1201731 - hammer discovery list command should show the correct subnet value
Summary: hammer discovery list command should show the correct subnet value
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Discovery Plugin
Version: 6.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: orabin
QA Contact: Sachin Ghai
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-03-13 11:22 UTC by Sachin Ghai
Modified: 2019-09-26 17:38 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-08-01 20:03:35 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
on UI subnet field shows subnet name along with actual subnet(192.168.100.0/24) (24.47 KB, image/png)
2015-03-13 11:41 UTC, Sachin Ghai
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 10193 0 None None None 2016-04-22 16:15:22 UTC

Description Sachin Ghai 2015-03-13 11:22:50 UTC
Description of problem:


# hammer --username admin --password changeme discovery list
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
---|-----------------|-------------------|---------------------|------|--------|------------|------------|---------|--------------------
ID | NAME            | MAC               | LAST REPORT         | CPUS | MEMORY | DISK COUNT | DISKS SIZE | SUBNET  | LAST REPORT        
---|-----------------|-------------------|---------------------|------|--------|------------|------------|---------|--------------------
2  | mac525400e0e8ab | 52:54:00:e0:e8:ab | 2015/03/13 03:58:44 | 1    | 995    | 1          | 8192       | libvirt | 2015/03/13 03:58:44
---|-----------------|-------------------|---------------------|------|--------|------------|------------|---------|--------------------


Here subnet column displays 'libvirt', it should show actual subnet too like in UI:  libvirt (192.168.100.0/24)

Version-Release number of selected component (if applicable):
sat6.1 beta snap6 compose2
Satellite-6.1.0-RHEL-6-20150311.1/

How reproducible:
always

Steps to Reproduce:
1.
2.
3.

Actual results:
Missing subnet in discovery list cmd.

Expected results:
Subnet column displays 'libvirt', it should show actual subnet too like in UI:  libvirt (192.168.100.0/24)


Additional info:

Comment 2 Sachin Ghai 2015-03-13 11:25:46 UTC
same issue with info command:

[root@dhcp201-163 boot]# hammer --username admin --password changeme discovery info --id 2
/usr/lib/ruby/gems/1.8/gems/hammer_cli-0.1.4/lib/hammer_cli/./apipie/../abstract.rb:68: warning: already initialized constant DEFAULT_LABEL_INDENT
ID:           2
Name:         mac525400e0e8ab
MAC:          52:54:00:e0:e8:ab
Last report:  2015/03/13 03:58:44
CPUs:         1
Memory:       995
Disk count:   1
Disks size:   8192
Subnet:       libvirt
Last Report:  2015/03/13 03:58:44
IP:           192.168.100.11
Model:        
Organization: Default Organization
Location:     Default Location

Comment 3 Sachin Ghai 2015-03-13 11:41:35 UTC
Created attachment 1001360 [details]
on UI subnet field shows subnet name along with actual subnet(192.168.100.0/24)

Comment 4 Bryan Kearney 2015-04-17 20:10:39 UTC
Created redmine issue http://projects.theforeman.org/issues/10193 from this bug

Comment 5 Bryan Kearney 2015-08-25 18:34:28 UTC
Upstream bug component is Discovery Plugin

Comment 6 Bryan Kearney 2016-12-29 09:20:40 UTC
Upstream bug assigned to orabin

Comment 7 Bryan Kearney 2016-12-29 09:20:44 UTC
Upstream bug assigned to orabin

Comment 8 Bryan Kearney 2017-01-02 17:20:49 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/10193 has been resolved.

Comment 9 Bryan Kearney 2017-08-01 20:03:35 UTC
The fix to this bug will be delivered with release 6.3 of Satellite.


Note You need to log in before you can comment on or make changes to this bug.