[proxy] - USers should be able to update partition

Bug #1230841 reported by Malini Kamalambal on 2013-09-25
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
High
Allele Dev

Bug Description

The proxy API currently allows only create, read, list & delete partitions.
The API needs to include an UPDATE partition as well
Users should be able to update partitions, i.e. add /remove hosts, change weights etc.

summary: - USers should be able to update partition
+ [PROXY] - USers should be able to update partition
summary: - [PROXY] - USers should be able to update partition
+ [proxy] - USers should be able to update partition
Allele Dev (alleledev) on 2013-09-25
Changed in marconi:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Alejandro Cabrera (cpp-cabrera)

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

Changed in marconi:
status: Triaged → In Progress

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

Reviewed: https://review.openstack.org/49262
Committed: http://github.com/openstack/marconi/commit/99aaa0272c1c0761606367d3afa201f4256241af
Submitter: Jenkins
Branch: master

commit 99aaa0272c1c0761606367d3afa201f4256241af
Author: Alejandro Cabrera <email address hidden>
Date: Fri Sep 27 15:54:05 2013 -0400

    feat (proxy/admin): allow partition modifications

    This patchset adds one new method to the partitions API
    - PATCH /v1/partitions/{partition}

    The partitions storage layer is also updated to support a simpler
    update interface:
    - update(self, name, **kwargs) [with kwargs in ('hosts', 'weight')]

    The partitions transport layer is cleared of cruft and reuses JSON
    handling utilities from the queues transport layer.

    This patch implementation uses jsonschema to achieve self-documenting,
    strict format validation of the incoming input.

    The partition storage controllers are modified accordingly, as are the
    tests. More tests are added for the partitions transport layer to
    verify the correctness and to ease future refactorings.

    Change-Id: Ifa92b1225421196f95131c2b74e3c07b07c4cfd4
    Implements: blueprint placement-service
    Closes-Bug: 1230841

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-12-19
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers