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

ZTP CLI commands #599

Merged
merged 18 commits into from
Feb 12, 2020
Merged

ZTP CLI commands #599

merged 18 commits into from
Feb 12, 2020

Conversation

rajendra-dendukuri
Copy link
Contributor

Implemented following commands to use Zero Touch Provisioning

show ztp status
config ztp enable
config ztp disable
config ztp run

Signed-off-by: Rajendra Dendukuri rajendra.dendukuri@broadcom.com

- What I did
Added following CLI commands for Zero Touch Provisioning feature.

show ztp status
config ztp enable
config ztp disable
config ztp run

If sonic-ztp package is not included in the image, ztp cli commands display following message.
"ZTP feature unavailable in this image version"

- How I did it
Modified config/main.py and show/main.py

- How to verify it
Execute individual commands and check ZTP flow

- 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)

-->

rajendra-dendukuri and others added 3 commits August 6, 2019 20:45
Implemented following commands to use Zero Touch Provisioning

show ztp status
config ztp enable
config ztp disable
config ztp run

Signed-off-by: Rajendra Dendukuri <rajendra.dendukuri@broadcom.com>
@rajendra-dendukuri
Copy link
Contributor Author

please retest this

1 similar comment
@rajendra-dendukuri
Copy link
Contributor Author

please retest this

@rajendra-dendukuri
Copy link
Contributor Author

retest

skg-net
skg-net previously approved these changes Oct 15, 2019
AkhileshSamineni and others added 11 commits October 16, 2019 03:58
 Signed-off-by: akhilesh.samineni@broadcom.com
 Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
  Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
Signed-off-by: Akhilesh Samineni <akhilesh.samineni@broadcom.com>
@rajendra-dendukuri
Copy link
Contributor Author

retest please

@lguohan
Copy link
Contributor

lguohan commented Dec 13, 2019

retest this please

2 similar comments
@lguohan
Copy link
Contributor

lguohan commented Jan 9, 2020

retest this please

@lguohan
Copy link
Contributor

lguohan commented Jan 15, 2020

retest this please

show/main.py Show resolved Hide resolved
@lguohan
Copy link
Contributor

lguohan commented Jan 29, 2020

retest this please

…_sonic

Changes in swss-utilities submodule to support NAT feature.
@jleveque
Copy link
Contributor

@rajendra-dendukuri: Please also fix conflicts.

@jleveque
Copy link
Contributor

jleveque commented Feb 5, 2020

Retest this please

@lguohan lguohan merged commit a4385f0 into sonic-net:master Feb 12, 2020
rlhui added a commit that referenced this pull request Feb 15, 2020
Implemented following commands to use Zero Touch Provisioning

show ztp status
config ztp enable
config ztp disable
config ztp run

Signed-off-by: Rajendra Dendukuri <rajendra.dendukuri@broadcom.com>
Co-authored-by: Akhilesh Samineni <47657796+AkhileshSamineni@users.noreply.github.com>
Co-authored-by: rlhui <48738894+rlhui@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants