Skip to content

Conversation

@a114j0y
Copy link
Contributor

@a114j0y a114j0y commented Jul 31, 2025

No description provided.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@jeflo
Copy link

jeflo commented Aug 6, 2025

Couple of comments on the 0.6 version

  1. It would be nice to add a section describing the “rack” view from a networking POV.
  2. Page8, “Ethernet MAC” section, diagram in this section mentioned the “AIFH” term without first defining it. We should also align the terminology with later “ID Lookup” section.
  3. Page17, “ID lookup” section. In this section, the two sentences that mentioned “VRF” seems abrupt. Good to elaborate how it relates to the ID lookup and the haders.
  4. Given this is an architectural document, feature details for features that are relevant to Scale-up (e.g. PFC, CBFC, LLR etc) are probably outside the scope of this document. But it would be good to add a section that references these features in more details (e.g. SONIC HLD or WIP design docs).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

- [Future Features in SONiC](#future-features-in-sonic)

## Revisions
- **July 27, 2025:** Moved version 0.6 to github by Joy Qin
Copy link
Contributor

Choose a reason for hiding this comment

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

We are actually at 0.7 after 8/30. The most recent updater forgot to update date inside the doc.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 17, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.


* **LLR Implementation**:

* [GitHub Pull Request #1](https://github.com/rck-innovium/SAI/pull/1)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use this link: opencomputeproject/SAI#2225

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use this link: opencomputeproject/SAI#2225

Got it. Thanks!

Copy link
Collaborator

Choose a reason for hiding this comment

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

sai pull request link updated

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

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.

7 participants