Skip to content

daemon: added timeout to learning functionality#66

Closed
aanm wants to merge 1 commit intomasterfrom
adding-a-timer-to-learning-feature
Closed

daemon: added timeout to learning functionality#66
aanm wants to merge 1 commit intomasterfrom
adding-a-timer-to-learning-feature

Conversation

@aanm
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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>
@aanm aanm force-pushed the adding-a-timer-to-learning-feature branch from 26366a9 to 9e91397 Compare December 6, 2016 17:57
@tgraf tgraf added wontfix and removed kind/enhancement This would improve or streamline existing functionality. postponed labels Mar 6, 2017
@tgraf tgraf closed this Mar 6, 2017
@aanm aanm deleted the adding-a-timer-to-learning-feature branch March 15, 2017 02:58
yoursanonymous pushed a commit to yoursanonymous/cilium that referenced this pull request Jan 31, 2026
fix: update hands-on section link
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.

2 participants