sonic-net / sonic-mgmt

Configuration management examples for SONiC
Other
200 stars 732 forks source link

Fix test_syslog_source_ip issue #15639

Open JibinBao opened 1 week ago

JibinBao commented 1 week ago

Description of PR

When removing mgmt vrf, sometimes dut connection will be lost for a while. So, before config reload we need to remove mgmt vrf, otherwise it will cause host unreachable.

Summary: Fixes # (issue)

Type of change

Back port request

Approach

What is the motivation for this PR?

Fix pytest_ansible.errors.AnsibleConnectionFailure: Host unreachable in the inventory in restore_config_by_config_reload

How did you do it?

Before config reload remove mgmt vrf

How did you verify/test it?

Run all test_syslog_source_ip case

Any platform specific information?

Any

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

Documentation

JibinBao commented 1 week ago

/azpw run Azure.sonic-mgmt

mssonicbld commented 1 week ago

/AzurePipelines run Azure.sonic-mgmt

azure-pipelines[bot] commented 1 week ago
Azure Pipelines successfully started running 1 pipeline(s).
JibinBao commented 1 week ago

/azpw run Azure.sonic-mgmt

mssonicbld commented 1 week ago

/AzurePipelines run Azure.sonic-mgmt

azure-pipelines[bot] commented 1 week ago
Azure Pipelines successfully started running 1 pipeline(s).
bingwang-ms commented 12 hours ago

The test case was skipped because of issue https://github.com/sonic-net/sonic-mgmt/issues/6479. Will the issue be fixed by this PR?

JibinBao commented 7 hours ago

The test case was skipped because of issue #6479. Will the issue be fixed by this PR?

No, for issue of https://github.com/sonic-net/sonic-mgmt/issues/6479, the fix is https://github.com/sonic-net/sonic-mgmt/pull/13280. Hi @StormLiangMS ,as what we discussed before, Can you close the issue https://github.com/sonic-net/sonic-mgmt/issues/6479?