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
Add timeout flag to pytest #3363
Conversation
67a8896
to
fe3d17b
Compare
At first, I inserted "--timeout=60" to this config file, but I have no idea how to install pytest-timeoout in CI enviroment. This is why this cli option is in every |
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.
To add pytest-timeout to CI, you need to add it to the installed packages in devel/ci/cico.pipeline
(look for pytest there).
Also, I think it would be better to add the CLI option to setup.cfg
, where options are already added to the command line.
fe3d17b
to
cde66f9
Compare
Yeah, this is the place I was looking for. |
8b0be3f
to
5c1d641
Compare
fix fedora-infra#3339 Signed-off-by: Sebastian Wojciechowski <s.wojciechowski89@gmail.com>
5c1d641
to
428f73f
Compare
@abompard It looks like integration tests are still missing pytest-timeout. In the PR I wrote the following command: |
Closing this due to inactivity |
fix #3339
Signed-off-by: Sebastian Wojciechowski s.wojciechowski89@gmail.com