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

[dash-p4] Update P4 to support PL redirect map #663

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jimmyzhai
Copy link
Collaborator

@jimmyzhai jimmyzhai commented Jan 24, 2025

PL redirect map implementation.

@mssonicbld
Copy link

/azp run

Copy link

Commenter does not have sufficient privileges for PR 663 in repo sonic-net/DASH

@jimmyzhai jimmyzhai requested a review from r12f January 24, 2025 07:52
@r12f
Copy link
Collaborator

r12f commented Jan 24, 2025

Hi Junhua, do you mind to help create another PR for the doc update? It will be great to have this feature explained in the community before the code merge.

@r12f
Copy link
Collaborator

r12f commented Jan 24, 2025

Hi Junhua, looks like CI failed for this PR, we will need to get it fixed.

@jimmyzhai
Copy link
Collaborator Author

jimmyzhai commented Feb 5, 2025

Hi Junhua, do you mind to help create another PR for the doc update? It will be great to have this feature explained in the community before the code merge.

PR #664

@mssonicbld
Copy link

/azp run

Copy link

Commenter does not have sufficient privileges for PR 663 in repo sonic-net/DASH

@jimmyzhai
Copy link
Collaborator Author

/azp run

Copy link

No pipelines are associated with this pull request.

@jimmyzhai
Copy link
Collaborator Author

/azp run

Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link

/azp run

Copy link

Commenter does not have sufficient privileges for PR 663 in repo sonic-net/DASH

@mssonicbld
Copy link

/azp run

Copy link

Commenter does not have sufficient privileges for PR 663 in repo sonic-net/DASH

@jimmyzhai
Copy link
Collaborator Author

Hi Junhua, looks like CI failed for this PR, we will need to get it fixed.

Fixed

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.

3 participants