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

adapt document to priority not changing through the DODAG #9

Closed
wants to merge 2 commits into from

Conversation

mcr
Copy link
Contributor

@mcr mcr commented Nov 22, 2021

No description provided.

Copy link
Collaborator

@nyrahul nyrahul left a comment

Choose a reason for hiding this comment

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

Please find my review inline. Thanks

@@ -1,7 +1,7 @@
---
title: Controlling Secure Network Enrollment in RPL networks
abbrev: join-metric
docname: draft-ietf-roll-enrollment-priority-07
docname: draft-ietf-roll-enrollment-priority-06
Copy link
Collaborator

@nyrahul nyrahul Nov 23, 2021

Choose a reason for hiding this comment

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

the new draft version lower than previous?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think that I goofed incrementing the number too many times. -05 is what is posted.


As explained in {{!RFC9032}}, higher values decrease the likelyhood of an unenrolled node sending enrollment traffic via this path.

A network operator can set this value to the maximum value allowed, effectively disable all new enrollment traffic.

Updates to this option propogate slowly through the network according to the trickle algorithm that periodically sends updates.
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we assume that we never reset the DIO trickle timer, the option propagation in worst case may take hours or even days to propagate depending on DIO trickle timer parameters.

However, the term *Join Proxy* is retained with it's meaning from {{!RFC9031}}.


# Protocol Definition

With this specification, the following option is defined for transmission in the DIO issued by the DODAG root and it MUST be propagated down the DODAG.

A 6LR which would otherwise be willing to act as a *Join Proxy*, will examine the minimum priority field, and to that number, add any additional local consideration (such as upstream congestion).
A 6LR which would otherwise be willing to act as a *Join Proxy*, will examine the minimum priority field, and to that number, add any additional local consideration (such as upstream congestion, number of NCE slots available, etc.)

The Enrollment Priority can only be increased by each 6LR in value, to the maximum value of 0x7f.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we allowing the 6LRs to change the Enrollment priority? Based on @mcr mail, it was not allowed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We aren't changing the global enrollment priority, but in translating this RPL metric to the one in RFC9032, the local node is allowed to take into consideration local conditions. Perhaps this isn't worded well.

@pthubert
Copy link
Collaborator

pthubert commented Nov 23, 2021 via email

@mcr
Copy link
Contributor Author

mcr commented Nov 24, 2021

We need to provide the math or at least indicate the the OF’ that computes the priority in the beacon from the min I. The DIO MUST be homogeneous across DODAGs of the RPL instance… A bientôt; Pascal

Are you saying that you want two nodes, from two different vendors, to emit the same Enhanced Beacon priority, given the same input from the min I?
(I'm not saying this is bad, I'm wondering if we know enough to write this down at this point)

@pthubert
Copy link
Collaborator

pthubert commented Nov 24, 2021 via email

@mcr
Copy link
Contributor Author

mcr commented May 24, 2023

already merged.

@mcr mcr closed this May 24, 2023
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

3 participants