Setting custom docker_registry in Ansible deploy requires a trailing slash

Bug #1479013 reported by Paul Bourke
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla
Fix Released
High
Paul Bourke

Bug Description

In /etc/kolla/globals.yml, set:

docker_registry: my-reg:5000

Nodes will try to pull:

my-reg:5000kollaglue/centos-rdo-rabbitmq:latest

Changed in kolla:
assignee: nobody → Paul Bourke (pauldbourke)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on kolla (master)

Change abandoned by Paul Bourke (<email address hidden>) on branch: master
Review: https://review.openstack.org/206593

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

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

Changed in kolla:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla (master)

Reviewed: https://review.openstack.org/207463
Committed: https://git.openstack.org/cgit/stackforge/kolla/commit/?id=2887c6d2e4d511888d9156668e2b75b48d58add8
Submitter: Jenkins
Branch: master

commit 2887c6d2e4d511888d9156668e2b75b48d58add8
Author: Vladislav Belogrudov <email address hidden>
Date: Thu Jul 30 16:49:35 2015 +0300

    Add missing slash if docker registry is specified

    If user specifies registry a full image name is constructed by
    concatenation of the registry, namespace and image. Currently
    concatenation does not include '/' if registry is non-empty but
    it should. If registry is empty '/' is not required.
    This fix covers both use cases with help of Ansible filter.

    Change-Id: I0588dd0da55d777e6caa7eb47d51b2435d38d5e0
    Closes-Bug: #1479013

Changed in kolla:
status: In Progress → Fix Committed
Changed in kolla:
milestone: none → liberty-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on kolla (master)

Change abandoned by Paul Bourke (<email address hidden>) on branch: master
Review: https://review.openstack.org/207120
Reason: Thanks Sam.

Steven Dake (sdake)
Changed in kolla:
importance: Undecided → High
Sam Yaple (s8m)
Changed in kolla:
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.