search

ManageIQ / manageiq-ui-classic

Classic UI of ManageIQ
Apache License 2.0
50 stars 358 forks source link

Spinner cleanup on the login screen #6950

Open skateman opened 4 years ago

skateman commented 4 years ago

The spinner has an extra feature on the login screen to display a message if the server is just starting: Screenshot from 2020-04-06 18-54-36

Does this make any sense? As far as I understand the UI is the last piece of the server to be started and so this message would be never displayed. I am also concerned about how it looks like, it's kinda hard to read.

Do we still need it?

skateman commented 4 years ago

@miq-bot add_label question

jrafanie commented 4 years ago

Note, this code was recently moved around a little but I think it's still the same overall idea.

See here The server is marked as starting, we configure roles, we do some cleanup of prior run's rows such as workers, we start new workers and then it's marked as started.

So, it's most very likely to be in starting due to waiting for workers to start. I'm not sure how you can get a UI worker/web service worker for auth while it's still starting workers. It's possible that it's starting the others workers but the UI/web service ones are already started. So, maybe it's needed?

miq-bot commented 1 year ago

This issue has been automatically marked as stale because it has not been updated for at least 3 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.