Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

EOS-23898 : motr-freespace-monitor: Add a delay between retry if capacity_info is 0 #869

Merged
merged 1 commit into from
Aug 24, 2021

Conversation

yeshpal-jain-seagate
Copy link
Contributor

@yeshpal-jain-seagate yeshpal-jain-seagate commented Jul 20, 2021

EOS-23898 : motr-freespace-monitor: Add a delay between retry if capacity_info

Problem : Hare may take sometime to retrieve capcacity info during
startup, if the capacity_info is not recieved, free space monitor would
timeout with error.
Solution: Add delay between retry, After 10 retries, if it still fails
than return failure.

Signed-off-by: Yeshpal Jain yeshpal.jain@seagate.com

@stale
Copy link

stale bot commented Jul 27, 2021

This issue/pull request has been marked as needs attention as it has been left pending without new activity for 4 days. Tagging @huanghua78 @atulsdeshmukh2312 @mehjoshi for appropriate assignment. Sorry for the delay & Thank you for contributing to CORTX. We will get back to you as soon as possible.

…failure

Problem : Hare may take sometime to retrieve capcacity info during
startup, if the capacity_info is not recieved, free space monitor would
timeout with error.
Solution: Add delay between retry, After 10 retries, if it still fails
than return failure.

Signed-off-by: Yeshpal Jain <yeshpal.jain@seagate.com>
@mehjoshi
Copy link

@yeshpal-jain-seagate , Is there a jira ticket for this? If not could you please create one so that we do not lose context of why this change was made?
Thanks,

@yeshpal-jain-seagate yeshpal-jain-seagate changed the title motr-freespace-monitor: Add a delay between retry if capacity_info is bug_fix : motr-freespace-monitor: Add a delay between retry if capacity_info is 0 Aug 24, 2021
@yeshpal-jain-seagate yeshpal-jain-seagate changed the title bug_fix : motr-freespace-monitor: Add a delay between retry if capacity_info is 0 EOS-23898 : motr-freespace-monitor: Add a delay between retry if capacity_info is 0 Aug 24, 2021
Copy link

@mehjoshi mehjoshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants