Bug 2319785 - Please branch and build python-keyring for EPEL 10
Summary: Please branch and build python-keyring for EPEL 10
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: python-keyring
Version: epel10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Ben Beasley
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2345927 2345928
Blocks: EPEL10Tracker 2313925 2345496 2345923
TreeView+ depends on / blocked
 
Reported: 2024-10-18 14:51 UTC by Xavier Bachelot
Modified: 2025-04-09 03:08 UTC (History)
7 users (show)

Fixed In Version: python-keyring-25.6.0-4.el10_1
Clone Of:
Environment:
Last Closed: 2025-04-09 03:08:37 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Xavier Bachelot 2024-10-18 14:51:38 UTC
Hi,

Could you please branch and build python-keyring for EPEL 10 ?
It is needed to build another package.

Regards,
Xavier

Comment 1 Xavier Bachelot 2024-10-18 14:52:15 UTC
Needed for ddupdate.
See https://tdawson.fedorapeople.org/epel/willit/epel10/status-wont-install.html

Comment 2 Ben Beasley 2024-10-18 17:21:07 UTC
Well, there is a missing test dependency on python3dist(pyfakefs), but it looks like its use in the tests is very minimal. I’m investigating whether conditionalizing that test dependency is enough to unblock this package for EPEL10.

Comment 3 Ben Beasley 2024-10-18 17:48:58 UTC
I have conditionalized the dependency on python3dist(pyfakefs) in Rawhide. Now:

$ fedpkg --release epel10 mockbuild -- --postinstall
[…]
No matching package to install: 'python3dist(jaraco-classes)'
No matching package to install: 'python3dist(jaraco-context)'
No matching package to install: 'python3dist(jaraco-functools)'
[…]
No matching package to install: 'python3dist(jeepney) >= 0.4.2'
[…]
No matching package to install: 'python3dist(secretstorage) >= 3.2'
[…]
No matching package to install: 'python3dist(shtab) >= 1.1'

Once all of these dependencies are available in EPEL10, I will be happy to branch, build, and maintain python-keyring there. However, someone who cares about keeping ddupdate in EPEL10 would need to handle actually requesting the dependencies (and any dependencies they might have, and so on).

Comment 4 Ben Beasley 2024-11-07 12:37:35 UTC
The remaining missing dependencies in EPEL10 are:

No matching package to install: 'python3dist(jeepney) >= 0.4.2'
No matching package to install: 'python3dist(secretstorage) >= 3.2'
No matching package to install: 'python3dist(shtab) >= 1.1'

Again, I’m happy to maintain this package in EPEL10, but I’m not interested in requesting or maintaining its dependencies myself.

Comment 5 Romain Geissler 2025-02-13 09:37:42 UTC
Checking right now, it seems we miss this:

No matching package to install: 'python3dist(jeepney) >= 0.4.2'
No matching package to install: 'python3dist(secretstorage) >= 3.2'

Comment 6 Romain Geissler 2025-03-27 00:15:31 UTC
With python-secretstorage which is for now in testing, this package can now be built but... the tests fail on EPEL 10...

There are two issues:
 - The tests inside test_multiprocess.py are deadlocked. These tests are actually marked as xfail upstream, so I am not sure it's actually ok to just skip them with something like:

%if 0%{?el10}
k="${k-}${k+ and }not test_multiprocess.py"
%endif

  - The libsecret tests seems to fail with errors like:

__________________ TestLibSecretKeyring.test_password_set_get __________________

self = <tests.backends.test_libsecret.TestLibSecretKeyring object at 0x7fcb87ada570>
    
    def test_password_set_get(self):
        password = random_string(20)
        username = random_string(20)
        service = random_string(20)
>       self.check_set_get(service, username, password)

keyring/testing/backend.py:71:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
keyring/testing/backend.py:60: in check_set_get
    self.set_password(service, username, password)
keyring/testing/backend.py:50: in set_password
    self.keyring.set_password(service, username, password)
keyring/backend.py:60: in wrapper 
    return orig(self, system, username, *args, **kwargs)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <keyring.backends.libsecret.Keyring object at 0x7fcb858fa4b0>
service = 'lRopHVQ7YMSflc0PLfup', username = 'IprHAxnOQDsI3qy3q1p4'
password = 'KUCxDGnHc9EuektcAc0E'   

    def set_password(self, service, username, password):
        """Set password for the username of the service"""
        attributes = self._query(service, username, application=self.appid)
        label = f"Password for '{username}' on '{service}'"
        try:
>           stored = Secret.password_store_sync(
                self.schema, attributes, self.collection, label, password, None
            )
E           gi.repository.GLib.GError: g-dbus-error-quark: Couldn't create item: The secret was transferred or encrypted in an invalid way. (0)

keyring/backends/libsecret.py:88: Error
---------------------------- Captured stdout setup -----------------------------
Testing LibSecretKeyring; the following password prompts are for this keyring

Comment 7 Romain Geissler 2025-03-27 00:23:03 UTC
Also note that pyfakefs is now available in EPEL 10.

Comment 8 Ben Beasley 2025-04-05 11:30:33 UTC
I’m investigating.

Comment 9 Fedora Update System 2025-04-05 14:40:01 UTC
FEDORA-EPEL-2025-62c2a1bbc0 (python-keyring-25.6.0-4.el10_1) has been submitted as an update to Fedora EPEL 10.1.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-62c2a1bbc0

Comment 10 Fedora Update System 2025-04-06 02:39:54 UTC
FEDORA-EPEL-2025-62c2a1bbc0 has been pushed to the Fedora EPEL 10.1 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-62c2a1bbc0

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 11 Fedora Update System 2025-04-09 03:08:37 UTC
FEDORA-EPEL-2025-62c2a1bbc0 (python-keyring-25.6.0-4.el10_1) has been pushed to the Fedora EPEL 10.1 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.