labels should use semicolon for lists

Bug #1497042 reported by Daneyon Hansen
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
mark phelan

Bug Description

Currently, labels use commas to separate value in a list. For example:

$ magnum --debug baymodel-create --name swarm --image-id fedora-21-atomic-3 --keypair-id danehans --external-network-id public --dns-nameserver 172.29.74.154 --flavor-id m1.small --docker-volume-size 3 --coe swarm --labels k1=v1,v2,v3

Using commas to separate multiple values and multiple key/value pairs is difficult on the human eye to read.

labels should be updated to use ; instead of , to separate multiple values of a key.

Tags: tech-debt
Changed in magnum:
assignee: nobody → Daneyon Hansen (danehans)
tags: added: tech-debt
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
Revision history for this message
mark phelan (phelanm) wrote :

Daneyon Hansen, tried to contact you, have patch ready, hoping to contrib to "Global OpenStack BUG SMASH", assigning to myself.. hopefully that's ok...

mark phelan (phelanm)
Changed in magnum:
assignee: Daneyon Hansen (danehans) → mark phelan (phelanm)
Revision history for this message
mark phelan (phelanm) wrote :
Changed in magnum:
status: New → In Progress
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/python-magnumclient 2.1.0

This issue was fixed in the openstack/python-magnumclient 2.1.0 release.

Changed in magnum:
status: In Progress → 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.