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

Add 16-bit Sink Indication field to INT header #57

Closed
wants to merge 1 commit into from

Conversation

@mickeyspiegel
Copy link
Contributor

commented Oct 18, 2018

A new 16-bit Sink Indication field is proposed to be added to the INT header, placed in the previous 16-bit Reserved field following the Instruction Bitmap in the INT header. This field allows the source and transit switches to notify the sink switch about the collected network state in a concise manner.

While INT allows network state to be collected on a per packet basis, for many packets the network state may not differ significantly from the state collected in previous packets belonging to the same flow. The sink may choose to report only a subset of packets, or it may choose to aggregate the collected network state.

The Sink Indication lets the sink switch look at only one field when deciding how to process the telemetry information from the packet, without having to scrape over each field added at each hop, just to decide whether and how to process the collected network state.

The Sink Indication is initially set by the source switch, and is updated by each transit switch, taking into account the received value as well as the local state added by that transit switch.

@mickeyspiegel mickeyspiegel requested review from mhira1 and jklr Oct 18, 2018

@rsivakolundu

This comment has been minimized.

Copy link
Contributor

commented Dec 3, 2018

Are we changing this in Ver=2 or Ver=1. Also, wasn't this field to be called Domain Specific ID. I am confused at this stage.

@jklr

This comment has been minimized.

Copy link
Contributor

commented Dec 4, 2018

We will do it in Ver2, by using domain specific extension. I think we can just close this PR. Mickey can decide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.