magnum services state must be restricted to only up/down

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

Bug Description

Magnum service "state" can have any string having length 1 to 254(https://github.com/openstack/magnum/blob/master/magnum/api/controllers/v1/magnum_services.py#L34). It must be restricted to only "up" or "down". Like we restrict bay status https://github.com/openstack/magnum/blob/master/magnum/api/controllers/v1/bay.py#L110

rajiv (rajiv-kumar)
Changed in magnum:
assignee: nobody → rajiv (rajiv-kumar)
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/350373

Changed in magnum:
status: New → In Progress
Adrian Otto (aotto)
Changed in magnum:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

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

commit 42ef3dc564e91fa3b0212980b7056de1697790c9
Author: Rajiv Kumar <email address hidden>
Date: Wed Aug 3 10:02:02 2016 +0530

    Restricted Magnum service state to 'up' and 'down'

    Magnum service state could have any string having length 1-255.
    This patch restricts value to be one of the following
      - up
      - down

    Change-Id: I740e0379b78c08cbd9ded7ca7bd1bbdd92d44e90
    Closes-bug: #1609216

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.

Other bug subscribers

Remote bug watches

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