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

repeat claim up to 3 times #7

Closed
christf opened this issue May 4, 2017 · 5 comments
Closed

repeat claim up to 3 times #7

christf opened this issue May 4, 2017 · 5 comments
Milestone

Comments

@christf
Copy link
Member

christf commented May 4, 2017

see: tcatm/l3roamd#14

In case of no response a claim should be tried up to three times, each after a short delay of about 1s if no response is received. Use the relinquished flag to determine when to stop.

@christf christf added this to the alpha milestone Sep 7, 2017
@christf
Copy link
Member Author

christf commented Sep 23, 2017

if there is no response, the client may be new.

@christf
Copy link
Member Author

christf commented Nov 25, 2017

@t-8ch what do you think, is this worth re-introducing a taskqueue and a scheduler?
I ripped that one out previously to make l3roamd netlink-event-driven but it seems we still want to be able to schedule actions in the future....
ideas?

@t-8ch
Copy link
Contributor

t-8ch commented Nov 27, 2017

I don't see a way of having for example the kernel do the scheduling for us directly.
(I hoped we could use a reliable transport protocol, but that does not work in the multicast case).
Maybe we can reintroduce the previous scheduler?

@christf
Copy link
Member Author

christf commented Nov 28, 2017

it is back now. Now it needs to be implemented after considering data structures. We do not want claims to happen 3 times, we want claims to happen up to 3 times. Since they are answered with an info packet, we can stop sending claims once we received an info packet for a claim.
At the same time, if a client is new, no info packet may be sent/received at all, so we need to stop repeating after a while.

@christf
Copy link
Member Author

christf commented Dec 14, 2017

this is addressed by #30
Closing this issue as one ticket is enough to track this.

@christf christf closed this as completed Dec 14, 2017
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

No branches or pull requests

2 participants