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.

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

Add SAI_OBJECT_TYPE_ROUTE_ENTRY:.*doesn't exist ERR to ignore list #15986

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ndancejic
Copy link
Contributor

Error can appear when dualtor systems remove a tunnel route but that route had already been processed. It does not affect functionality since the route had already been removed.

Description of PR

Summary: Adding ignore regex for SAI_OBJECT_TYPE_ROUTE_ENTRY:.*doesn't exist error, which is sometimes failing dualtor tests. The error isn't harmful since it is often caused by tunnel route already being removed shortly before.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

Dualtor tests are sometimes failing because of loganalyzer catching this error. However, test cases pass so the error itself should not fail the test.

How did you do it?

# Ignore meta_sai_validate_route_entry ROUTE_ENTRY doesn't exist
r, ".* ERR swss#orchagent: :- meta_sai_validate_route_entry: object key SAI_OBJECT_TYPE_ROUTE_ENTRY:.*doesn't exist.*"

Added to ansible/roles/test/files/tools/loganalyzer/loganalyzer_common_ignore.txt

How did you verify/test it?

Run locally on dualtor testbed

Any platform specific information?

N/A

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

N/A

Documentation

N/A

ADOs:
29870594
30294047

Error can appear when dualtor systems remove a tunnel route but that
route had already been processed. It does not affect functionality since
the route had already been removed.

Signed-off-by: Nikola Dancejic <[email protected]>
@yxieca
Copy link
Collaborator

yxieca commented Jan 14, 2025

@Ndancejic are we risking missing any actual failure by adding this to global whitelist?

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.

4 participants