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.

Sign up for GitHub

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

Jump to bottom

WIP: Execute initial checks immediately #5089

Closed
wants to merge 1 commit into from
Closed

WIP: Execute initial checks immediately #5089

wants to merge 1 commit into from

Conversation

tobiasvdk
Copy link
Contributor

refs #5083

@tobiasvdk tobiasvdk force-pushed the fix/execute-initial-checks-immediately-5083 branch from 687153b to b577d5c Compare March 24, 2017 11:01
@dnsmichi dnsmichi added area/checks Check execution and results needs feedback We'll only proceed once we hear from you again labels May 9, 2017
@dnsmichi
Copy link
Contributor

dnsmichi commented May 9, 2017

Test that patch in various scenarios. Small low end hardware with a couple of checks, and large scale environments with 10k services. We should not run again into the scenario where a reload and newly added objects generate load spikes and cause troubles.

@dnsmichi dnsmichi changed the title Execute initial checks immediately WIP: Execute initial checks immediately Jun 16, 2017
@dnsmichi dnsmichi added wontfix Deprecated, not supported or not worth any effort and removed needs feedback We'll only proceed once we hear from you again labels Aug 14, 2017
@dnsmichi
Copy link
Contributor

Thanks for your tests. I'll reject this change, sorry.

@dnsmichi dnsmichi closed this Aug 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/checks Check execution and results wontfix Deprecated, not supported or not worth any effort
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tobiasvdk @dnsmichi @Stefar77