Skip to content

SER : Skip memory address for th3 platform#3032

Merged
yxieca merged 2 commits intosonic-net:masterfrom
chitra-raghavan:z9332_ser_test
Feb 26, 2021
Merged

SER : Skip memory address for th3 platform#3032
yxieca merged 2 commits intosonic-net:masterfrom
chitra-raghavan:z9332_ser_test

Conversation

@chitra-raghavan
Copy link
Copy Markdown
Contributor

Description of PR

SER : Skip memory address for th3 platform

Summary:
SER test has some memory locations randomly fail when testing under basic mode. Add those memory to skip list.

Type of change

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

How did you verify/test it?

Running ser test in device for more than 20 times and verify no failures.

Any platform specific information?

Th3

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

Documentation

@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Feb 24, 2021

This pull request introduces 1 alert when merging 68551a2 into 795d794 - view on LGTM.com

new alerts:

  • 1 for Syntax error

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Feb 26, 2021

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Feb 26, 2021

@chitra-raghavan
Copy link
Copy Markdown
Contributor Author

@chitra-raghavan can you open another PR to update this function https://github.com/Azure/sonic-mgmt/blob/master/tests/common/devices.py#L1157 ?

Raised PR #3044

@yxieca yxieca merged commit 16919df into sonic-net:master Feb 26, 2021
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#17173)

#### Why I did it
src/sonic-utilities
```
* 3609e417 - (HEAD -> 202305, origin/202305) [sonic-package-manager] do not modify config_db.json (sonic-net#3032) (2 hours ago) [Stepan Blyshchak]
* 354dfe80 - [sonic_installer]: Improve exception handling: introduce notes. (sonic-net#3028) (3 hours ago) [Nazarii Hnydyn]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

3 participants