Showtech sonic mgmt framework: Add Management Framework functionality for "show tech-support"#86
Merged
renukamanavalan merged 5 commits intosonic-net:masterfrom Jan 20, 2022
Conversation
This file is being added to prepare for a SONiC Community Pull Request for the "show tech-support" feature.
This file is being added to prepare for a SONiC Community Pull Request for the "show tech-support" feature.
Provide changes needed in the Dell Enterprise version of this feature to make it compatible with the SONiC Community code base.
|
This pull request introduces 1 alert when merging cff479f into fd1a854 - view on LGTM.com new alerts:
|
4 tasks
…rrors - Avoid handling a "BaseException" for REST server invocation by specifying a finer granularity for the type of exception being handled. - Print the returned REST server invocation error message when applicable instead of a generic error message.
…eld. These changes make use of an additional output field for the "show tech-support" command. The corresponding output-status leaf is added to the set of output leaves in the Yang definition for the sonic-show-techsupport module. This additional field is specified in the SONiC Community HLD for the Management Framework "show tech-support" feature, but was not present in the initial implementation of the feature. The purpose of the additional field is to provide an indication of "success" or detailed failure status that is separate from the output-filename field. With this change, the output-filename contains either an output filename or a null string, and is no longer used to describe errors in the event of a failure.
|
Hi @renukamanavalan - could you please review and merge, thanks |
|
reviewed |
leeprecy
approved these changes
Oct 21, 2021
|
@renukamanavalan Please review/merge, thanks. |
|
@sujinmkang, Can you please help with the review? |
sujinmkang
approved these changes
Jan 14, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Provide the changes required for supporting the "show-techsupport" command via the SONiC Management Framework front end mechanisms (CLI, REST, and gNOI). The Management Framework functionality implemented by this PR improves on the the capabilities currently provided by the SONiC Click CLI interface via the "show techsupport" command by providing the following additional features:
Unit test results are attached to this PR.
unit_test_log_0607.txt
Corequisite PRs:
sonic-net/sonic-mgmt-common#49