glance-cache-manage doesn't have support for custom endpoint_type

Bug #1636755 reported by Denis Kostryukov
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
High
Kairat Kushaev

Bug Description

When we pass internalURL for glance-cache-manage by exporting environment variable "GLANCE_ENDPOINT_TYPE=internalURL",
it doesn't get the endpoint for internalURL from glance, rather it gets public endpoint.

The code snippet from the below link:

https://github.com/openstack/glance/blob/stable/mitaka/glance/common/auth.py#L215-L217

Here while retrieving endpoint for glance service, there is no feasibility to pass endpoint_type, so it fetches by default endpoint_type which is publicURL.

MOS 9.0

tags: added: customer-found sla1
Revision history for this message
Alexander Rubtsov (arubtsov) wrote :

sla1 for 9.0-updates

Changed in mos:
importance: Undecided → High
milestone: none → 9.2
Changed in mos:
assignee: nobody → MOS Glance (mos-glance)
status: New → Confirmed
Changed in mos:
assignee: MOS Glance (mos-glance) → Alexander Tivelkov (ativelkov)
Changed in mos:
assignee: Alexander Tivelkov (ativelkov) → Kairat Kushaev (kkushaev)
Revision history for this message
Kairat Kushaev (kkushaev) wrote :
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/glance (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Kairat Kushaev <email address hidden>
Review: https://review.fuel-infra.org/28148

Changed in mos:
status: Confirmed → In Progress
Revision history for this message
Kairat Kushaev (kkushaev) wrote :

I pushed the patch to upstream also and I will try to backport this as soon as it will have been merged

Revision history for this message
Kairat Kushaev (kkushaev) wrote :
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/glance (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/28148
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: e4477e5c20d6a4e7ed88ae853c1523b4dae75cea
Author: Kairat Kushaev <email address hidden>
Date: Mon Nov 14 11:32:30 2016

Do not use service catalog for cache client

Previously cache client used endpoints from service catalog to
request cache items from node. It works perfectly on devstack
but it doesn't work when OS clous is deployed with HA
(3/5/7 controllers). We need to use -H option instead. This
option by default interacts with localhost endpoint which is
applicable in most cases because cache management client is
dependant on glance code and usually placed on controller.

Closes-Bug: #1636755

Change-Id: Icb5676fe6b36f7df55cfbb5257c943fa456c9154

Changed in mos:
status: In Progress → Fix Committed
tags: added: on-verification
Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Verified on 9.2 snapshot #498

tags: removed: on-verification
Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/glance (mcp/newton)

Fix proposed to branch: mcp/newton
Change author: Kairat Kushaev <email address hidden>
Review: https://review.fuel-infra.org/33418

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/glance (11.0/ocata)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: 11.0/ocata
Review: https://review.fuel-infra.org/33804
Reason: The patch has been merged into upstream, so no need to have one in downstream.

    https://review.openstack.org/#/c/393229/

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/glance (mcp/ocata)

Fix proposed to branch: mcp/ocata
Change author: Kairat Kushaev <email address hidden>
Review: https://review.fuel-infra.org/34890

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/glance (mcp/ocata)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/ocata
Review: https://review.fuel-infra.org/34890
Reason: The patch has been merged into upstream, so no need to have one in downstream.

 https://review.openstack.org/#/c/393229/

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/glance (mcp/newton)

Change abandoned by Ihor Kalnytskyi <email address hidden> on branch: mcp/newton
Review: https://review.fuel-infra.org/33418
Reason: The patch has been merged into upstream, so no need to have one in downstream.

 https://review.openstack.org/#/c/393229/

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.