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

[sonic-ztp] Add dynamic port breakout(DPB) to ZTP temolate #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dennis0113
Copy link

- Why I did it
To support DPB with the ZTP template.

-How I did it
Generate 'BREAKOUT_CFG' when dynamic port breakout(DPB) is supported on the platform.
Revise the ZTP template so that the DPB CLI can be executed when config is generated from the ZTP template.

-How to verify it
Execute DPB cli when the config is generated from ZTP template.

port breakout(DPB) is supported on the platform. Revise the ZTP template
so that the DPB CLI can be excuted when config is generated from the ZTP
template.
@rajendra-dendukuri
Copy link
Collaborator

@dennis0113 Can you please confirm that the BREAKOUT_CFG included will be compatible with the PORT table defined in the same template? Also it is to be noted that you are just trying to use the default port breakout mode to light up an in-band port link.

@rajendra-dendukuri
Copy link
Collaborator

@dennis0113 Can you please confirm that the BREAKOUT_CFG included will be compatible with the PORT table defined in the same template? Also it is to be noted that you are just trying to use the default port breakout mode to light up an in-band port link.

@dennis0113 Can you please respond to my question.

@dennis0113
Copy link
Author

Sorry for the late response.

I consider that BREAKOUT_CFG can be compatible with the PORT table.
Because PORT table will be generated from "platform.json" rather than "port_config.ini" if the "platform.json" exists.

And I am not trying to use the default port breakout mode to light up an in-band port link.
I just want the SONiC Cli for the breakout command can be executed even if the config is generated from ztp template.

Thanks

@liushilongbuaa
Copy link
Contributor

/easycla

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 8, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

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.

4 participants