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

HLD for the SONiC Management Framework "show tech-support" command #756

Merged
merged 10 commits into from Jan 31, 2022

Conversation

kerry-meyer
Copy link
Contributor

This initial version of the SONiC Management Framework "show tech-support" command describes the SONiC Management Framework interface for initiating the collection of technical support information. The specification describes the associated Yang model and flow of control for relaying the command to the SONiC host and receiving a completion status and the location of the resulting techsupport "bundle".

I'm moving this diagram to the correct directory. (doc/mgmt/images)
- Reduce and simplify output field definitions to provide a single completion (output) status field. 
- Provide minor punctuation and formatting fixes.
Delete the incorrectly named version of this file. (The correctly named version, using a title starting with "SONiC_" has been updated.)
Add line breaks to eliminate line overflow in a list of "Feature Overview" items.
@anshuv-mfst
Copy link
Collaborator

Hi @lguohan - please assign for merge & review.

@zhangyanzhao zhangyanzhao added this to In progress in 202111 Release HLD via automation Oct 12, 2021
@zhangyanzhao zhangyanzhao moved this from In progress to Reviewed in 202111 Release HLD Oct 12, 2021
@kerry-meyer
Copy link
Contributor Author

kerry-meyer commented Oct 19, 2021

I'm adding a summary of related PRs for this feature, along with their current status.

Repo PR title State
sonic-buildimage Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 7816 Open
sonic-mgmt-common Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 49  Open
sonic-mgmt-framework Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 86 Open

@zhangyanzhao
Copy link
Collaborator

I'm adding a summary of related PRs for this feature, along with their current status.

Repo PR title State
sonic-buildimage Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 7816 Open
sonic-mgmt-common Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 49  Open
sonic-mgmt-framework Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support" 86 Open

would you please update this table by following example of #806 to show the code PR status? Thanks.

@zhangyanzhao
Copy link
Collaborator

BRCM team will help to review this PR and related code PRs. Thanks.

Modify the "Overview" and "Other Process" sections to note the dependency on sonic-hostservices and the procedure for querying/starting the service as needed.
@renukamanavalan
Copy link
Contributor

@sujinmkang, can you please help with review ?

@renukamanavalan renukamanavalan merged commit faf8731 into sonic-net:master Jan 31, 2022
202111 Release HLD automation moved this from Reviewed to Done Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants