Skip to content

Update vrf unbind command in Command-References.md#2331

Merged
prsunny merged 1 commit intosonic-net:masterfrom
KhoobBabe:update-vrf-unbind
Aug 24, 2022
Merged

Update vrf unbind command in Command-References.md#2331
prsunny merged 1 commit intosonic-net:masterfrom
KhoobBabe:update-vrf-unbind

Conversation

@KhoobBabe
Copy link
Contributor

What I did

Updated VRF unbind command to remove additional <vrf_name> argument.

How I did it

  • N/A

How to verify it

  • The argument <vrf_name> is unrecognized for unbind command.
    image

  • --help for VRF unbind shows its USAGE without the <vrf_name> argument.
    image

Previous command output (if the output of a command-line utility has changed)

  • N/A

New command output (if the output of a command-line utility has changed)

  • N/A

@lgtm-com
Copy link

lgtm-com bot commented Aug 24, 2022

This pull request introduces 1 alert and fixes 1 when merging c2985db into e14f679 - view on LGTM.com

new alerts:

  • 1 for Syntax error

fixed alerts:

  • 1 for Unused local variable

@KhoobBabe
Copy link
Contributor Author

@Blueve @judyjoseph Could you please review?

@prsunny prsunny merged commit 7d409a0 into sonic-net:master Aug 24, 2022
@KhoobBabe KhoobBabe deleted the update-vrf-unbind branch August 30, 2022 03:55
@KhoobBabe
Copy link
Contributor Author

@prsunny Can you also merge the same PR (same vrf unbind command change) made by my colleague here? sonic-net/SONiC#1070

preetham-singh pushed a commit to preetham-singh/sonic-utilities that referenced this pull request Nov 21, 2022
*Update vrf unbind command in Command-References.md
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.

2 participants