Nailgun "json" should be replaced with "jsonutils"
Bug #1292561 reported by
Nikolay Markov
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Fuel for OpenStack |
Fix Committed
|
Low
|
Alexander Kislitsky | ||
Bug Description
We explicitly use built-in JSON library anywhere in Nailgun code, despite of the fact that Oslo provides module called "jsonutils" which we already have in our project. It already uses "anyjson" module, which automatically selects best available JSON parser, and it is widely used in OpenStack. Let's just import and use it in our code, it has the same interface.
| tags: | added: low-hanging-fruit |
| Changed in fuel: | |
| status: | New → Confirmed |
| importance: | Undecided → Low |
| Changed in fuel: | |
| assignee: | Fuel Python Team (fuel-python) → Ivan Kolodyazhny (e0ne) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix proposed to fuel-web (master) | #1 |
| Changed in fuel: | |
| status: | Confirmed → In Progress |
| Changed in fuel: | |
| milestone: | 5.0 → 5.1 |
| Changed in fuel: | |
| assignee: | Ivan Kolodyazhny (e0ne) → Alexander Kislitsky (akislitsky) |
Revision history for this message
| OpenStack Infra (hudson-openstack) wrote Fix merged to fuel-web (master) | #2 |
| Changed in fuel: | |
| status: | In Progress → Fix Committed |
To post a comment you must log in.
Fix proposed to branch: master
Review: https:/