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.

Sign up for GitHub

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

Jump to bottom

[test-gap] Add a testcase to validate L2 switch config and functionality across reboots/upgrades #8777

Closed
vaibhavhd opened this issue Jun 29, 2023 · 4 comments
Closed

[test-gap] Add a testcase to validate L2 switch config and functionality across reboots/upgrades #8777

vaibhavhd opened this issue Jun 29, 2023 · 4 comments
Assignees
@hdwhdw @qiluo-msft
Labels
P2 Test gap

Comments

@vaibhavhd
Copy link
Contributor

Description

  1. Cold reboot a device.
  2. Perform L2 config steps via L2 Switch mode · sonic-net/SONiC Wiki (github.com)
  3. Install target image.
  4. Cold reboot
  5. Check config_db

Expectation: config db should not have configuration coming from minigraph (as there is no minigraph.xml).

Describe the results you received:

Describe the results you expected:

Additional information you deem important:

**Output of `show version`:**

```
(paste your output here)
```

**Attach debug file `sudo generate_dump`:**

```
(paste your output here)
```
@qiluo-msft
Copy link
Contributor

This is to prevent potential bug in db_migrator.

@vaibhavhd vaibhavhd mentioned this issue Jun 29, 2023
Merged
@ganglyu ganglyu mentioned this issue Dec 1, 2023
Merged
@StormLiangMS
Copy link
Collaborator

hi @qiluo-msft this one seems a manage plane related test gap, assigned to you for further triage.

@ganglyu ganglyu added the P2 label Jun 20, 2024
@qiluo-msft qiluo-msft assigned hdwhdw and unassigned ganglyu Sep 12, 2024
@hdwhdw hdwhdw mentioned this issue Sep 24, 2024
Merged
7 tasks
@qiluo-msft
Copy link
Contributor

@vaibhavhd Do we really need this step?
Install target image.

I believe a reboot is good enough.

qiluo-msft pushed a commit that referenced this issue Nov 5, 2024
@hdwhdw
Create a test case for testing L2 configuration. (#14714)
9c12485 
Reference: https://github.com/sonic-net/SONiC/wiki/L2-Switch-mode

### Description of PR
Create a test case for configuring switch to L2 mode.

Summary:
Add a test case for writing L2 configuration into config DB. This is precursor to a testcase requested in
issue 8777 #8777

### Approach
Add a test case
#### What is the motivation for this PR?
Address #8777

#### How did you do it?

#### How did you verify/test it?
Ran on virtual switch.
veronica-arista pushed a commit to veronica-arista/sonic-mgmt that referenced this issue Nov 7, 2024
@hdwhdw @veronica-arista
Create a test case for testing L2 configuration. (sonic-net#14714)
b43860b 
Reference: https://github.com/sonic-net/SONiC/wiki/L2-Switch-mode

### Description of PR
Create a test case for configuring switch to L2 mode.

Summary:
Add a test case for writing L2 configuration into config DB. This is precursor to a testcase requested in
issue 8777 sonic-net#8777

### Approach
Add a test case
#### What is the motivation for this PR?
Address sonic-net#8777

#### How did you do it?

#### How did you verify/test it?
Ran on virtual switch.
sreejithsreekumaran pushed a commit to sreejithsreekumaran/sonic-mgmt that referenced this issue Nov 15, 2024
@hdwhdw @sreejithsreekumaran
Create a test case for testing L2 configuration. (sonic-net#14714)
a8563f0 
Reference: https://github.com/sonic-net/SONiC/wiki/L2-Switch-mode

### Description of PR
Create a test case for configuring switch to L2 mode.

Summary:
Add a test case for writing L2 configuration into config DB. This is precursor to a testcase requested in
issue 8777 sonic-net#8777

### Approach
Add a test case
#### What is the motivation for this PR?
Address sonic-net#8777

#### How did you do it?

#### How did you verify/test it?
Ran on virtual switch.
yutongzhang-microsoft pushed a commit to yutongzhang-microsoft/sonic-mgmt that referenced this issue Nov 21, 2024
@hdwhdw @yutongzhang-microsoft
Create a test case for testing L2 configuration. (sonic-net#14714)
f0dec5c 
Reference: https://github.com/sonic-net/SONiC/wiki/L2-Switch-mode

### Description of PR
Create a test case for configuring switch to L2 mode.

Summary:
Add a test case for writing L2 configuration into config DB. This is precursor to a testcase requested in
issue 8777 sonic-net#8777

### Approach
Add a test case
#### What is the motivation for this PR?
Address sonic-net#8777

#### How did you do it?

#### How did you verify/test it?
Ran on virtual switch.
@qiluo-msft
Copy link
Contributor

Closed by #14714

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hdwhdw hdwhdw

@qiluo-msft qiluo-msft

Labels
P2 Test gap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@hdwhdw @qiluo-msft @vaibhavhd @ganglyu @StormLiangMS