Project

Profile

Help

Issue #582

format of repo fields list for pulp-admin --fields option

Added by igulina@redhat.com almost 7 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.5
Platform Release:
2.6.1
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Description of problem:
--fields option for pulp-admin doesn't except names of repo's fields as they are written in the output, i.e. with capital letters and spaces

1) pulp-admin rpm repo list --fields "id,description,display name" doesn't work
but
pulp-admin rpm repo list --fields "id,description,display_name" is ok

2) pulp-admin rpm repo list --fields 'Id,Description' doesn't work
but
pulp-admin rpm repo list --fields 'id,description' is ok

Suggestion: It's not obvious that repo's fields should be written in lowcase and with underscore instead of space. If it's difficult to fix, let's put an example in --help output:

pulp-admin repo list --help

Command: list
Description: lists repositories on the Pulp server

Available Arguments:

--summary, -s - if specified, a condensed view with just the repository ID and
                name is displayed
--details     - if specified, detailed configuration information is displayed
                for each repository
--fields      - comma-separated list of repository fields, for example   
                "id,description,display_name,content_unit_counts"; if
                specified, only the given fields will displayed

Version-Release number of selected component (if applicable):

rpm -qa | grep pulp-server

pulp-server-2.5.0-0.8.beta.fc20.noarch

How reproducible:
always

Steps to reproduce:

pulp-admin repo list

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Display Name: gena
Description: None
Content Unit Counts:

pulp-admin repo list --help

Command: list
Description: lists repositories on the Pulp server

Available Arguments:

--summary, -s - if specified, a condensed view with just the repository ID and
                name is displayed
--details     - if specified, detailed configuration information is displayed
                for each repository
--fields      - comma-separated list of repository fields; if specified, only
                the given fields will displayed

pulp-admin repo list --fields "id,description"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Description: None

pulp-admin repo list --fields "id,description,display name"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Description: None

pulp-admin repo list --fields "id,description,display_name"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Description: None
Display Name: gena

pulp-admin repo list --fields "id,description,display_name,content_unit_counts"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Content Unit Counts:
Description: None
Display Name: gena

pulp-admin repo list --fields "Id,description"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena
Description: None

pulp-admin repo list --fields "Id,Description"

--------------------------------------------------------------------
Repositories
--------------------------------------------------------------------

Id: gena

+ This bug was cloned from Bugzilla Bug #1154150 +

Associated revisions

Revision ec802f5a View on GitHub
Added by ipanova@redhat.com over 6 years ago

Updated docs with --field option.

closes #582

Revision ec802f5a View on GitHub
Added by ipanova@redhat.com over 6 years ago

Updated docs with --field option.

closes #582

History

#1 Updated by jortel@redhat.com almost 7 years ago

Document this behavior. Ensure CLI replacement does not do this.

+ This comment was cloned from Bugzilla #1154150 comment 1 +

#2 Updated by ipanova@redhat.com almost 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

#3 Updated by ipanova@redhat.com almost 7 years ago

  • Status changed from ASSIGNED to POST

#4 Updated by ipanova@redhat.com almost 7 years ago

  • Status changed from POST to MODIFIED

#5 Updated by ipanova@redhat.com almost 7 years ago

  • Platform Release set to 2.6.1

#6 Updated by bmbouter over 6 years ago

  • Severity changed from Medium to 2. Medium

#7 Updated by bcourt over 6 years ago

  • Status changed from MODIFIED to 5

#8 Updated by igulina@redhat.com over 6 years ago

  • Status changed from 5 to ASSIGNED

rpm -qa pulp-server

pulp-server-2.6.1-0.2.beta.fc20.noarch

Examples were added into --field option and available on

- pulp-admin auth user list --help
- pulp-admin auth role list --help
- pulp-consumer puppet repos --help
- pulp-consumer rpm repos --help
- pulp-admin consumer list --help
- pulp-admin bindings search --help
- pulp-admin docker repo list --help
- pulp-admin node list --help and others

for example: >> pulp-admin node list --feilds
Command: list
Description: list child nodes

Available Arguments:

--fields   - comma-separated list of consumer fields; if specified only the
             given fields will be displayed. Example: "id,display_name".
...

Docs to update:
http://pulp.readthedocs.org/en/latest/user-guide/admin-client/nodes.html?highlight=fields
http://pulp.readthedocs.org/en/latest/user-guide/admin-client/consumer.html?highlight=fields

#9 Updated by ipanova@redhat.com over 6 years ago

  • Status changed from ASSIGNED to POST

#10 Updated by ipanova@redhat.com over 6 years ago

  • Status changed from POST to 5

#11 Updated by ipanova@redhat.com over 6 years ago

  • Status changed from 5 to MODIFIED
  • % Done changed from 0 to 100

#12 Updated by dkliban@redhat.com over 6 years ago

  • Status changed from MODIFIED to 5

#13 Updated by pthomas@redhat.com over 6 years ago

  • Status changed from 5 to 6

verified

https://pulp.readthedocs.org/en/latest/user-guide/general-reference.html#criteria

#14 Updated by dkliban@redhat.com about 6 years ago

  • Status changed from 6 to CLOSED - CURRENTRELEASE

#16 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF