-
Notifications
You must be signed in to change notification settings - Fork 27
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
nodeProgress
in frontend
#3786
nodeProgress
in frontend
#3786
Conversation
…c-simcore into feature/progress-messages
progressStatus
in frontendnodeProgress
in frontend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Code Climate has analyzed commit fdbb4f8 and detected 0 issues on this pull request. View more on Code Climate. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
You might consider not showing the old messages (the ones that shows up directly below the service thumbnail) or showing a generic one e.g. "Loading you project/study".
I think they are not old, just a kind of a subtask progress. @sanderegg ? |
So I am trying to differentiate Of course this is a work in progress and we can discuss whether we should at some point replace one with another. |
What do these changes do?
This PR brings the frontend for #3773.
The node start-up sequence is as follows:
1 - CLUSTER_UP_SCALING
2 - SIDECAR_PULLING
3a - SERVICE_INPUTS_PULLING
3b - SERVICE_OUTPUTS_PULLING
3c - SERVICE_STATE_PULLING
4 - SERVICE_IMAGES_PULLING
All these steps have a progress bar that is shown when the frontend gets some step progress. Regarding the up-scaling, since there is no progress, when we are in this step a message saying
This step can take up to 3 minutes
is shown.In the node loading page, below the node state, this is shown: (due to several issues, the messages for the animation were faked)
Related issue/s
Show progress. Related to ITISFoundation/osparc-issues#765
closes #3788
How to test
Checklist