Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 1746853

Summary: python-subunit2sql fails to build with Python 3.8 on Fedora 32+
Product: [Fedora] Fedora Reporter: Miro Hrončok <mhroncok>
Component: python-subunit2sqlAssignee: Miro Hrončok <mhroncok>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 32CC: chkumar246, chkumar, mplch
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-04-01 13:14:33 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1750908, 1750909, 1686977, 1785415    

Description Miro Hrončok 2019-08-29 11:45:54 UTC
python-subunit2sql fails to build with Python 3.8.0b3 in Fedora 32.

See the build failures at https://koji.fedoraproject.org/koji/search?match=glob&type=package&terms=python-subunit2sql

==============================
Failed 1 tests - output below:
==============================
subunit2sql.tests.migrations.test_migrations.TestWalkMigrations.test_sqlite_opportunistically
---------------------------------------------------------------------------------------------
Captured traceback:
~~~~~~~~~~~~~~~~~~~
    b'Traceback (most recent call last):'
    b'  File "/builddir/build/BUILD/subunit2sql-1.9.0/subunit2sql/tests/migrations/test_migrations.py", line 208, in test_sqlite_opportunistically'
    b'    self._walk_versions(engine)'
    b'  File "/builddir/build/BUILD/subunit2sql-1.9.0/subunit2sql/tests/migrations/test_migrations.py", line 107, in _walk_versions'
    b'    self._migrate_up(engine, dest, curr, with_data=True)'
    b'  File "/builddir/build/BUILD/subunit2sql-1.9.0/subunit2sql/tests/migrations/test_migrations.py", line 120, in _migrate_up'
    b'    check(engine, data)'
    b'  File "/builddir/build/BUILD/subunit2sql-1.9.0/subunit2sql/tests/migrations/test_migrations.py", line 324, in _check_5332fe255095'
    b'    result = tests.select(where).execute()'
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/sql/selectable.py", line 331, in select'
    b'    return Select([self], whereclause, **params)'
    b'  File "<string>", line 2, in __init__'
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/util/deprecations.py", line 128, in warned'
    b'    return fn(*args, **kwargs)'
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/sql/selectable.py", line 2989, in __init__'
    b'    self._whereclause = _literal_as_text(whereclause).self_group('
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 4526, in _literal_as_text'
    b'    return _literal_as(element, _no_text_coercion)'
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 4512, in _literal_as'
    b'    return text_fallback(element)'
    b'  File "/usr/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 4551, in _no_text_coercion'
    b'    raise exc_cls('
    b'sqlalchemy.exc.ArgumentError: Textual SQL expression "tests.id=\'fake_null_test_..." should be explicitly declared as text("tests.id=\'fake_null_test_...")'
    b''
======
Totals
======
Ran: 302 tests in 74.1100 sec.
 - Passed: 132
 - Skipped: 169
 - Expected Fail: 0
 - Unexpected Success: 0
 - Failed: 1
Sum of execute time for each test: 348.3258 sec.
==============
Worker Balance
==============
 - Worker 0 (51 tests) => 0:01:04.271906
 - Worker 1 (51 tests) => 0:00:47.849951
 - Worker 2 (50 tests) => 0:00:36.009444
 - Worker 3 (50 tests) => 0:01:13.893848
 - Worker 4 (50 tests) => 0:00:57.099807
 - Worker 5 (50 tests) => 0:01:09.512572
RPM build errors:
BUILDSTDERR: error: Bad exit status from /var/tmp/rpm-tmp.WW0njX (%check)
BUILDSTDERR:     Bad exit status from /var/tmp/rpm-tmp.WW0njX (%check)

It is not important whether the problem is relevant to Python 3.8, this issue is blocking the Python 3.8 rebuilds.
If this package won't build with 3.8, it won't be installable, along with all its dependent packages, in Fedora 32 and further.

Furthermore, as it fails to install, its dependent packages will fail to install and/or build as well.

Please rebuild the package in Fedora 32 (rawhide).

Let us know here if you have any questions. Thank You!

Comment 1 Chandan Kumar 2019-11-12 12:07:48 UTC
Hello Miro,

I am looking into it. Subunit2sql requires a new release upstream in order to have the tests fixed.

Thanks,

Chandan Kumar

Comment 2 Ben Cotton 2020-02-11 17:30:11 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 32 development cycle.
Changing version to 32.

Comment 3 Miro Hrončok 2020-02-29 17:35:26 UTC
Chandan, what is the status here? Is this doable in the Fedora 32 time frame?

Comment 4 Miro Hrončok 2020-03-08 23:41:04 UTC
FWIW I'd like this package to be retired from Fedora 32 before the final freeze if it still fails to build by then.

Comment 5 Miro Hrončok 2020-03-29 15:02:43 UTC
If there is no reply, I'll retire the package one week before the final freeze, so we have time to utretie it if it's needed for the GA.

That's on Tuesday 2020-03-31.

Comment 6 Miro Hrončok 2020-03-31 10:30:08 UTC
Retired.