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

P4 artefacts commit for 1.6.2 migration. #220

Closed
wants to merge 2 commits into from

Conversation

mchelliyil
Copy link
Contributor

The RH MVP P4 artefacts built from the P4 changes on 1.6.2 (for base functionality, of VF to VF with and without NF)

Signed-off-by: Mincy Chelliyil [email protected]

Copy link
Contributor

@saynb saynb left a comment

Choose a reason for hiding this comment

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

Can we make the changes from bfrt -> tdi as part of this change too since they are also needed for 1.6.2? The file name and the conf file changes

@saynb
Copy link
Contributor

saynb commented Aug 22, 2024

The conf file is here

"bfrt-config": "/opt/p4/rh_mvp/rh_mvp.bfrt.json",

@mchelliyil
Copy link
Contributor Author

Can we make the changes from bfrt -> tdi as part of this change too since they are also needed for 1.6.2? The file name and the conf file changes

Addressed. Please check the updated changeset.

@mchelliyil mchelliyil requested a review from saynb August 22, 2024 18:05
@mchelliyil mchelliyil closed this Aug 22, 2024
@saynb saynb reopened this Aug 22, 2024
@mchelliyil mchelliyil closed this Aug 23, 2024
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.

2 participants