Skip to content

fix pretest#18653

Open
arlakshm wants to merge 1 commit intosonic-net:masterfrom
arlakshm:dev/arlakshm/fix_pretest
Open

fix pretest#18653
arlakshm wants to merge 1 commit intosonic-net:masterfrom
arlakshm:dev/arlakshm/fix_pretest

Conversation

@arlakshm
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes #18598

Type of change

  • [ x] Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Fix the issue #18652

How did you do it?

Add bmp as one of the critical services for the asic

How did you verify/test it?

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm arlakshm requested review from tjchadaga and ysmanman May 27, 2025 13:42
Copy link
Copy Markdown
Contributor

@YatishSVC YatishSVC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@arista-nwolfe
Copy link
Copy Markdown
Contributor

@arlakshm is there still plans to push this change in?

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.

Regression: test_pretest.py::test_disable_rsyslog_rate_limit failed on multi-asic

5 participants