[RFE] Replace deprecated wsme library with alternative

Bug #1651346 reported by Madhuri Kumari
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Madhuri Kumari

Bug Description

Currently ironic uses wsme to validate input parameters which is no longer maintained[1][2]

[1] http://osdir.com/ml/openstack-dev/2016-01/msg00987.html
[2] https://openstack.nimeyo.com/76822/openstack-dev-all-reminder-wsme-being-actively-maintained

One option is to use jsonschema (https://pypi.python.org/pypi/jsonschema). A jsonschema validator object can be used to check each resource against an appropriate schema for that resource. If the validation passes, the request can follow the existing flow of control to the resource manager to the backend. If the request body parameters fail the validation specified by the resource schema, a validation error will be returned from the server.

Tags: needs-spec rfe
Changed in ironic:
assignee: nobody → Madhuri Kumari (madhuri-rai07)
Revision history for this message
Jay Faulkner (jason-oldos) wrote :

I've marked this as an RFE; I agree that in theory it could be a bug but IMO it needs enough design that we should treat it like an RFE and have a spec written. Can you please lay out your proposed changes in a spec so that the cores can agree on the solution? This would be a major change in our API code :).

summary: - wsme is deprecated
+ [RFE] Replace deprecated wsme library with alternative
tags: added: needs-spec rfe
Changed in ironic:
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Madhuri Kumari (madhuri-rai07) wrote :

@Jay, thanks for confirming this bug. I will submit a spec soon.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-specs (master)

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

Changed in ironic:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic-specs (master)

Change abandoned by Dmitry Tantsur (<email address hidden>) on branch: master
Review: https://review.opendev.org/415894
Reason: This effort has been already approved and is currently under way. There is no need for a spec at this point (but thank you for writing it!).

Dmitry Tantsur (divius)
Changed in ironic:
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.