Bug 221029 - REGRESSION(r271634): keyring still failing to retrieve secrets
Summary: REGRESSION(r271634): keyring still failing to retrieve secrets
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Philippe Normand
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-01-27 01:52 PST by Philippe Normand
Modified: 2021-10-07 18:55 PDT (History)
3 users (show)

See Also:


Attachments
Patch (2.55 KB, patch)
2021-01-27 01:55 PST, Philippe Normand
svillar: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Normand 2021-01-27 01:52:23 PST
Bug 220736 is still happening. Downgrading keyring to 7.3.1 makes it work again. This module is now optional in the github scm module, I'm not sure why the version was bumped in the first place, anyway.
Comment 1 Philippe Normand 2021-01-27 01:55:29 PST
Created attachment 418516 [details]
Patch
Comment 2 Michael Catanzaro 2021-01-27 06:54:41 PST
Comment on attachment 418516 [details]
Patch

I agree this is the version of the patch that should land.

But could you please also test if the newest version, 22.0.1, works? We can't actually upgrade to that now, because it doesn't support python2, but if it doesn't work then we need to investigate further and figure out what has gone wrong.

P.S. The keyring package was upgraded for no reason, see https://bugs.webkit.org/show_bug.cgi?id=220736#c12. So downgrading should be fine. But it will be upgraded again eventually....
Comment 3 Philippe Normand 2021-01-27 07:01:52 PST
(In reply to Michael Catanzaro from comment #2)
> Comment on attachment 418516 [details]
> Patch
> 
> I agree this is the version of the patch that should land.
> 
> But could you please also test if the newest version, 22.0.1, works? We
> can't actually upgrade to that now, because it doesn't support python2, but
> if it doesn't work then we need to investigate further and figure out what
> has gone wrong.
> 

This could be done as part of the migration of webkitpy to python3. IDK if there's a tracking bug for that though.

> P.S. The keyring package was upgraded for no reason, see
> https://bugs.webkit.org/show_bug.cgi?id=220736#c12. So downgrading should be
> fine. But it will be upgraded again eventually....

Thanks for confirming!
Comment 4 Michael Catanzaro 2021-01-27 07:21:23 PST
(In reply to Philippe Normand from comment #3)
> This could be done as part of the migration of webkitpy to python3. IDK if
> there's a tracking bug for that though.

webkitpy already fully supports both python2 and python3, and both are tested on the EWS. The tracking bug for that was bug #184986. I was going to say: "If your runtime is based on freedesktop-sdk, it is probably already using python3, because python2 was removed in 19.08 and that was a pretty long time ago." But I see you are manually building python2 on top of freedesktop-sdk 20.08. So... if /usr/bin/python is still pointing to python2, that might indeed explain the behavior difference.

Is there some strong reason that is still needed? It looks like all the python2 elements under Tools/buildstream/elements are only needed by sdk-platform.bst and that suggests they can probably just be dropped... assuming we don't notice any explosions when removed. Nothing in WebKit should still require python2.
Comment 5 Michael Catanzaro 2021-01-27 07:22:16 PST
(afaik webkitpy maintains python2 support only for Apple internal reasons. All our scripts and dependencies should already be fully-compatible with python3. In theory.)
Comment 6 Philippe Normand 2021-01-27 07:30:04 PST
I added Python2 in the SDK indeed, and then moved on with other things :) I suppose it could be removed now indeed.
Comment 7 Michael Catanzaro 2021-01-27 07:33:38 PST
Well maybe not, looks like bug #203701 might need fixed. (I guess it's been a very long time since I last attempted to run layout tests....)
Comment 8 Philippe Normand 2021-01-27 07:38:02 PST
Committed r271952: <https://trac.webkit.org/changeset/271952>
Comment 9 Radar WebKit Bug Importer 2021-01-27 07:39:12 PST
<rdar://problem/73661761>