Bug 1377952 - atomic verify on non-Atomic OS always says too few arguments
Summary: atomic verify on non-Atomic OS always says too few arguments
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 7
Classification: Red Hat
Component: atomic
Version: 7.3
Hardware: x86_64
OS: Linux
high
high
Target Milestone: rc
: ---
Assignee: Brent Baude
QA Contact: atomic-bugs@redhat.com
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-21 07:12 UTC by Alex Jia
Modified: 2016-12-06 17:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-12-06 17:41:56 UTC
Target Upstream Version:

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


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:2857 0 normal SHIPPED_LIVE atomic bug fix and enhancement update 2016-12-06 22:40:27 UTC

Description Alex Jia 2016-09-21 07:12:07 UTC 
Description of problem:
atomic verify on non-Atomic OS always says too few arguments.


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

# cat /etc/redhat-release 
Red Hat Enterprise Linux Server release 7.2 (Maipo)

# rpm -q atomic ostree
atomic-1.12.3-2.el7.x86_64
ostree-2016.10-1.atomic.el7.x86_64

How reproducible:
always

Steps to Reproduce:
1. docker pull busybox
2. atomic verify busybox


Actual results:

# atomic verify busybox
/usr/bin/atomic verify busybox: too few arguments
Try '/usr/bin/atomic verify busybox --help' for more information.

Expected results:


Additional info:

it's okay to append --no-validate option.

# atomic verify -v --no-validate busybox

busybox contains the following images:

     Local Version                                                                       Latest Version                                                                       
     -------------                                                                       --------------                                                                       
     busybox (sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4)   busybox (sha256:8ddc19f16526912237dd8af81971d5e4dd0587907234be2b83e249518d5b673f)   *
     busybox (sha256:8ddc19f16526912237dd8af81971d5e4dd0587907234be2b83e249518d5b673f)   busybox (None)                                                                      *

     * = version difference


# atomic  verify -h
usage: atomic verify [-h] [-v] [--no-validate] image

positional arguments:
  image          container image

optional arguments:
  -h, --help     show this help message and exit
  -v, --verbose  Report status of each layer
  --no-validate  disable validating system images
Comment 2 Daniel Walsh 2016-10-10 15:55:33 UTC 
Fixed in atomic-1.13
Comment 5 Daniel Walsh 2016-10-28 11:54:48 UTC 
Do we need a new version of ostree for RHEL7.2?  Should we add this as requires for atomic command?
Comment 6 Alex Jia 2016-11-01 08:08:15 UTC 
[root@bootp-73-3-203 ~]# cat /etc/redhat-release 
Red Hat Enterprise Linux Server release 7.3 (Maipo)

[root@bootp-73-3-203 ~]# rpm -q atomic skopeo
atomic-1.13.5-1.el7.x86_64
skopeo-0.1.17-0.5.git1f655f3.el7.x86_64

[root@bootp-73-3-203 ~]# atomic --debug verify busybox
{
    "index": 0,
    "Name": "busybox:latest",
    "Parent": "",
    "Version": "",
    "Id": "e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba",
    "RepoTags": [
        "busybox:latest"
    ]
}
[
    {
        "search": true,
        "hostname": "registry-1.docker.io",
        "name": "docker.io",
        "secure": true
    },
    {
        "search": true,
        "hostname": "registry.access.redhat.com",
        "name": "registry.access.redhat.com",
        "secure": true
    }
]
URL: https://None/v2/
GET_URL: https://None/v2/
GET_HEADER: {'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
('Connection aborted.', gaierror(-2, 'Name or service not known'))
/usr/bin/atomic --debug verify busybox: too few arguments
Try '/usr/bin/atomic --debug verify busybox --help' for more information.
Traceback (most recent call last):
  File "/usr/bin/atomic", line 186, in <module>
    sys.exit(_func())
  File "/usr/lib/python2.7/site-packages/Atomic/verify.py", line 109, in verify
    base_images = self.get_tagged_images(uniq_names, layers)
  File "/usr/lib/python2.7/site-packages/Atomic/verify.py", line 133, in get_tagged_images
    ri.ping()
  File "/usr/lib/python2.7/site-packages/Atomic/discovery.py", line 343, in ping
    if rc in [2, 4] and not self.rc.hostname_has_port():
  File "/usr/lib/python2.7/site-packages/Atomic/discovery.py", line 178, in hostname_has_port
    if len(self.hostname.split(':')) > 1:
AttributeError: 'NoneType' object has no attribute 'split'
Comment 7 Daniel Walsh 2016-11-01 12:27:15 UTC 
Brent it looks like self.hostname is never set?
Comment 8 Alex Jia 2016-11-01 13:17:26 UTC 
I saw a different result between RHEL7.3 and 7.3.0-3 atomic cloud images.

1. in RHEL7.3
please see Comment 6


2. in 7.3.0-3 atomic cloud images

[root@atomic-00 cloud-user]# cat /etc/redhat-release 
Red Hat Enterprise Linux Atomic Host release 7.3

[root@atomic-00 cloud-user]# atomic host status
State: idle
Deployments:
?rhel-atomic-host-ostree:rhel-atomic-host/7/x86_64/standard
       Version: 7.3 (2016-10-26 14:24:09)
        Commit: 90c9735becfff1c55c8586ae0f2c904bc0928f042cd4d016e9e0e2edd16e5e97
        OSName: rhel-atomic-host
      Unlocked: development

[root@atomic-00 cloud-user]# rpm -q atomic skopeo docker
atomic-1.13.5-1.el7.x86_64
skopeo-0.1.17-0.5.git1f655f3.el7.x86_64
docker-1.10.3-57.el7.x86_64

[root@atomic-00 cloud-user]# atomic verify busybox

NOTE: there is no any output in here

[root@atomic-00 cloud-user]# echo $?
0
[root@atomic-00 cloud-user]# atomic --debug verify busybox
{
    "index": 0,
    "Name": "docker.io/busybox:latest",
    "Parent": "",
    "Version": "",
    "Id": "e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba",
    "RepoTags": [
        "docker.io/busybox:latest"
    ]
}
[
    {
        "search": true,
        "hostname": "registry-1.docker.io",
        "name": "docker.io",
        "secure": true
    },
    {
        "search": true,
        "hostname": "registry.access.redhat.com",
        "name": "registry.access.redhat.com",
        "secure": true
    }
]
URL: https://registry-1.docker.io/v2/
GET_URL: https://registry-1.docker.io/v2/
GET_HEADER: {'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
Set token_scope to repository:library/busybox:pull
URL: http://registry-1.docker.io/v2/
GET_URL: https://auth.docker.io/token?service=registry.docker.io&scope=repository:library/busybox:pull
GET_HEADER: {'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
Set token to eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA
GET_URL: http://registry-1.docker.io/v2/
GET_HEADER: {'Authorization': 'Bearer eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA', 'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
URL: http://registry-1.docker.io/v2/
GET_URL: http://registry-1.docker.io/v2/
GET_HEADER: {'Authorization': 'Bearer eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA', 'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
GET_URL: http://registry-1.docker.io/v2/library/busybox/manifests/latest
GET_HEADER: {'Authorization': 'Bearer eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA', 'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
GET_URL: http://registry-1.docker.io/v2/library/busybox/blobs/sha256:e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba
GET_HEADER: {'Authorization': 'Bearer eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA', 'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
GET_URL: http://registry-1.docker.io/v2/library/busybox/tags/list
GET_HEADER: {'Authorization': 'Bearer eyJhbGciOiJFUzI1NiIsInR5cCI6IkpXVCIsIng1YyI6WyJNSUlDTHpDQ0FkU2dBd0lCQWdJQkFEQUtCZ2dxaGtqT1BRUURBakJHTVVRd1FnWURWUVFERXp0Uk5Gb3pPa2RYTjBrNldGUlFSRHBJVFRSUk9rOVVWRmc2TmtGRlF6cFNUVE5ET2tGU01rTTZUMFkzTnpwQ1ZrVkJPa2xHUlVrNlExazFTekFlRncweE5qQTFNekV5TXpVNE5UZGFGdzB4TnpBMU16RXlNelU0TlRkYU1FWXhSREJDQmdOVkJBTVRPMUV6UzFRNlFqSkpNenBhUjFoT09qSlhXRTA2UTBWWFF6cFVNMHhPT2tvMlYxWTZNbGsyVHpwWlFWbEpPbGhQVTBRNlZFUlJTVG8wVWtwRE1Ga3dFd1lIS29aSXpqMENBUVlJS29aSXpqMERBUWNEUWdBRVo0NkVLV3VKSXhxOThuUC9GWEU3U3VyOXlkZ3c3K2FkcndxeGlxN004VHFUa0N0dzBQZm1SS2VLdExwaXNTRFU4LzZseWZ3QUFwZWh6SHdtWmxZR2dxT0JzakNCcnpBT0JnTlZIUThCQWY4RUJBTUNCNEF3RHdZRFZSMGxCQWd3QmdZRVZSMGxBREJFQmdOVkhRNEVQUVE3VVROTFZEcENNa2t6T2xwSFdFNDZNbGRZVFRwRFJWZERPbFF6VEU0NlNqWlhWam95V1RaUE9sbEJXVWs2V0U5VFJEcFVSRkZKT2pSU1NrTXdSZ1lEVlIwakJEOHdQWUE3VVRSYU16cEhWemRKT2xoVVVFUTZTRTAwVVRwUFZGUllPalpCUlVNNlVrMHpRenBCVWpKRE9rOUdOemM2UWxaRlFUcEpSa1ZKT2tOWk5Vc3dDZ1lJS29aSXpqMEVBd0lEU1FBd1JnSWhBTzYxSWloN1FUcHNTMFFIYUNwTDFZTWNMMnZXZlNydlhHbHpSRDEwN2NRUEFpRUFtZXduelNYRHplRGxqcDc4T1NsTFFzbnROYWM5eHRyYW0xU0kxY0ZXQ2tJPSJdfQ.eyJhY2Nlc3MiOlt7InR5cGUiOiJyZXBvc2l0b3J5IiwibmFtZSI6ImxpYnJhcnkvYnVzeWJveCIsImFjdGlvbnMiOlsicHVsbCJdfV0sImF1ZCI6InJlZ2lzdHJ5LmRvY2tlci5pbyIsImV4cCI6MTQ3ODAwNjQwMSwiaWF0IjoxNDc4MDA2MTAxLCJpc3MiOiJhdXRoLmRvY2tlci5pbyIsImp0aSI6IjBWLThPeDRmTGw4SUU4Q3RyaGlVIiwibmJmIjoxNDc4MDA2MTAxLCJzdWIiOiIifQ.ap_CNTyHRIw7ozT4drJupWPdtM7BS7maKFL9sIDIgj6-z0eDDmWUGVDB1pJ3RdDG0vByXj_hAzl5epSTNmA9LA', 'Accept': '[application/vnd.oci.image.manifest.v1+json,application/vnd.docker.distribution.manifest.v2+json,application/vnd.docker.distribution.manifest.v1+prettyjws,application/vnd.docker.distribution.manifest.v1+json]'}
GET_VERIFY: True
{
    "index": 0,
    "Name": "docker.io/busybox:latest",
    "Parent": "",
    "Version": "e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba",
    "Remote Version": "e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba",
    "Id": "e02e811dd08fd49e7f6032625495118e63f597eb150403d02e3238af1df240ba",
    "RepoTags": [
        "docker.io/busybox:latest"
    ]
}
Comment 9 Daniel Walsh 2016-11-01 13:19:43 UTC 
Yes it works fine on my laptop also.  The problem is in some codepaths we end up without a hostname, which is causing the errors.  I believe this is a configuration issue.  Something to do with registries?
Comment 10 Brent Baude 2016-11-01 13:29:15 UTC 
can i get access to this for debug?
Comment 11 Alex Jia 2016-11-01 23:46:41 UTC 
(In reply to Brent Baude from comment #10)
> can i get access to this for debug?

sure, send a email to you.
Comment 12 Brent Baude 2016-11-03 14:43:26 UTC 
Fixed in https://github.com/projectatomic/atomic/pull/744.  Will get a new atomic release and package cut.
Comment 13 Brent Baude 2016-11-03 16:20:18 UTC 
Rebuilt in -> https://brewweb.engineering.redhat.com/brew/buildinfo?buildID=521700
Comment 14 Daniel Walsh 2016-11-03 17:46:19 UTC 
Fixed in atomic-1.13.7
Comment 15 Alex Jia 2016-11-04 02:14:50 UTC 
It works well in atomic-1.13.7-2, so moving the bug to VERIFIED status.

# rpm -q atomic
atomic-1.13.7-2.el7.x86_64

# atomic verify -v busybox
Trying docker.io/library/busybox:latest

busybox:latest contains the following images:

NAME                           LOCAL VERSION        REMOTE VERSION       DIFFERS
busybox:latest                 e02e811dd08fd49e7f60 e02e811dd08fd49e7f60 No


# atomic verify -v --no-validate busybox
Trying docker.io/library/busybox:latest


busybox:latest contains the following images:

NAME                           LOCAL VERSION        REMOTE VERSION       DIFFERS
busybox:latest                 e02e811dd08fd49e7f60 e02e811dd08fd49e7f60 No


# atomic verify registry.access.redhat.com/rhel7

registry.access.redhat.com/rhel7:latest contains the following images:

NAME                           LOCAL VERSION        REMOTE VERSION       DIFFERS
rhel7/rhel                     rhel7/rhel-7.2-104   rhel7/rhel-7.3-45    Yes


NOTE: it seems we have moved related commands of some images  into atomic images, e.g atomic images {help, info, prune, verify, version}, my question is whether atomic verify is completely equal to atomic images verify, if so, we should remove atomic verify from atomic commands, thanks.

# atomic images verify -v busybox
Trying docker.io/library/busybox:latest

busybox:latest contains the following images:

NAME                           LOCAL VERSION        REMOTE VERSION       DIFFERS
busybox:latest                 e02e811dd08fd49e7f60 e02e811dd08fd49e7f60 No
Comment 16 Brent Baude 2016-11-04 13:59:07 UTC 
Alex, what is the need info setting marked to me for?  Please elaborate if you need anything.
Comment 17 Alex Jia 2016-11-07 02:19:06 UTC 
(In reply to Brent Baude from comment #16)
> Alex, what is the need info setting marked to me for?  Please elaborate if
> you need anything.

Brent,  it seems we have moved related commands of some images into atomic images, e.g atomic images {help, info, prune, verify, version}, my question is whether atomic verify is completely equal to atomic images verify, if so, we should remove atomic verify from atomic commands, thanks.
Comment 18 Brent Baude 2016-11-07 14:24:04 UTC 
Dan has made this decision. I'll set needinfo from him.
Comment 19 Daniel Walsh 2016-11-07 14:50:29 UTC 
Alex Brent can we verify a Container?  Should we?

I have no problem moving it also.
Comment 20 Alex Jia 2016-11-08 13:42:34 UTC 
(In reply to Daniel Walsh from comment #19)
> Alex Brent can we verify a Container?  Should we?
> 
> I have no problem moving it also.

Daniel, atomic verify only works for an image.

# atomic verify -h
usage: atomic verify [-h] [--no-validate] [--storage STORAGE] [-v] image

positional arguments:
  image              container image

XXX

# docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
8e1c6a4d9856        busybox             "/bin/sh"           3 seconds ago       Up 1 seconds                            small_blackwell

# atomic verify 8e1c6a4d9856
Could not find any image matching '8e1c6a4d9856'
Comment 21 Daniel Walsh 2016-11-08 14:25:38 UTC 
Alex, we have removed atomic verify, it is in atomic image verify, they are the same.

 man atomic-verify
No manual entry for atomic-verify

atomic --help
usage: atomic [-h] [-v] [--debug] [-y]
              {containers,diff,help,images,info,install,mount,pull,push,upload,run,scan,sign,stop,storage,migrate,top,trust,uninstall,unmount,umount,update,verify,version}
              ...

Atomic Management Tool

positional arguments:
  {containers,diff,help,images,info,install,mount,pull,push,upload,run,scan,sign,stop,storage,migrate,top,trust,uninstall,unmount,umount,update,verify,version}
                        commands
    containers          operate on containers
    diff                Show differences between two container images, file
                        diff or RPMS.
    images              operate on images
    install             execute container image install method
    mount               mount container image to a specified directory
    pull                pull latest image from a repository
    push (upload)       push latest image to repository
    run                 execute container image run method
    scan                scan an image or container for CVEs
    sign                Sign an image
    stop                execute container image stop method
    storage (migrate)   manage container storage
    top                 Show top-like stats about processes running in
                        containers
    trust               Manage system container trust policy
    uninstall           execute container image uninstall method
    unmount (umount)    unmount container image
    update              pull latest container image from repository

optional arguments:
  -h, --help            show this help message and exit
  -v, --version         show atomic version and exit
  --debug               show debug messages
  -y, --assumeyes       automatically answer yes for all questions

I have not figured out how to get the links out of the main help page.  But their is no description.

Basically it is depracated, but I did not want to break existin functionality.
Comment 22 Alex Jia 2016-11-08 14:40:08 UTC 
(In reply to Daniel Walsh from comment #21)
          
> {containers,diff,help,images,info,install,mount,pull,push,upload,run,scan,
> sign,stop,storage,migrate,top,trust,uninstall,unmount,umount,update,verify,
> version}                                                               ^
                                                                         |
--------------------------------------------------------------------------

> 
> Atomic Management Tool
> 
> positional arguments:
>  
> {containers,diff,help,images,info,install,mount,pull,push,upload,run,scan,
> sign,stop,storage,migrate,top,trust,uninstall,unmount,umount,update,verify,
> version}                                                               ^
                                                                         |
--------------------------------------------------------------------------

# atomic verify -h
usage: atomic verify [-h] [--no-validate] [--storage STORAGE] [-v] image
XXX

We also should remove above. 


> 
> I have not figured out how to get the links out of the main help page.  But
> their is no description.
> 
> Basically it is depracated, but I did not want to break existin
> functionality.

Daniel, thanks for your details.
Comment 23 Daniel Walsh 2016-11-08 14:42:30 UTC 
The way argparse works in python this is difficult, it is just showing the command that you executed.

 atomic images verify -h
usage: atomic images verify [-h] [--no-validate] [--storage STORAGE] [-v]
                            image
...

They are actually executing the exact same code.
Comment 25 errata-xmlrpc 2016-12-06 17:41:56 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.

https://rhn.redhat.com/errata/RHBA-2016-2857.html
Note You need to log in before you can comment on or make changes to this bug.