Magnum

common templates path should be const var

Bug #1616259 reported by yuanying
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Low
yuanying

Bug Description

See https://review.openstack.org/#/c/337965/11/magnum/drivers/k8s_fedora_atomic_v1/template_def.py

../../common/templates/environments/ path seems to be redundant in each if else.
So it should be const variable.

yuanying (yuanying)
Changed in magnum:
assignee: nobody → yuanying (ootsuka)
importance: Undecided → Low
yuanying (yuanying)
Changed in magnum:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

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

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

Reviewed: https://review.openstack.org/359497
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=b4b32267c6e20ddeecc848770384989d95e66b3f
Submitter: Jenkins
Branch: master

commit b4b32267c6e20ddeecc848770384989d95e66b3f
Author: OTSUKA, Yuanying <email address hidden>
Date: Wed Aug 24 10:00:40 2016 +0900

    Make templates env path be const variable

    "../../common/templates/environments/" path seems to be redundant
    in each drivers.
    So it should be const variable.

    Change-Id: I5cbd30e350c861b611b327a4dda3f92a6316e45b
    Closes-Bug: #1616259

Changed in magnum:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/magnum 3.0.0

This issue was fixed in the openstack/magnum 3.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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