Fix the VRouters cache calls

Bug #1574964 reported by Manoj
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.0
Fix Committed
Undecided
Manoj
Trunk
Fix Committed
Undecided
Manoj

Bug Description

After moving to MVC code the vrouters cache calls needs changes. The order of calls should be as follows.

Case 1: On Login:
1. Fetch from Server cache
2. Then issue forceRefresh to update.

Case 2: Navigating between Dashboard Summary
1. Fetch from local cache
2. Issue forcerefresh

When user logs out the cache should be cleared and the case 1 should be followed.

Tags: ui
Revision history for this message
Manoj (manojgn) wrote :

This further to Bug #1457814 after MVC.

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

Review in progress for https://review.opencontrail.org/19622
Submitter: Manoj (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/19623
Submitter: Manoj (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/19623
Committed: http://github.org/Juniper/contrail-web-controller/commit/1aacf17b55603f8a930b387494d6234fe7e04ebf
Submitter: Zuul
Branch: master

commit 1aacf17b55603f8a930b387494d6234fe7e04ebf
Author: Manoj Naik <email address hidden>
Date: Tue Apr 26 11:32:48 2016 +0530

Closes-Bug: #1574964 - Setting the cache timeout for vrouterListModel same as
session.timeout.

Change-Id: Id2570ebf92e28161c0043810708a21b5654f8e3e

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

Reviewed: https://review.opencontrail.org/19622
Committed: http://github.org/Juniper/contrail-web-core/commit/ab48d9c49212fe44a5e8b71c892ee02689dc3794
Submitter: Zuul
Branch: master

commit ab48d9c49212fe44a5e8b71c892ee02689dc3794
Author: Manoj Naik <email address hidden>
Date: Tue Apr 26 11:31:15 2016 +0530

Closes-Bug: #1574964 - Changes as follows
Case 1: On Login:
1. Fetch from Server cache
2. Then issue forceRefresh to update.

Case 2: Navigating between Dashboard Summary
1. Fetch from local cache
2. Issue forcerefresh

When user logs out the cache should be cleared and the case 1 should be
followed.

Change-Id: If48fd580104ce6ba66bc9741fb56195a6034fa9b

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

Review in progress for https://review.opencontrail.org/19792
Submitter: Manoj (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/19793
Submitter: Manoj (<email address hidden>)

Jeba Paulaiyan (jebap)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/19792
Committed: http://github.org/Juniper/contrail-web-core/commit/f35bfee32cfe4bfea3240ed62c99d8ae696ec3af
Submitter: Zuul
Branch: R3.0

commit f35bfee32cfe4bfea3240ed62c99d8ae696ec3af
Author: Manoj Naik <email address hidden>
Date: Tue Apr 26 11:31:15 2016 +0530

Closes-Bug: #1574964 - Changes as follows
Case 1: On Login:
1. Fetch from Server cache
2. Then issue forceRefresh to update.

Case 2: Navigating between Dashboard Summary
1. Fetch from local cache
2. Issue forcerefresh

When user logs out the cache should be cleared and the case 1 should be
followed.

Conflicts:
src/serverroot/utils/common.utils.js

Change-Id: If48fd580104ce6ba66bc9741fb56195a6034fa9b

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

Reviewed: https://review.opencontrail.org/19793
Committed: http://github.org/Juniper/contrail-web-controller/commit/a2ad8228bbca193f9610d79f954e804ad40774eb
Submitter: Zuul
Branch: R3.0

commit a2ad8228bbca193f9610d79f954e804ad40774eb
Author: Manoj Naik <email address hidden>
Date: Tue Apr 26 11:32:48 2016 +0530

Closes-Bug: #1574964 - Setting the cache timeout for vrouterListModel same as
session.timeout.

Change-Id: Id2570ebf92e28161c0043810708a21b5654f8e3e

Revision history for this message
Naga Kiran (nagakiran) wrote :

Need to update local cache after getting force refresh data from server.

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

Review in progress for https://review.opencontrail.org/21148
Submitter: Manoj (<email address hidden>)

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

Review in progress for https://review.opencontrail.org/21151
Submitter: Manoj (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/21151
Committed: http://github.org/Juniper/contrail-web-controller/commit/3fe8b5d4c18a2fd97dc5296144cd242884c8a728
Submitter: Zuul
Branch: R3.0

commit 3fe8b5d4c18a2fd97dc5296144cd242884c8a728
Author: Manoj Naik <email address hidden>
Date: Tue Jun 14 16:51:13 2016 +0530

Closes-Bug: #1574964 - Whenever the forcerefresh used to fail or timeout the
original data gotten by servercache or localcache would get removed. To avoid
this added a check to update the model only if there is no error in the
forcerefresh call.
Increased the timeout of the force refresh call to 5 minutes.

Change-Id: I1bfa9701a69c55ac21ea8cce008e0d1fd1b59750

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

Reviewed: https://review.opencontrail.org/21148
Committed: http://github.org/Juniper/contrail-web-controller/commit/ede02ef8482624306686420e2232882f42bbd6cf
Submitter: Zuul
Branch: master

commit ede02ef8482624306686420e2232882f42bbd6cf
Author: Manoj Naik <email address hidden>
Date: Tue Jun 14 16:51:13 2016 +0530

Closes-Bug: #1574964 - Whenever the forcerefresh used to fail or timeout the
original data gotten by servercache or localcache would get removed. To avoid
this added a check to update the model only if there is no error in the
forcerefresh call.
Increased the timeout of the force refresh call to 5 minutes.

Change-Id: I1bfa9701a69c55ac21ea8cce008e0d1fd1b59750

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.