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

Testsuite fails on alpha due to incorrect filename for libc.so.6.1 #21396

Closed
glaubitz opened this issue Dec 7, 2024 · 2 comments · Fixed by #21431
Closed

Testsuite fails on alpha due to incorrect filename for libc.so.6.1 #21396

glaubitz opened this issue Dec 7, 2024 · 2 comments · Fixed by #21431
Assignees
Labels

Comments

@glaubitz
Copy link
Contributor

glaubitz commented Dec 7, 2024

Explain what happens

The testsuite fails on alpha as it assumes that the name of the shared C library is always libc.so.6:

=========================== short test summary info ============================
ERROR test/pytest/test_beiboot.py - OSError: libc.so.6: cannot open shared object file: No such file or directory
ERROR test/pytest/test_bridge.py - OSError: libc.so.6: cannot open shared object file: No such file or directory
ERROR test/pytest/test_pcp.py - OSError: libc.so.6: cannot open shared object file: No such file or directory
ERROR test/pytest/test_peer.py - OSError: libc.so.6: cannot open shared object file: No such file or directory
ERROR test/pytest/test_transport.py - OSError: libc.so.6: cannot open shared object file: No such file or directory
!!!!!!!!!!!!!!!!!!! Interrupted: 5 errors during collection !!!!!!!!!!!!!!!!!!!!
================== 1 deselected, 1 warning, 5 errors in 3.96s ==================

However, on alpha and ia64, it's actually libc.so.6.1.

Full build log at: https://buildd.debian.org/status/fetch.php?pkg=cockpit&arch=alpha&ver=330-1&stamp=1733378756&raw=0

Version of Cockpit

330

Where is the problem in Cockpit?

None

Server operating system

Debian

Server operating system version

No response

What browsers are you using?

No response

System log

No response

@glaubitz glaubitz added the bug label Dec 7, 2024
@martinpitt martinpitt self-assigned this Dec 12, 2024
@martinpitt
Copy link
Member

martinpitt commented Dec 12, 2024

Dang, there are no porter boxes for alpha nor ia64, so I suppose I need to fix this blindly.

This first needs to be fixed in ferny, then synced over, and then do the same fallback in

src/client/cockpit-client:libc6 = ctypes.cdll.LoadLibrary('libc.so.6')
src/cockpit/transports.py:libc6 = ctypes.cdll.LoadLibrary('libc.so.6')

martinpitt added a commit to allisonkarlitskaya/ferny that referenced this issue Dec 12, 2024
This is the SONAME on ia64/alpha.

See cockpit-project/cockpit#21396
@martinpitt
Copy link
Member

I sent allisonkarlitskaya/ferny#32 . Once that lands, I'll fix it in cockpit.

@martinpitt martinpitt added the blocked Don't land until something else happens first (see task list) label Dec 12, 2024
martinpitt added a commit to allisonkarlitskaya/ferny that referenced this issue Dec 13, 2024
ia64/alpha has SONAME "libc.so.6.1"

`None` works because Python is linked against libc, so the symbol is
already in our address space.

See cockpit-project/cockpit#21396
allisonkarlitskaya pushed a commit to allisonkarlitskaya/ferny that referenced this issue Dec 13, 2024
ia64/alpha has SONAME "libc.so.6.1"

`None` works because Python is linked against libc, so the symbol is
already in our address space.

See cockpit-project/cockpit#21396
martinpitt added a commit to martinpitt/cockpit that referenced this issue Dec 13, 2024
ia64/alpha has SONAME "libc.so.6.1"

`None` works because Python is linked against libc, so the symbol is
already in our address space.

Fixes cockpit-project#21396
@martinpitt martinpitt removed the blocked Don't land until something else happens first (see task list) label Dec 13, 2024
martinpitt added a commit to martinpitt/cockpit that referenced this issue Dec 13, 2024
ia64/alpha has SONAME "libc.so.6.1"

`None` works because Python is linked against libc, so the symbol is
already in our address space.

Fixes cockpit-project#21396
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants