Skip to content

[run_tests.sh] Add module_utils path#2697

Closed
lolyu wants to merge 1 commit intosonic-net:masterfrom
lolyu:add_ansible_module_utils_path
Closed

[run_tests.sh] Add module_utils path#2697
lolyu wants to merge 1 commit intosonic-net:masterfrom
lolyu:add_ansible_module_utils_path

Conversation

@lolyu
Copy link
Collaborator

@lolyu lolyu commented Dec 23, 2020

Description of PR

For Pytest, we need to let pytest-ansible aware of the module_utils
path.

Signed-off-by: Longxiang Lyu [email protected]

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

  • Pytest conn_graph_facts fixture failed with error:
>           raise RunAnsibleModuleFail("run module {} failed".format(self.module_name), res)
E           RunAnsibleModuleFail: run module conn_graph_facts failed, Ansible Results =>
E           {
E               "failed": true,
E               "msg": "Could not find imported module support code for conn_graph_facts.  Looked for either print_debug_msg.py or debug_utils.py"
E           }

How did you do it?

Add the environment variable ANSIBLE_MODULE_UTILS to the test runner script.

How did you verify/test it?

Run Pytest.

Any platform specific information?

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

Documentation

For Pytest, we need to let pytest-ansible aware of the `module_utils`
path.

Signed-off-by: Longxiang Lyu <[email protected]>
@lolyu lolyu requested a review from a team December 23, 2020 07:32
@lolyu lolyu closed this Dec 23, 2020
@lolyu lolyu deleted the add_ansible_module_utils_path branch December 23, 2020 07:53
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (sonic-net#14796)

src/sonic-swss

* 113cadd - (HEAD -> 202205, origin/202205) [portsorch]: Set default hostif TX queue (sonic-net#2697) (11 days ago) [prabhataravind]
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#14943)

src/sonic-utilities

* b49d78c4 - (HEAD -> 202205, origin/202205) Change default CDB run mode to non-hitless (sonic-net#2817) (2 hours ago) [mihirpat1]
* 2e9c99e8 - [show][muxcable] fix `show mux hwmode muxdirection` RC (sonic-net#2812) (2 hours ago) [Jing Zhang]
* 6b5bf990 - [config]config reload should generate sysinfo if missing (sonic-net#2778) (2 hours ago) [jingwenxie]
* a8455fd3 - Fix bug in GCU vlanintf_validator (sonic-net#2765) (2 hours ago) [jingwenxie]
* 79c99718 - [GCU] Add vlanintf-validator (sonic-net#2697) (2 hours ago) [jingwenxie]
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (sonic-net#16335)

#### Why I did it
src/sonic-swss
```
* 16817324 - (HEAD -> 202211, origin/202211) [mux]: Fix UTs segmentation fault (sonic-net#2760) (12 hours ago) [Nazarii Hnydyn]
* 0fa5d880 - [orchagent]: Handle additional SAI error conditions gracefully (sonic-net#2755) (2 days ago) [prabhataravind]
* 3726aebc - [mux]: Implement rollback for failed mux switchovers (sonic-net#2714) (2 days ago) [Lawrence Lee]
* a8e50e7d - [portsorch]: Set default hostif TX queue (sonic-net#2697) (2 days ago) [prabhataravind]
* 0689d656 - Add missing parameter to on_switch_shutdown_request method. (sonic-net#2567) (2 days ago) [Hua Liu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

1 participant