Skip to content

[SAI] Add SAI_IPMC_GROUP_ATTR_LABEL and SAI_NEXT_HOP_GROUP_ATTR_LABEL.#2283

Open
ksravani-hcl wants to merge 1 commit into
opencomputeproject:masterfrom
ksravani-hcl:label_update
Open

[SAI] Add SAI_IPMC_GROUP_ATTR_LABEL and SAI_NEXT_HOP_GROUP_ATTR_LABEL.#2283
ksravani-hcl wants to merge 1 commit into
opencomputeproject:masterfrom
ksravani-hcl:label_update

Conversation

@ksravani-hcl

@ksravani-hcl ksravani-hcl commented May 5, 2026

Copy link
Copy Markdown

Description:
Introduce unique label attributes to IPMC groups, L2MC groups, and Next Hop objects:

This PR adds SAI_IPMC_GROUP_ATTR_LABEL, SAI_L2MC_GROUP_ATTR_LABEL, and SAI_NEXT_HOP_ATTR_LABEL as string-type attributes across their respective headers. These new attributes are defined with CREATE_AND_SET flags and a default empty string to enable unique identification and improved management of these resources.

what i did:
Added a LABEL attribute to IPMC groups, L2MC groups, and Next Hop objects.

why i did:
To allow unique string-based identification and easier tracking of these resources.

@ksravani-hcl ksravani-hcl marked this pull request as ready for review May 5, 2026 09:51
@kishanps

kishanps commented May 5, 2026

Copy link
Copy Markdown

@vivekmoorthy Please review

@vivekmoorthy

Copy link
Copy Markdown
Contributor

LGTM.

@divyagayathri-hcl

Copy link
Copy Markdown

@kcudnik, @prsunny, Could you please review.

@prsunny prsunny requested a review from tjchadaga May 13, 2026 16:47
@tjchadaga

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread inc/sainexthop.h Outdated
Comment thread inc/saiipmcgroup.h Outdated
@kcudnik

kcudnik commented May 19, 2026

Copy link
Copy Markdown
Collaborator

you will need to refactor, rebase to master and force push

Signed-off-by: SRAVANI KANASANI <kanasanis@google.com>
@ksravani-hcl

Copy link
Copy Markdown
Author

you will need to refactor, rebase to master and force push
rebased with master, refactored and force pushed.

@ksravani-hcl

Copy link
Copy Markdown
Author

@tjchadaga, Could you please review the PR and help to merge.
Thank you!

@kcudnik

kcudnik commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ksravani-hcl

Copy link
Copy Markdown
Author

@tjchadaga, Can you please review the changes and approve the PR.
Thank you!

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