Skip to content

Fix regular expression in test_platform_info.py#2587

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
Junchao-Mellanox:fix_re_review
Nov 26, 2020
Merged

Fix regular expression in test_platform_info.py#2587
liat-grozovik merged 1 commit intosonic-net:masterfrom
Junchao-Mellanox:fix_re_review

Conversation

@Junchao-Mellanox
Copy link
Contributor

Description of PR

Summary:
Fixes regluar expression error in test_platform_info.py

Type of change

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

Approach

What is the motivation for this PR?

Fixes regluar expression error in test_platform_info.py, the charactor "()" in regular expression must be escaped by ""

How did you do it?

Escapte "()" with "()"

How did you verify/test it?

Manual test

Any platform specific information?

N/A

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

N/A

Documentation

@wangxin
Copy link
Collaborator

wangxin commented Nov 26, 2020

retest vsimage please

@liat-grozovik liat-grozovik merged commit 2371718 into sonic-net:master Nov 26, 2020
@Junchao-Mellanox Junchao-Mellanox deleted the fix_re_review branch November 27, 2020 01:43
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
#### Why I did it

Update for following swss commits:

96180bf - 2023-01-13 : [202012] Bfd default multiplier change  (sonic-net#2615) [siqbal1986]
07506ac - 2023-01-11 : Add missing parameter to on_switch_shutdown_request method. (sonic-net#2567) [Hua Liu]
3253cc8 - 2022-11-30 : Use github code scanning instead of LGTM (sonic-net#2546) [Liu Shilong]
f4df524 - 2023-01-11 : [orchagent]:add local_discriminator to state_db (sonic-net#2587) [Baorong Liu]
f3cd02d - 2022-12-05 : [202012][muxorch] Adding case for maintaining current state (sonic-net#2500) [Nikola Dancejic]
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.

4 participants