Skip to content

[vlan] Bring up LAGs after shutdown them for vlan test configuration#835

Merged
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:vlan-config-pr
Mar 22, 2019
Merged

[vlan] Bring up LAGs after shutdown them for vlan test configuration#835
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:vlan-config-pr

Conversation

@wangxin
Copy link
Collaborator

@wangxin wangxin commented Mar 21, 2019

Description of PR

Summary:
Fixes # (issue)

In vlan_configuration.yml, the LAGs are shutdown for loading
/etc/sonic/vlan_configuration.json. Need to bring them back after the
configuration is loaded. Otherwise, the LAGs will remain down and PTF
test would fail.

Type of change

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

Approach

How did you do it?

Bring back the LAGs after loaded configuration for VLAN testing.

How did you verify/test it?

Tested on Mellanox platform.

Any platform specific information?

None

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

Documentation

In vlan_configuration.yml, the LAGs are shutdown for loading
/etc/sonic/vlan_configuration.json. Need to bring them back after the
configuration is loaded. Otherwise, the LAGs will remain down and PTF
test would fail.

Signed-off-by: Xin Wang <[email protected]>
@lguohan lguohan merged commit a8e3b9a into sonic-net:master Mar 22, 2019
@wangxin wangxin deleted the vlan-config-pr branch May 24, 2019 03:33
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
6cfb3ecb0248768da0a91e5f7fb4477c5da7eb4e (HEAD -> 201911, origin/201911) [build]: allow to use extra inc/lib location to build the package (sonic-net#595)
40d34872d3b7f354adac67f084eebf6ee467f779 Merge pull request sonic-net#846 from xumia/azp-201911
76ac50f147a7d820b19d8d7628a67f2fe4f5159b Disable the build test
6c9cf655b8b5b152cab1d578e05eddf8238b81b0 Fix branch reference error
ca8d81d37a9b0294098f161b036d330d9ff461e0 [ci]: download artifacts from master branch (sonic-net#768)
0cbf4d55c67a9f8f52715f95536f3588acf06c4a [ci]: use sonicbld pool (sonic-net#766)
b6f1265ee9bd86f8a5e909a6f1e9b2384497c906 [ci]: add build for arm64 and armhf (sonic-net#757)
9ec0a7da64d479b124815edc5b505fb88b2532a0 CI: add azure pipeline CI/CD (sonic-net#754)
1436dbe02cd3c56f796c6b3398d4075cd05d97e0 Fix RIF issue (sonic-net#835)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants