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

[BFN] Added support for tofino3 profiles naming #1

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

taras-keryk
Copy link
Owner

Signed-off-by: Taras Keryk [email protected]

What I did

Added support for tofino3 P4 profiles naming.

How I did it

Changed P4 profile naming format for support new tofino3 profiles. Added backward compatibility support for previous profile naming format.

How to verify it

The next commands should correctly work with the old and new profile naming format

show platform barefoot profile

and

config platform barefoot profile

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch 10 times, most recently from 878707c to b97a299 Compare May 12, 2022 08:25
@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch from dfb5e81 to d40555c Compare July 22, 2022 08:17
@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch 4 times, most recently from 713a9ca to 8fa6adc Compare November 7, 2022 15:22
@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch 3 times, most recently from 5dccfe8 to 14da5ab Compare December 13, 2022 09:50
@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch 6 times, most recently from 1271899 to 12546fb Compare January 6, 2023 13:33
@taras-keryk taras-keryk force-pushed the tk_tof3_prof_nam_bf branch 3 times, most recently from 9b3189e to b1fc78f Compare January 10, 2023 11:34
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