Project

General

Profile

Bug #11671

docker registry command does not provide IDs

Added by Partha Aji over 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1259498
Description of problem:

The new 'docker registry' command has several subcommands that can use the ID for a registry to perform several CRUD actions but the ID is never displayed by 'docker registry list' or 'docker registry info'. Furthermore, the following fields are also never displayed: description, username, password

See the output of using hammer registry below:

hammer> docker registry
Usage:
docker registry [OPTIONS] SUBCOMMAND [ARG] ...

Parameters:
SUBCOMMAND subcommand
[ARG] ... subcommand arguments

Subcommands:
create Create a docker registry
delete Delete a docker registry
info Show a docker registry
list List all docker registries
update Update a docker registry

Options:
h, --help print help
hammer> docker registry create
Could not create the registry:
Missing arguments for 'name', 'url'
hammer> docker registry create --name="Red Hat Docker Registry" --url="https://registry.access.redhat.com"
Docker registry created
hammer> docker registry list
------------------------|----------------------------------

NAME | URL
------------------------|-----------------------------------
Red Hat Docker Registry | https://registry.access.redhat.com
------------------------|-----------------------------------
hammer> docker registry info -h
Usage:
docker registry info [OPTIONS]

Options:
--id ID
--name NAME Name to search by
-h, --help print help
hammer> docker registry info --name='Red Hat Docker Registry'
Name: Red Hat Docker Registry
URL: https://registry.access.redhat.com
Created at: 2015/09/02 18:44:24
Updated at: 2015/09/02 18:44:24

hammer> docker registry update -h
Usage:
docker registry update [OPTIONS]

Options:
--description DESCRIPTION
--id ID
--name NAME Name to search by
--new-name NEW_NAME
--password PASSWORD
--url URL
--username USERNAME
-h, --help print help
hammer> docker registry update --name='Red Hat Docker Registry' --description='Updated this field'
Docker registry updated
hammer> docker registry info --name='Red Hat Docker Registry'
Name: Red Hat Docker Registry
URL: https://registry.access.redhat.com
Created at: 2015/09/02 18:44:24
Updated at: 2015/09/02 18:47:19

hammer> docker registry update --name='Red Hat Docker Registry' --username='Updated this field'
Docker registry updated
hammer> docker registry info --name='Red Hat Docker Registry'
Name: Red Hat Docker Registry
URL: https://registry.access.redhat.com
Created at: 2015/09/02 18:44:24
Updated at: 2015/09/02 18:47:46

hammer> docker registry update --name='Red Hat Docker Registry' --url='http://localhost'
Docker registry updated
hammer> docker registry info --name='Red Hat Docker Registry'
Name: Red Hat Docker Registry
URL: http://localhost
Created at: 2015/09/02 18:44:24
Updated at: 2015/09/02 18:48:15

hammer> docker registry update --name='Red Hat Docker Registry' --new-name='RH'
Docker registry updated
hammer> docker registry info --name='Red Hat Docker Registry'
Error: registry not found
hammer> docker registry info --name='RH'
Name: RH
URL: http://localhost
Created at: 2015/09/02 18:44:24
Updated at: 2015/09/02 18:48:46

hammer> docker registry delete -h
Usage:
docker registry delete [OPTIONS]

Options:
--id ID
--name NAME Name to search by
h, --help print help
hammer> docker registry delete --name='RH'
Docker registry deleted
hammer> docker registry list
-----|---

NAME | URL
-----|----
hammer> docker registry info --name='RH'
Error: registry not found

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

This is Sat6.1.2-SNAP1

  • docker-1.7.1-108.el7.x86_64
  • docker-selinux-1.7.1-108.el7.x86_64
  • pulp-docker-plugins-0.2.5-1.el7sat.noarch
  • python-pulp-docker-common-0.2.5-1.el7sat.noarch
  • ruby193-rubygem-docker-api-1.17.0-1.1.el7sat.noarch
  • ruby193-rubygem-foreman_docker-1.2.0.20-1.el7sat.noarch
  • rubygem-hammer_cli_foreman_docker-0.0.3.9-1.el7sat.noarch

How reproducible:

Steps to Reproduce:
1. Use the 'docker registry' command to create a new registry and then attempt to use any of its subcommand and pass the ID (i.e. --id) argument to perform any actions against the registry
2.
3.

Actual results:

The ID is never displayed by hammer and therefore one cannot use --id with its subcommands

Expected results:

Additional info:

Associated revisions

Revision f5a92241 (diff)
Added by Partha Aji over 7 years ago

Fixes #11671- Added ID information to registry commands

Revision c59ad199
Added by David Davis over 7 years ago

Merge pull request #14 from parthaa/add-id

Fixes #11671- Added ID information to registry commands

Revision eac921d6 (diff)
Added by Partha Aji over 7 years ago

Refs #11671 - Added description field for registry

Revision 0cb319e4
Added by David Davis over 7 years ago

Merge pull request #15 from parthaa/add-description

Refs #11671 - Added description field for registry

History

#1 Updated by Partha Aji over 7 years ago

  • Pull request https://github.com/theforeman/hammer_cli_foreman_docker/pull/14 added
  • Pull request deleted ()

#2 Updated by Partha Aji over 7 years ago

  • Project changed from Katello to Docker

#3 Updated by Partha Aji over 7 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Partha Aji

#4 Updated by David Davis almost 7 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF