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

Changing show chassis-module cli command as per PR 805 #3762

Merged
merged 1 commit into from Oct 29, 2021

Conversation

rawal01
Copy link
Contributor

@rawal01 rawal01 commented Jul 9, 2021

  • Bug fix
  • [x ] Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

show chassis-modules command is now being changed to show chassis module please refer to PRs below:

sonic-net/SONiC#805
sonic-net/SONiC#801

This PR has dependency on above PRs and can only be merged after they are merged

How did you do it?

tests/platform_tests/cli/test_show_chassis_module.py change cli command from show chassis-modules to show chassis modules

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

  tests/platform_tests/cli/test_show_chassis_module.py change cli command from show chassis-modules to show chassis modules
@anshuv-mfst
Copy link

Hi @arlakshm - could you please review, related to #801 and #805, thanks

@arlakshm
Copy link
Contributor

This PR needs to merged after the code PR sonic-net/sonic-utilities#1689 is merged

@anshuv-mfst anshuv-mfst requested review from a team and removed request for jleveque August 3, 2021 23:11
@arlakshm arlakshm merged commit 15d0793 into sonic-net:master Oct 29, 2021
harish-kalyanaraman pushed a commit to harish-kalyanaraman/sonic-mgmt that referenced this pull request Nov 18, 2021
…- rawal01:platform-cli-change)

  Changing show chassis-module cli command as per PR sonic-net/SONiC#805
  tests/platform_tests/cli/test_show_chassis_module.py change cli command from show chassis-modules to show chassis modules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants