Bug 1196758 - Containers api should be /docker/api/v2/conatiners and not /api/v2/containers
Summary: Containers api should be /docker/api/v2/conatiners and not /api/v2/containers
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: API
Version: Nightly
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: Unspecified
Assignee: David Davis
QA Contact: jaudet
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks: 1190289
TreeView+ depends on / blocked
 
Reported: 2015-02-26 16:36 UTC by Og Maciel
Modified: 2017-02-23 20:25 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 05:28:00 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 9562 0 None None None 2016-04-22 16:40:58 UTC
Red Hat Product Errata RHSA-2015:1592 0 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 09:04:35 UTC

Description Og Maciel 2015-02-26 16:36:49 UTC
Description of problem:

While testing the API layer for Docker containers QE noticed that the following API call was returning HTML content and a 404 response.

GET /api/v2/containers 

Here's how you can test this using our Python library (install nailgun from Pypi with pip install nailgun)

>>> from nailgun import client
>>> response = client.get(
...     'https://cloud-qe-8.idmqe.lab.eng.bos.redhat.com/api/v2/containers',
...     auth=('admin', 'changeme'),
...     verify=False,
... )
/home/ichimonji10/.virtualenvs/robottelo/lib/python2.7/site-packages/requests/packages/urllib3/connectionpool.py:734: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.org/en/latest/security.html
  InsecureRequestWarning)
>>> response.status_code
404
>>> response.headers['content-type']
'text/html; charset=utf-8'
>>> response.text
u'<!DOCTYPE html>\n<html>\n<head>\n  <title>The page you were looking for doesn\'t exist (404)</title>\n  <style type="text/css">\n    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }\n    div.dialog {\n      width: 25em;\n      padding: 0 4em;\n      margin: 4em auto 0 auto;\n      border: 1px solid #ccc;\n      border-right-color: #999;\n      border-bottom-color: #999;\n    }\n    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }\n  </style>\n</head>\n\n<body>\n  <!-- This file lives in public/404.html -->\n  <div class="dialog">\n    <h1>The page you were looking for doesn\'t exist.</h1>\n    <p>You may have mistyped the address or the page may have moved.</p>\n  </div>\n</body>\n</html>\n'

Something else that is interesting is that the web UI doesn't seem to be using this API at all unless I'm missing something.


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


    candlepin-0.9.43-1.el7.noarch
    candlepin-common-1.0.21-1.el7.noarch
    candlepin-guice-3.0-2_redhat_1.el7.noarch
    candlepin-scl-1-5.el7.noarch
    candlepin-scl-quartz-2.1.5-6.el7.noarch
    candlepin-scl-rhino-1.7R3-3.el7.noarch
    candlepin-scl-runtime-1-5.el7.noarch
    candlepin-selinux-0.9.43-1.el7.noarch
    candlepin-tomcat-0.9.43-1.el7.noarch
    elasticsearch-0.90.10-7.el7.noarch
    katello-2.2.0.5-1.el7sat.noarch
    katello-certs-tools-2.2.1-1.el7sat.noarch
    katello-common-2.2.0.5-1.el7sat.noarch
    katello-default-ca-1.0-1.noarch
    katello-installer-2.2.0.7-1.el7sat.noarch
    katello-installer-base-2.2.0.7-1.el7sat.noarch
    katello-server-ca-1.0-1.noarch
    pulp-docker-plugins-0.2.2-1.el7sat.noarch
    pulp-katello-0.3-4.el7sat.noarch
    pulp-nodes-common-2.6.0-0.7.beta.el7sat.noarch
    pulp-nodes-parent-2.6.0-0.7.beta.el7sat.noarch
    pulp-puppet-plugins-2.6.0-0.7.beta.el7sat.noarch
    pulp-puppet-tools-2.6.0-0.7.beta.el7sat.noarch
    pulp-rpm-plugins-2.6.0-0.7.beta.el7sat.noarch
    pulp-selinux-2.6.0-0.7.beta.el7sat.noarch
    pulp-server-2.6.0-0.7.beta.el7sat.noarch
    python-gofer-qpid-2.5.3-1.el7sat.noarch
    python-isodate-0.5.0-4.pulp.el7sat.noarch
    python-kombu-3.0.24-5.pulp.el7sat.noarch
    python-pulp-bindings-2.6.0-0.7.beta.el7sat.noarch
    python-pulp-common-2.6.0-0.7.beta.el7sat.noarch
    python-pulp-docker-common-0.2.2-1.el7sat.noarch
    python-pulp-puppet-common-2.6.0-0.7.beta.el7sat.noarch
    python-pulp-rpm-common-2.6.0-0.7.beta.el7sat.noarch
    python-qpid-0.30-5.el7.noarch
    python-qpid-qmf-0.30-4.el7.x86_64
    qe-sat6-rhel71.usersys.redhat.com-qpid-broker-1.0-1.noarch
    qe-sat6-rhel71.usersys.redhat.com-qpid-client-cert-1.0-1.noarch
    qpid-cpp-client-0.30-6.el7.x86_64
    qpid-cpp-client-devel-0.30-6.el7.x86_64
    qpid-cpp-server-0.30-6.el7.x86_64
    qpid-cpp-server-linearstore-0.30-6.el7.x86_64
    qpid-java-client-0.30-3.el7.noarch
    qpid-java-common-0.30-3.el7.noarch
    qpid-proton-c-0.9-1.20150223.el7.x86_64
    qpid-qmf-0.30-4.el7.x86_64
    qpid-tools-0.30-3.el7.noarch
    ruby193-rubygem-katello-2.2.0.15-1.el7sat.noarch
    ruby193-rubygem-qpid_messaging-0.30.0-1.el7sat.x86_64
    rubygem-hammer_cli_katello-0.0.7.2-1.el7sat.noarch
    rubygem-smart_proxy_pulp-1.0.1.1-1.el7.noarch

How reproducible:


Steps to Reproduce:
1. Create a new docker-based compute resource
2. List all containers from the Containers > All containers menu
3.

Actual results:

The page is populated (assuming you do have docker images already) but under the hood the API is returning HTML output


Expected results:

API should be returning JSON

Additional info:

Comment 1 Og Maciel 2015-02-26 16:44:06 UTC
Ok, so this seems to be a documentation issue as the path should be /docker/api/v2/containers

Comment 3 Bryan Kearney 2015-03-01 09:05:30 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9562 has been closed
-------------
David Davis
Applied in changeset commit:foreman-docker|291115972b9a4d3f3a551dec8fb554bc92ae62ee.

Comment 6 jaudet 2015-03-04 15:59:21 UTC
Tested against Satellite-6.1.0-RHEL-6-20150303.0 and Satellite-6.1.0-RHEL-7-20150303.0.

Comment 7 Bryan Kearney 2015-08-11 13:19:14 UTC
This bug is slated to be released with Satellite 6.1.

Comment 8 errata-xmlrpc 2015-08-12 05:28:00 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://access.redhat.com/errata/RHSA-2015:1592


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