broken endpoint usage in python-muranoclient and dashboard

Bug #1318794 reported by Alexander Tivelkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-muranoclient
Fix Released
Medium
Timur Nurlygayanov

Bug Description

Current implementation of murano-dashboard and murano CLI do not properly discover murano-api in Keystone: dashboard looks for service called "murano", CLI - for service 'metering' (this looks like an obvious copy-paste from Ceilometer), while the proper service type should be "application_catalog" (see murano-api/contrib/devstack/lib/murano for an example of the keystone registration script).
Another important issue is that the endpoint, registered in the catalog should include the version suffix (i.e. the murano endpoint should be "https://host:port/version", not just "https://host:port"). Current implementation has the version suffix right inside the implementations of specific client version.

Both these problems cause the current dashboard and CLI not to work with the murano-api registered in the keystone - only with the explicitly provided endpoint.

Changed in murano:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano-dashboard (master)

Fix proposed to branch: master
Review: https://review.openstack.org/93355

Changed in murano:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-muranoclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/93358

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano-api (master)

Fix proposed to branch: master
Review: https://review.openstack.org/93490

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-muranoclient (master)

Reviewed: https://review.openstack.org/93358
Committed: https://git.openstack.org/cgit/stackforge/python-muranoclient/commit/?id=b71e3fc1383ae0b9ef7b7fc368006bcf73e332e1
Submitter: Jenkins
Branch: master

commit b71e3fc1383ae0b9ef7b7fc368006bcf73e332e1
Author: Alexander Tivelkov <email address hidden>
Date: Mon May 12 13:04:58 2014 -0700

    Fixed Keystone endpoint query

    The proper service type value is used to get the endpoint from keystone for usage in CLI

    Change-Id: I019cb1d3b8b21d8e7a7d2240ccbb6a2f82162b82
    Partial-Bug: #1318794

Changed in murano:
assignee: Alexander Tivelkov (ativelkov) → Timur Sufiev (tsufiev-x)
Changed in murano:
assignee: Timur Sufiev (tsufiev-x) → Alexander Tivelkov (ativelkov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano-dashboard (master)

Reviewed: https://review.openstack.org/93355
Committed: https://git.openstack.org/cgit/stackforge/murano-dashboard/commit/?id=302530fa499950f7e6de953060038fca434d4cec
Submitter: Jenkins
Branch: master

commit 302530fa499950f7e6de953060038fca434d4cec
Author: Alexander Tivelkov <email address hidden>
Date: Mon May 12 12:54:22 2014 -0700

    Fixed Keystone endpoint query

    Murano-api is supposed to be registered in the keystone having the type of "application_catalog", not "murano",

    Change-Id: I28a25c7a75526d09a7e8d48a00a0e0aed84d6005
    Partial-Bug: #1318794

Changed in murano:
assignee: Alexander Tivelkov (ativelkov) → Timur Nurlygayanov (tnurlygayanov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano-api (master)

Reviewed: https://review.openstack.org/93490
Committed: https://git.openstack.org/cgit/stackforge/murano-api/commit/?id=dfe024d2d3325ba7ff840510d11b9b6e57e261b1
Submitter: Jenkins
Branch: master

commit dfe024d2d3325ba7ff840510d11b9b6e57e261b1
Author: Alexander Tivelkov <email address hidden>
Date: Tue May 13 12:26:10 2014 -0700

    Removed version prefix from the endpoint urls

    The service endpoint being registered in Keystone should not contain the version suffix,
    as the different versions of an API should co-exist at the same endpoint

    Partial-Bug: #1318794
    Closes-Bug: #1320937
    Change-Id: I853dfa30d1ad3538fad55826bb498dedae3e14f6

ruhe (ruhe)
Changed in murano:
milestone: juno-1 → juno-2
Changed in murano:
status: In Progress → Fix Committed
status: Fix Committed → Fix Released
milestone: juno-2 → none
milestone: none → juno-2
status: Fix Released → Fix Committed
ruhe (ruhe)
Changed in python-muranoclient:
importance: Undecided → Medium
assignee: nobody → Timur Nurlygayanov (tnurlygayanov)
milestone: none → 0.5.3
assignee: Timur Nurlygayanov (tnurlygayanov) → nobody
status: New → Fix Committed
assignee: nobody → Timur Nurlygayanov (tnurlygayanov)
ruhe (ruhe)
no longer affects: murano
ruhe (ruhe)
Changed in python-muranoclient:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.