Can't update the pod, service and rc with a manifest change

Bug #1444383 reported by Lan Qi song
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Invalid
Undecided
Lan Qi song

Bug Description

Currently, we can't update a pod with a manifest change,

When I trid to use the following command to update a pod, it would be failed:

curl -i -X PATCH -H 'X-Auth-Token: 6819fe161fa047afa925dbe5f5db7cc4' -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'User-Agent: python-magnumclient' -d '[{"path": "/manifest", "value:"manifest value ", "op": "replace"}]' http://9.123.137.192:9511/v1/pods/fded8118-2ba3-4296-9418-4b90ea8f587e

Error message: Reason: can't replace non-existent object 'manifest'\"}

Update the pod manifest will be useful when we need to update the label or resource limit of a pod.

This issue also affect the manifest update of service and rc.

Lan Qi song (lqslan)
Changed in magnum:
assignee: nobody → Lan Qi song (lqslan)
Changed in magnum:
status: New → In Progress
Lan Qi song (lqslan)
summary: - Can't update the pod with a manifest change
+ Can't update the pod, service and rc with a manifest change
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

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

commit 6ed3ccb9df4365f6ea62c55e3f872afff0aad8eb
Author: Lan Qi song <email address hidden>
Date: Wed Apr 15 16:25:59 2015 +0800

    Update pod support a manifest change

    Currently, we don't support a manifest change when we update a pod.

    This patch fix this problem so that we can update the label or other
    useful attributes of a pod.

    Partial-Bug: #1444383

    Change-Id: I87e220e88bc32d7eee9d37c552aea9920a027056

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/174208

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :

both https://review.openstack.org/#/c/174209/ and https://review.openstack.org/#/c/174208/ have merged. Do we know what else needs to be done?

Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
Changed in magnum:
status: In Progress → Invalid
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.