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

[core] set watchdog duration in seconds #2320

Merged
merged 1 commit into from
Mar 24, 2016
Merged

[core] set watchdog duration in seconds #2320

merged 1 commit into from
Mar 24, 2016

Conversation

yannmh
Copy link
Member

@yannmh yannmh commented Mar 4, 2016

Watchdog constructor takes duration parameter in seconds, not
milliseconds. Initialize the forwarder accordingly.

@yannmh
[core] set watchdog duration in seconds
4c74fb2 
Watchdog constructor takes `duration` parameter in seconds, not
milliseconds. Initialize the forwarder accordingly.
@yannmh yannmh self-assigned this Mar 4, 2016
@yannmh yannmh added this to the 5.7.1 milestone Mar 4, 2016
@yannmh yannmh removed their assignment Mar 4, 2016
@olivielpeau olivielpeau modified the milestones: 5.7.2, 5.7.1 Mar 10, 2016
@olivielpeau olivielpeau modified the milestones: 5.7.3, 5.7.2 Mar 21, 2016
@olivielpeau
Copy link
Member

👍 Thanks for fixing this!

CI failures are unrelated, feel free to merge

@olivielpeau olivielpeau self-assigned this Mar 23, 2016
@yannmh yannmh merged commit 6bb65cc into master Mar 24, 2016
@yannmh yannmh deleted the yann/forwarder-watchdog-sigalarm branch March 24, 2016 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@olivielpeau olivielpeau

Labels
bugfix core
Projects
None yet
Milestone
5.7.3
Development

Successfully merging this pull request may close these issues.
2 participants
@yannmh @olivielpeau