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

daemon: added timeout to learning functionality #66

Closed
wants to merge 1 commit into from

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 4, 2016

Signed-off-by: André Martins andre@cilium.io

@aanm aanm added kind/enhancement This would improve or streamline existing functionality. pending-review labels Aug 4, 2016
@tgraf
Copy link
Member

tgraf commented Aug 4, 2016

I see where you want to go with this but I'm not sure this is a good idea. Applications might need to run longer than 60s. Let's defer this to later.

@aanm aanm force-pushed the adding-a-timer-to-learning-feature branch from 56765fe to 26366a9 Compare December 6, 2016 16:10
Signed-off-by: André Martins <andre@cilium.io>
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.

None yet

2 participants