Skip to content

[202311] [PR:13213] ipv6_fixture needs to wait for all services to be up#13241

Merged
yejianquan merged 1 commit intosonic-net:202311from
sdszhang:cherry/202311/13213
Jun 12, 2024
Merged

[202311] [PR:13213] ipv6_fixture needs to wait for all services to be up#13241
yejianquan merged 1 commit intosonic-net:202311from
sdszhang:cherry/202311/13213

Conversation

@sdszhang
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes MSFT ADO 28332318

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?

Fix case error with the following error message:
failed on setup with "Failed: <ipv6 address> not appeared in xxx netstat"

On some physical testbed, the snmpd process took around 3 minutes to come up.
Current convert_and_restore_config_db_to_ipv6_only needs to check all process to be up before proceeding with SNMP checking.

How did you do it?

wait_critical_processes() checks for the critical process which includes snmpd.

How did you verify/test it?

Tested it locally.

Any platform specific information?

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

Documentation

@sdszhang
Copy link
Copy Markdown
Contributor Author

Original PR: #13213

Copy link
Copy Markdown
Contributor

@wsycqyz wsycqyz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@yejianquan yejianquan left a comment

Choose a reason for hiding this comment

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

LGTM

@yejianquan yejianquan merged commit bdc905b into sonic-net:202311 Jun 12, 2024
@sdszhang sdszhang deleted the cherry/202311/13213 branch June 16, 2024 02:04
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