Skip to content

[kvm/setup] Add script to setup VS management network#1952

Merged
yxieca merged 1 commit intosonic-net:masterfrom
daall:mgmt_setup_script
Jul 24, 2020
Merged

[kvm/setup] Add script to setup VS management network#1952
yxieca merged 1 commit intosonic-net:masterfrom
daall:mgmt_setup_script

Conversation

@daall
Copy link
Contributor

@daall daall commented Jul 22, 2020

Signed-off-by: Danny Allen [email protected]

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

I got tired of setting up the management network whenever I rebooted my dev VM so I added a script to do it in one step. :P

How did you do it?

Put the steps from the README into a script and added some validation checks.

How did you verify/test it?

daall@sahale ~/repo/sonic-mgmt/ansible % ./setup-management-network.sh 
Root privilege required
daall@sahale ~/repo/sonic-mgmt/ansible % sudo ./setup-management-network.sh 
STEP 1: Checking for bridge-utils package...
/usr/sbin/brctl

STEP 2: Checking for net-tools package...
/usr/sbin/ifconfig

STEP 3: Checking if bridge br1 already exists...
bridge br1 does not exist!

STEP 4: Creating management bridge br1...

COMPLETE. Bridge info:

bridge name	bridge id		STP enabled	interfaces
br1		8000.000000000000	no		

br1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet 10.250.0.1  netmask 255.255.255.0  broadcast 10.250.0.255
        inet6 fe80::416:30ff:feba:f80e  prefixlen 64  scopeid 0x20<link>
        ether 06:16:30:ba:f8:0e  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

Any platform specific information?

KVM

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

N/A

Documentation

Updated README

@yxieca
Copy link
Collaborator

yxieca commented Jul 23, 2020

retest vsimage please

@yxieca yxieca merged commit 6ab6159 into sonic-net:master Jul 24, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
f81043b1f9ff02196629655f4735b33afd7f0ae1 (HEAD -> 202111, origin/202111) [port2alias]: Fix to get right number of return values (sonic-net#1906)
bbbf65943ec46e9330eadaed8bcdf1612cb8bd55 [CLI][show bgp] On chassis don't show internal BGP sessions by default (sonic-net#1927)
e12de7e7bf6cff3ec127f261bf88e4d29776d27b [port] Fix port speed set (sonic-net#1952)
cae7af752d484956d7fe40e4c3a849ddad460976 Fix invalid output of syslog IPv6 servers (sonic-net#1933)
6009341ddf790094166be5f0a81b4c114f00220b Routed subinterface enhancements (sonic-net#1821)
6ab9d67ca6550c592b97afb513804be474f84eb0 Enhance sfputil for CMIS QSFP (sonic-net#1949)
76cc67ba4f81c69b20efb3341808037c9db8f703 [debug dump] Refactoring Modules and Unit Tests (sonic-net#1943)
cff58a8171423e4012bc8caf9748996a1e98b7e2 Add command reference for trap flow counters (sonic-net#1876)
71cf3ee43524d56ad57dd90b937cfbf4bf63ba6a [Reclaim buffer] [Mellanox] Db migrator support reclaiming reserved buffer for unused ports (sonic-net#1822)
e699b49fb722e6d6fe5a1d2dacd2d39eb085c1e4 Add show command for BFD sessions (sonic-net#1942)
bb6c5774c843dbfad5f1ba00ee76dae7720902d1 [warm-reboot] Fix failures of warm reboot on disconnect of ssh session (sonic-net#1529)
2e8bbb308477862a76d2327fcf696875e8f08650 Add trap flow counter support (sonic-net#1868)
58407c1386ef13772a9a9320a795e380f162ab2c [load_minigraph] Delay pfcwd start until the buffer templates are rendered (sonic-net#1937)
eb388e0584ba1fe8d8dba58f1c5a148036ffe047 [sonic-package-manager] support sonic-cli-gen and packages with YANG model (sonic-net#1650)
2371d84e7d281bdb9988b5a1a012498dbbfb89ec generic_config_updater: Filename changed & VLAN validator added (sonic-net#1919)
7c0718dfaf23289d4ecc3ada9332e465c9a4e56b [config reload] Update command reference (sonic-net#1941)

Signed-off-by: Abhishek Dosi <[email protected]>
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.

3 participants