-
Notifications
You must be signed in to change notification settings - Fork 34
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
Feat/cluster status #605
Feat/cluster status #605
Conversation
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.
I left some comments, and questions on the PR, otherwise good job.
One more thing I want to mention is that the 40s might be a bit long IMO. I would prefer to have it every ~10s, but let's see what others will say.
made the time to check for the state 10 seconds
|
# Conflicts: # manifests/claudie/kustomization.yaml # manifests/testing-framework/kustomization.yaml
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.
LGTM
Closes #440
The
ErrorMessage
field in the mongo document was replaced by amap<string, Workflow>
which holds. the information of the current state for each cluster of the config.This is also used when the config is being processed by scheduler mainly to indicate where it failed (before it will be picked up by builder to store the states of each cluster)
Frontend polls contextbox every ~40 seconds to get the state of each cluster to display it more readable in the logs