OpenContrail

vnc_api uses json which is slower than simplejson

Bug #1463436 reported by Sylvain Afchain
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R1.1
In Progress
Undecided
Sylvain Afchain
R2.20
Fix Committed
Medium
Sylvain Afchain
R2.21.x
Fix Committed
Medium
Sylvain Afchain
Juniper Openstack r2.21.2
Trunk
Fix Committed
Medium
Sylvain Afchain
OpenContrail
New
Medium
Sylvain Afchain

Bug Description

Currently the vnc_api uses json as json library, however simplejson library is faster than json. Doing benchmarks on a deployment using simplejson instead of json we saw some improvements.

See original description
Tags: api config
Sylvain Afchain (sylvain-afchain)
Changed in opencontrail:
assignee: nobody → Sylvain Afchain (sylvain-afchain)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11446
Submitter: Sylvain Afchain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/11448
Submitter: Sylvain Afchain (<email address hidden>)

Nagabhushana R (bhushana)
Changed in opencontrail:
importance: Undecided → Medium
tags: added: api config
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/11446
Submitter: Sylvain Afchain (<email address hidden>)

Sylvain Afchain (sylvain-afchain)
description: updated
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R1.10

Review in progress for https://review.opencontrail.org/11922
Submitter: Sylvain Afchain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/11923
Submitter: Sylvain Afchain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11446
Committed: http://github.org/Juniper/contrail-controller/commit/9cb0488f9304d647a75a4457177cb1ec5465954d
Submitter: Zuul
Branch: master

commit 9cb0488f9304d647a75a4457177cb1ec5465954d
Author: Sylvain Afchain <email address hidden>
Date: Wed Jun 10 08:00:52 2015 +0000

Add a jsonutils file in order to manager imports

This new file is a wrapper around json libraries,
using by default simplejson when available.

Change-Id: I37f46558ea065c73ae09f34b293c15f9d5dcf9c3
Closes-Bug: #1463436

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/11448
Submitter: Sylvain Afchain (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/11448
Committed: http://github.org/Juniper/contrail-generateDS/commit/193f90385143566a49516cccdbeaeb1816543176
Submitter: Zuul
Branch: master

commit 193f90385143566a49516cccdbeaeb1816543176
Author: Sylvain Afchain <email address hidden>
Date: Wed Jun 10 09:05:41 2015 +0000

Add a jsonutils file in order to manage imports

This patch depends on this review
https://review.opencontrail.org/#/c/11446/

Change-Id: Icaf198993ff36bb15a103f2da9d6db14acdf4bf2
Closes-Bug: #1463436

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.22-dev

Review in progress for https://review.opencontrail.org/12872
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/12872
Committed: http://github.org/Juniper/contrail-controller/commit/aaabf4a65621ee57aef32a895ad72b28068466a9
Submitter: Zuul
Branch: R2.22-dev

commit aaabf4a65621ee57aef32a895ad72b28068466a9
Author: Sylvain Afchain <email address hidden>
Date: Wed Jun 10 08:00:52 2015 +0000

Add a jsonutils file in order to manager imports

This new file is a wrapper around json libraries,
using by default simplejson when available.

Change-Id: I37f46558ea065c73ae09f34b293c15f9d5dcf9c3
Closes-Bug: #1463436
(cherry picked from commit 9cb0488f9304d647a75a4457177cb1ec5465954d)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/13475
Submitter: Jean-Philippe Braun (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/13586
Submitter: ?douard Thuleau (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/13475
Committed: http://github.org/Juniper/contrail-controller/commit/4489c35b7c66fec7e75a0c4f432eeca9b8f72e11
Submitter: Zuul
Branch: R2.20

commit 4489c35b7c66fec7e75a0c4f432eeca9b8f72e11
Author: Sylvain Afchain <email address hidden>
Date: Wed Jun 10 08:00:52 2015 +0000

Add a jsonutils file in order to manager imports

This new file is a wrapper around json libraries,
using by default simplejson when available.

Change-Id: I37f46558ea065c73ae09f34b293c15f9d5dcf9c3
Closes-Bug: #1463436
(cherry picked from commit 9cb0488f9304d647a75a4457177cb1ec5465954d)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

Review in progress for https://review.opencontrail.org/18764
Submitter: Jean-Philippe Braun (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/18764
Committed: http://github.org/Juniper/contrail-controller/commit/02715829c72ece3a5cec2c1362e52050bf368f23
Submitter: Zuul
Branch: R2.21.x

commit 02715829c72ece3a5cec2c1362e52050bf368f23
Author: Sylvain Afchain <email address hidden>
Date: Wed Jun 10 08:00:52 2015 +0000

Add a jsonutils file in order to manager imports

This new file is a wrapper around json libraries,
using by default simplejson when available.

Change-Id: I37f46558ea065c73ae09f34b293c15f9d5dcf9c3
Closes-Bug: #1463436
(cherry picked from commit 9cb0488f9304d647a75a4457177cb1ec5465954d)
(cherry picked from commit 4489c35b7c66fec7e75a0c4f432eeca9b8f72e11)

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.