Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vic-admin vsphere session timeout leads to weird errors on the screen #3569

Open
vburenin opened this issue Jan 5, 2017 · 7 comments
Open
Labels
area/ui The Virtual Container Host administration UI component/vicadmin priority/p4 resolution/invalid The issue is intended behavior or otherwise invalid status/need-info Additional information is needed to make progress

Comments

@vburenin
Copy link
Contributor

vburenin commented Jan 5, 2017

  1. Log into vic-admin UI. Do not use it for a while(several hours I guess).
  2. Refresh the page, you will still be logged into vic-admin UI, but lots of errors will be displayed regarding to the connection to vSphere API.

vSphere session timeout is just a guess, it might be something else. Unfortunately, all related data including logs is no longer available.

@vburenin vburenin added area/ui The Virtual Container Host administration UI component/vicadmin labels Jan 5, 2017
@vburenin vburenin added this to the v0.9.0 milestone Jan 5, 2017
@anchal-agrawal
Copy link
Contributor

anchal-agrawal commented Jan 5, 2017

@vburenin Did you see a screen like the second picture here #3106 (comment)?

This is expected behavior currently. It was implemented in #3106 towards issues #2811 and #3082. One UX improvement we can make is to prompt the user that logging out and in again could resolve the session issue.

@vburenin
Copy link
Contributor Author

vburenin commented Jan 5, 2017

@anchal-agrawal So, you are saying it is a know bug that is a less even than the other. Ok. Let's keep it open.

@anchal-agrawal
Copy link
Contributor

I came across this screen today:

vicadmin

The error message was improved in #3270 to prompt the user to re-login. If you saw the same screen, we can close this issue since this is expected behavior.

@anchal-agrawal anchal-agrawal added the status/need-info Additional information is needed to make progress label Jan 9, 2017
@gigawhitlocks
Copy link
Contributor

This is by design

@vburenin
Copy link
Contributor Author

@gigawhitlocks This design has to be improved.

@gigawhitlocks
Copy link
Contributor

@vburenin with regard to the planning we did this week, probably doesn't make much sense to spend a lot of time refining this UI.

I initially had vicadmin log the user out in this case and @anchal-agrawal was told to change it to do what it does now. The messaging about "Logging in again may resolve the issue" was added to clarify things in this situation.

I don't personally think it's worth spending the time to change this behavior, but if you do then you should bring it up with @hickeng et al and then you should be able to open an issue with a specific description of the agreed-upon behavior.

This issue is worded like this is a bug, which it is not.

@vburenin
Copy link
Contributor Author

@gigawhitlocks I am not pushing to address it right now, it should be addressed at some point eventually.This bug can just stay open until it is addressed.

@mdubya66 mdubya66 removed this from the v0.9.0 milestone Jan 19, 2017
@anchal-agrawal anchal-agrawal added the resolution/invalid The issue is intended behavior or otherwise invalid label Mar 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui The Virtual Container Host administration UI component/vicadmin priority/p4 resolution/invalid The issue is intended behavior or otherwise invalid status/need-info Additional information is needed to make progress
Projects
None yet
Development

No branches or pull requests

5 participants