Skip to content

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

Open
dennis0113 wants to merge 1 commit intosonic-net:masterfrom
dennis0113:master
Open

[sonic-ztp] Add dynamic port breakout(DPB) to ZTP temolate#22
dennis0113 wants to merge 1 commit intosonic-net:masterfrom
dennis0113:master

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