You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by GitBox <gi...@apache.org> on 2020/05/11 10:45:31 UTC

[GitHub] [cloudstack-primate] rhtyd opened a new issue #332: [FEATURE] Cache API discovery list

rhtyd opened a new issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332


   **Is your feature request related to a problem? Please describe.**
   Everytime a user logs in, or opens a new tab for a logged in session the API discovery is fired that causes the UI to load slow.
   
   **Describe the solution you'd like**
   Explore ways to speedup API discovery or caching mechanism by use of local storage.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] onitake commented on issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
onitake commented on issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332#issuecomment-626640869


   Good idea.
   
   There needs to be some mechanism to ensure the API list is up to date, there are at least three scenarios where it may change:
   
   * CloudStack is upgraded
   * The user's role changes
   * The user's role's permissions change
   
   Is it possible to calculate some sort of "API hash" server-side?
   A cache timeout may also be a good idea, and a manual "clear API cache" action.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] rhtyd commented on issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332#issuecomment-633114254


   @onitake please check/test latest master and give your feedback, I've tried to implement client side caching for:
   - Load Primate early and let API discovery add dynamic routes without completely blocking the view
   - After login, the APIs are cached in local storage in case Primate is refreshed or a link is opened in another tab


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] rhtyd closed issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
rhtyd closed issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] onitake commented on issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
onitake commented on issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332#issuecomment-633652798


   I have to admit, I didn't see a big difference.
   
   My test was as follows:
   
   1. Login
   2. Wait for discovery completion
   3. Access list of VMs
   4. Close browser tab
   5. Open primate in a new browser tab
   6. Observe message "Login detected, using cached APIs" in the console
   
   I assume that the old behaviour was that a new API discovery task would be launched in the new tab, correct? In this case, this is definitely an improvement.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] rhtyd commented on issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
rhtyd commented on issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332#issuecomment-633245471


   Yes @onitake API cache stored in browser local storage will reset on logout, login and session expiry. 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [cloudstack-primate] onitake commented on issue #332: [FEATURE] Cache API discovery list

Posted by GitBox <gi...@apache.org>.
onitake commented on issue #332:
URL: https://github.com/apache/cloudstack-primate/issues/332#issuecomment-633242978


   Sounds good!
   Does that mean that the cache is cleared when you log out?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org