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

fixes pod monitoring time calculation #112

Merged
merged 2 commits into from
Jul 16, 2024
Merged

Conversation

tsebastiani
Copy link
Collaborator

@tsebastiani tsebastiani commented Jul 15, 2024

the recovery time with very fast rescheduling pod (like prometheus) was not calculated correctly ( closes krkn-chaos/krkn#670 )

…ometheus

Signed-off-by: Tullio Sebastiani <tsebasti@redhat.com>
@tsebastiani tsebastiani changed the title fixes krkn-chaos/krkn#670 fixes pod monitoring time calculation Jul 15, 2024
@tsebastiani tsebastiani requested a review from chaitanyaenr July 15, 2024 16:15
Signed-off-by: Tullio Sebastiani <tsebasti@redhat.com>

lint

Signed-off-by: Tullio Sebastiani <tsebasti@redhat.com>
@tsebastiani tsebastiani force-pushed the monitoring_time_bug branch from fb27459 to e676cf0 Compare July 15, 2024 17:06
@chaitanyaenr chaitanyaenr merged commit d51918e into main Jul 16, 2024
5 checks passed
@tsebastiani tsebastiani deleted the monitoring_time_bug branch July 16, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scenario should fail if the actual recovery time exceeds the threshold set
2 participants