Some optimization of svc chain test code can be done

Bug #1720965 reported by Vedamurthy Joshi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R4.1
Fix Committed
Undecided
Vedamurthy Joshi
Trunk
Fix Committed
Undecided
Vedamurthy Joshi

Bug Description

Some optimization of svc chain test code can be done

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

Review in progress for https://review.opencontrail.org/36176
Submitter: Vedamurthy Joshi (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/36177
Submitter: Vedamurthy Joshi (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/36177
Committed: http://github.com/Juniper/contrail-test/commit/2eabc870db579fdef0e7f0535de01db4df74c45c
Submitter: Zuul (<email address hidden>)
Branch: master

commit 2eabc870db579fdef0e7f0535de01db4df74c45c
Author: Vedu Joshi <email address hidden>
Date: Tue Oct 3 13:46:36 2017 +0530

Update test based on review 36176

Ref https://review.opencontrail.org/#/c/36176/

Partial-Bug: 1720965
Change-Id: Iec60280ffaf1dba19605f63d4038b76ababd35c9

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

Reviewed: https://review.opencontrail.org/36176
Committed: http://github.com/Juniper/contrail-test-ci/commit/3e3cfb36707afa9a84e38b2bee12726f4b1bc1f9
Submitter: Zuul (<email address hidden>)
Branch: master

commit 3e3cfb36707afa9a84e38b2bee12726f4b1bc1f9
Author: Vedu Joshi <email address hidden>
Date: Tue Oct 3 13:19:55 2017 +0530

Optimize svc chain cases a bit

* Reduce some repeated nova/contrail api calls which were taking time
* and use the info present in VMFixture
* Remove some duplicate code while getting tap intf of a svm using vrf
* In create_service_vms(), create all svms in one-shot and verify them
* together, which brings down execution time
* In config_svc_chain(), no need to verify left and right vms when they
* are passed as args

Change-Id: I5cad5dd4ea5ba10bde14aa39479b09c92b505ef3
Closes-Bug: 1720965

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

Review in progress for https://review.opencontrail.org/36711
Submitter: Vedamurthy Joshi (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/36712
Submitter: Vedamurthy Joshi (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/36711
Committed: http://github.com/Juniper/contrail-test-ci/commit/0bf4bac91303cb553cf6ca602643d2dea77990d6
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 0bf4bac91303cb553cf6ca602643d2dea77990d6
Author: Vedu Joshi <email address hidden>
Date: Tue Oct 3 13:19:55 2017 +0530

Optimize svc chain cases a bit

* Reduce some repeated nova/contrail api calls which were taking time
* and use the info present in VMFixture
* Remove some duplicate code while getting tap intf of a svm using vrf
* In create_service_vms(), create all svms in one-shot and verify them
* together, which brings down execution time
* In config_svc_chain(), no need to verify left and right vms when they
* are passed as args

Change-Id: I5cad5dd4ea5ba10bde14aa39479b09c92b505ef3
Closes-Bug: 1720965
(cherry picked from commit 3e3cfb36707afa9a84e38b2bee12726f4b1bc1f9)

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

Reviewed: https://review.opencontrail.org/36712
Committed: http://github.com/Juniper/contrail-test/commit/60bad765d997a82fb0f5c7f4c3684202e02c4565
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit 60bad765d997a82fb0f5c7f4c3684202e02c4565
Author: Vedu Joshi <email address hidden>
Date: Tue Oct 3 13:46:36 2017 +0530

Update test based on review 36176

Ref https://review.opencontrail.org/#/c/36176/

Partial-Bug: 1720965
Change-Id: Iec60280ffaf1dba19605f63d4038b76ababd35c9
(cherry picked from commit 2eabc870db579fdef0e7f0535de01db4df74c45c)

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

Review in progress for https://review.opencontrail.org/40095
Submitter: Ankit Jain (<email address hidden>)

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.