Skip to content

[action] [PR:13213] ipv6_fixture needs to wait for all services to be up#13237

Merged
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/13213
Jun 12, 2024
Merged

[action] [PR:13213] ipv6_fixture needs to wait for all services to be up#13237
mssonicbld merged 1 commit intosonic-net:202405from
mssonicbld:cherry/202405/13213

Conversation

@mssonicbld
Copy link
Collaborator

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

Description of PR
Summary:
Fixes MSFT ADO 28332318

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.

co-authorized by: [email protected]
@mssonicbld
Copy link
Collaborator Author

Original PR: #13213

@mssonicbld mssonicbld merged commit fe70870 into sonic-net:202405 Jun 12, 2024
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.

2 participants