Skip to content

[test_platform_info] Fix issues: sometimes the expected error log is not caught by log analyzer#2575

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

[test_platform_info] Fix issues: sometimes the expected error log is not caught by log analyzer#2575
liat-grozovik merged 1 commit intosonic-net:masterfrom
Junchao-Mellanox:fix_load_invalid_file_test

Conversation

@Junchao-Mellanox
Copy link
Contributor

Description of PR

Summary:
Fixes issue: sometimes the expected error log is not caught by log analyzer. There is chance that the expect error log is in syslog but not caught by log analyzer, move the log analyzer to outter level to make sure it caught the expected log.

Type of change

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

Approach

What is the motivation for this PR?

Fixes issue: sometimes the expected error log is not caught by log analyzer.

How did you do it?

Move the log analyzer to outter level to make sure it caught the expected log

How did you verify/test it?

Manually run the regression

Any platform specific information?

N/A

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

N/A

Documentation

@liat-grozovik liat-grozovik merged commit f1193ee into sonic-net:master Nov 24, 2020
@Junchao-Mellanox Junchao-Mellanox deleted the fix_load_invalid_file_test branch November 27, 2020 01:45
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Update sonic-utilities submodule pointer to include the following:
* c9ed09d [202205] [sonic_installer] use /etc/resolv.conf from the host when migrating packages (sonic-net#2573) ([sonic-net#2575](sonic-net/sonic-utilities#2575))

Signed-off-by: dprital <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants