Solum

Improvement: solum-api should support YAML (e.g for Plan resource)

Bug #1324931 reported by Pierre Padrixe (stannie)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Solum
Fix Released
Medium
Pierre Padrixe (stannie)
Solum juno-1 "j1"

Bug Description

solum-api should support YAML format. We currently use JSON format for the Plan resource just because the wsme cannot support JSON. We should migrate the Plan resource to Pecan to be able to use YAML.

Pierre Padrixe (stannie) (pierre-padrixe)
Changed in solum:
assignee: nobody → Pierre Padrixe (stannie) (pierre-padrixe)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to solum (master)

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

Revision history for this message
Adrian Otto (aotto) wrote :

Pierre,

The API resource will be consumed primarily by tools, not humans. Adding YAML support to resources may be interesting, but requires a lot of fooling around and bypassing of WSME that I would rather we not bother with. I would prefer to focus your talents on something that will make a more meaningful impact on our users.

Please do not take offense to my resistance, as I don't think this is a bad idea. I simply value your contributions, and want your help with other work items. If you would still like to revisit this work, let's have an IRC chat to be sure you have explored other options for contribution first.

Thanks,

Adrian

Changed in solum:
status: In Progress → Won't Fix
Revision history for this message
Angus Salkeld (asalkeld) wrote :

Adrian, I think this is necessary work for https://bugs.launchpad.net/solum/+bug/1291114 to be complete (I don't really care what bug it is against) but it's important that we separate out the api versioning and the file versioning. Pierre's patch does that, which I think is neat. We could using this as a basis for the new api.

Adrian Otto (aotto)
Changed in solum:
status: Won't Fix → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to solum (master)

Reviewed: https://review.openstack.org/96804
Committed: https://git.openstack.org/cgit/stackforge/solum/commit/?id=326d872ded099c7aea86b61882b7d1b40672a813
Submitter: Jenkins
Branch: master

commit 326d872ded099c7aea86b61882b7d1b40672a813
Author: Pierre Padrixe <email address hidden>
Date: Thu Jun 5 17:47:36 2014 +0200

    Add YAML format support for Plan resource by using Pecan

    Closes-Bug: #1324931
    Change-Id: If060bb39761831c2790c7d847a0fd02de5ae2d25

Changed in solum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in solum:
milestone: none → juno-1
importance: Undecided → Medium
Adrian Otto (aotto)
Changed in solum:
status: Fix Committed → Fix Released
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.