-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
Update path to sanity test requirements.txt #63725
Conversation
The path `test/runner/requirements/sanity.txt` does not exist. Opening this PR based on the discussion from IRC #ansible-aws. ```<PrasadK_> Prasad Katti I am trying to run sanity tests but it seems my environment is missing some packages. I found a page that says `pip install --user -r test/runner/requirements/sanity.txt` should give me the required packages. But that looks like an old doc. I can't the file sanity.txt at that path. 3:12:00 PM <felixfontein> Felix Fontein PrasadK_: if you have docker, it's best to run the tests in a docker container by adding `--docker default` to the `ansible-test sanity` arguments 3:12:11 PM that container contains everything that is needed to run the sanity tests 3:13:15 PM also, I think the file is now test/sanity/requirements.txt 3:13:17 PM <PrasadK_> Prasad Katti thanks, that exactly what I was just trying out. :) 3:13:46 PM <felixfontein> Felix Fontein please create a docs PR to fix the path in the docs :) 3:14:06 PM <PrasadK_> Prasad Katti ah, okay. I will do that. 3:14:29 PM <felixfontein> Felix Fontein thanks a lot!```
@felixfontein - created this PR as discussed on IRC |
How about also mentioning |
I mentioned that on IRC. I found out about that command from the output of |
@@ -29,7 +29,7 @@ To ensure that your module documentation matches your ``argument_spec``: | |||
|
|||
.. code-block:: bash | |||
|
|||
pip install --user -r test/runner/requirements/sanity.txt | |||
pip install --user -r test/sanity/requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I recommend directing users to use either the --docker
or --venv
options. If running in their own venv, using --requirements
also works. If requirements are to be installed manually then things get more complicated, as there are multiple requirements files involved, depending on the tests being executed and whether testing is being done on ansible/ansible or an Ansible Collection.
@prasadkatti Thank you for this PR. Would you be able to take a look at Matt's suggestion above? |
Duplicate of #62314? |
SUMMARY
The path
test/runner/requirements/sanity.txt
does not exist.Opening this PR based on the discussion from IRC #ansible-aws.
ISSUE TYPE
COMPONENT NAME
sanity testing guide
ADDITIONAL INFORMATION