Bug 811594 - [RFE] facts command should default to list
Summary: [RFE] facts command should default to list
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: subscription-manager
Version: 5.9
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: 5.9
Assignee: William Poteat
QA Contact: Entitlement Bugs
URL:
Whiteboard:
Depends On:
Blocks: 771748
TreeView+ depends on / blocked
 
Reported: 2012-04-11 13:48 UTC by Bryan Kearney
Modified: 2013-01-08 03:50 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-01-08 03:50:21 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2013:0033 0 normal SHIPPED_LIVE subscription-manager bug fix and enhancement update 2013-01-08 08:38:27 UTC

Description Bryan Kearney 2012-04-11 13:48:43 UTC
listing is the natural default. Please make it the default instead of erroring.

Comment 1 Bryan Kearney 2012-04-11 13:53:17 UTC
please make --list the default for the repos command as well.

Comment 2 RHEL Program Management 2012-04-11 13:57:47 UTC
This request was evaluated by Red Hat Product Management for inclusion
in a Red Hat Enterprise Linux release.  Product Management has
requested further review of this request by Red Hat Engineering, for
potential inclusion in a Red Hat Enterprise Linux release for currently
deployed products.  This request is not yet committed for inclusion in
a release.

Comment 3 Alex Wood 2012-04-11 20:31:06 UTC
The config, servicelevel, and release commands all use --list.  Would you like to review them while I'm working on this issue?

Comment 4 Bryan Kearney 2012-04-11 20:43:40 UTC
Lets do the same for config. The others, I like the default of show.

But, if you are in there...

[root@bkearney src]# ./subscription-manager service-level 
Current service level: 
[root@bkearney src]# ./subscription-manager release
Release not set


Seems like we could be consistent on that^^^

Comment 5 Alex Wood 2012-04-17 18:15:38 UTC
commit 62fe5323b17443ec873b10ee45c63e40ca1d21bd
Author: Alex Wood <awood>
Date:   Fri Apr 6 22:42:07 2012 -0400

    811594: The config, repos, and facts commands should default to --list if no options are provided.

Comment 7 J.C. Molet 2012-06-19 15:02:29 UTC
[root@jmolet-vm0 rhsm]# subscription-manager facts
cpu.core(s)_per_socket: 2
cpu.cpu(s): 2
cpu.cpu_socket(s): 1
distribution.id: Tikanga
distribution.name: Red Hat Enterprise Linux Server
distribution.version: 5.8
dmi.bios.address: 0xe8000
dmi.bios.bios_revision: 1.0
dmi.bios.relase_date: 01/01/2007
dmi.bios.rom_size: 64 KB
dmi.bios.runtime_size: 96 KB
dmi.bios.vendor: Bochs
...truncated..


[root@jmolet-vm0 rhsm]#  diff <(subscription-manager facts --list) <(subscription-manager facts)

#NO DIFF

subscription-manager facts is shown to be the same as with --list.

========================================

[root@jmolet-vm0 rhsm]# subscription-manager config
[server]
   baseurl = [https://cdn.redhat.com]
   ca_cert_dir = [/etc/rhsm/ca/]
   certfrequency = [240]
   consumercertdir = [/etc/pki/consumer]
   entitlementcertdir = [/etc/pki/entitlement]
   healfrequency = [1440]
   hostname = mgmt5.rhq.lab.eng.bos.redhat.com
   insecure = [0]
   manage_repos = [1]
   port = [8443]
...truncated...

[root@jmolet-vm0 rhsm]# diff <(subscription-manager config --list) <(subscription-manager config)

#NO DIFF

subscription-manager config is shown to work the same as with --list

=========================================


[root@jmolet-vm0 rhsm]# subscription-manager repos
Error: This command requires that you use a --list option or specify a repo with --enable or --disable.


^^ repos does not run with an implied --list. Throwing back.

Comment 8 J.C. Molet 2012-06-19 15:22:28 UTC
^^ Comment 7 was tested with:
 subscription-manager-1.0.4-1.git.3.1768f90.el5
 python-rhsm-1.0.2-1.git.1.4659671.el5

Comment 9 William Poteat 2012-06-21 17:39:17 UTC
master commit 38450bb2ede02f8b5d16c90efe9af2e12f77c545

Comment 10 J.C. Molet 2012-06-22 13:46:59 UTC
Testing with Version: 
subscription-manager-1.0.4-1.git.11.02cf004.el5
python-rhsm-1.0.2-1.git.2.d5c3d05.el5


[root@jmolet-vm0 rpms]# subscription-manager register
Username: stage_test_12
Password: 
The system has been registered with id: 1fd10e81-c5bf-4b00-8d1a-3283a35179b4 
[root@jmolet-vm0 rpms]# subscription-manager subscribe --auto
Installed Product Current Status:
Product Name:         	Red Hat Enterprise Linux Server
Status:               	Subscribed

[root@jmolet-vm0 rpms]# subscription-manager repos
+----------------------------------------------------------+
    Entitled Repositories in /etc/yum.repos.d/redhat.repo
+----------------------------------------------------------+
Repo Id:              	rhel-5-server-rpms
Repo Name:            	Red Hat Enterprise Linux 5 Server (RPMs)
Repo Url:             	https://cdn.redhat.com/content/dist/rhel/server/5/$releasever/$basearch/os
Enabled:              	1

Repo Id:              	rhel-5-server-cf-tools-1-source-rpms
Repo Name:            	Red Hat CloudForms Tools for RHEL 5 (Source RPMs)
Repo Url:             	https://cdn.redhat.com/content/dist/rhel/server/5/$releasever/$basearch/cf-tools/1.0/source/SRPMS
Enabled:              	0
...truncated...


subscription-manager repos now performs the same function as repos --list.

marking VERIFIED.

Comment 12 errata-xmlrpc 2013-01-08 03:50:21 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2013-0033.html


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