Skip to content
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

Check 6.0.0rc1 against Fedora packages #7532

Closed
nicoddemus opened this issue Jul 23, 2020 · 11 comments
Closed

Check 6.0.0rc1 against Fedora packages #7532

nicoddemus opened this issue Jul 23, 2020 · 11 comments
Assignees
Milestone

Comments

@nicoddemus
Copy link
Member

nicoddemus commented Jul 23, 2020

That makes it simple to just look at the milestone and see the outstanding issues for the release.

Originally posted by @nicoddemus in #7528 (comment)

Fedora PR with comments: https://src.fedoraproject.org/rpms/pytest/pull-request/17

@nicoddemus nicoddemus added this to the 6.0 milestone Jul 23, 2020
@nicoddemus
Copy link
Member Author

nicoddemus commented Jul 23, 2020

@hroncok as I mentioned, just created this issue to give this some visibility regarding the 6.0 milestone. Please don't take this trying to create pressure of any kind from our side, take your time or even go ahead and close this if you are OK with 6.0.0rc1 or won't have the time to continue testing other Fedora packages. We definitely appreciate all your efforts on testing pytest and helping out the project. 😁

@hroncok
Copy link
Member

hroncok commented Jul 23, 2020

What I'd like to see addressed before the final 6.0.0 release:

pytest-dev/pytest-xdist#554 (comment)
pytest-dev/pytest-forked#45
pytest-dev/pytest-xdist#558 release
#7534

@hroncok
Copy link
Member

hroncok commented Jul 23, 2020

The analysis is basically done. I can dig deeper if you think so.

@The-Compiler
Copy link
Member

I'd be interested in what happened for the python-sphinxcontrib-programoutput failure (logs). I can't reproduce it locally (and I note upstream seems to not use pytest for their tests). Here's the relevant part of the logs:

+ cd sphinxcontrib-programoutput-0.14
+ PYTHONPATH=build/lib/
+ py.test-3.9 -v build/lib/sphinxcontrib -k 'not test_standard_error_disabled'
============================= test session starts ==============================
platform linux -- Python 3.9.0b5, pytest-6.0.0rc1, py-1.9.0, pluggy-0.13.1 -- /usr/bin/python3
cachedir: .pytest_cache
rootdir: /builddir/build/BUILD/sphinxcontrib-programoutput-0.14
collecting ... collected 0 items / 5 errors

==================================== ERRORS ====================================
__ ERROR collecting build/lib/sphinxcontrib/programoutput/tests/test_cache.py __
ImportError while importing test module '/builddir/build/BUILD/sphinxcontrib-programoutput-0.14/build/lib/sphinxcontrib/programoutput/tests/test_cache.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib64/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
E   ModuleNotFoundError: No module named 'sphinxcontrib.programoutput'

No idea why the import would fail here, given that PYTHONPATH is set in a way that should allow it to be imported. Also no idea what could've changed in pytest here, and as I can't reproduce locally I can't bisect.

@hroncok
Copy link
Member

hroncok commented Jul 23, 2020

Note that running the tests on build/lib/sphinxcontrib is a hack. The problem is we have no way to actually install the package in the sphinxcontrib namespace and run the tests on installed, during rpmbuild.

@The-Compiler
Copy link
Member

The-Compiler commented Jul 24, 2020

One down, three to go:

Should we wait a bit more until those are taken care of before releasing the final 6.0?

@hroncok
Copy link
Member

hroncok commented Jul 24, 2020

Since many CI setups ignore prereleases and will start to pick up 6.0.0 once released, I'd advise to hold on with the release until the issues are fixed (and released).

@nicoddemus
Copy link
Member Author

Since many CI setups ignore prereleases and will start to pick up 6.0.0 once released, I'd advise to hold on with the release until the issues are fixed (and released).

Agreed, thanks for the list @hroncok!

I will try to get to those this weekend. 👍

@webknjaz
Copy link
Member

webknjaz commented Jul 27, 2020

@The-Compiler

Fixed

@The-Compiler
Copy link
Member

Yay! So I think the only thing left is a pytest-xdist release now, correct?

@nicoddemus
Copy link
Member Author

pytest-xdist-1.34.0 released. 😄

Closing this then, thanks everyone!

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

No branches or pull requests

4 participants