Hi Tom,

I think you're right, and the symptoms are similar but the tracebacks are different.  Different calls to `getpwnam` ;).  I put a comment on the issue for posterity so it's clear what happened there.



On Aug 30, 2022, at 12:12 AM, Tom Most <twm@freecog.net> wrote:

Hi glyph!

]It looks like that is a different issue, not part of Conch's twisted.cred checkers. I filed https://github.com/twisted/twisted/issues/11626 with the traceback you posted, but it would be helpful if you could add steps to reproduce for folks like me who are just tiptoeing into Conch.

As is often the case, the place where this is happening "in production" is muddled together with a ton of unrelated configuration.  I'll try to minify the steps to use the system OpenSSH keys and moduli to make it a one-liner when I can.

I think that this may also have been impacted by this PyPy change: https://foss.heptapod.net/pypy/pypy/-/commit/106f751add4e0d9efe614b0bebd75844ccb43bca. If you were using a less buggies compatible version of PyPy before it would have worked.

I think that the changelog entry is technically correct, though it could be misread to say that Conch's CLI works. We could absolutely clarify that.

You're right.  "So it works on Python 3" is rather awkward wording that maybe we could fix, but it doesn't say anything about the CLI and any misreading would not really be substantiated by the text.  Let's just focus on getting the bug fixed and rolling another release when we can :-).

Best,
Tom

On Mon, Aug 29, 2022, at 12:37 AM, Glyph wrote:
Thanks for another release, Adi!

This isn't a blocker, but we should probably fix the changelog, since https://github.com/twisted/twisted/issues/10286 is not actually fixed.  I just installed it on my conch server and saw the traceback in https://github.com/twisted/twisted/issues/10286#issuecomment-1229896032.

And then of course we should chase down the bug and actually fix it this time :).

-g

On Aug 28, 2022, at 4:52 AM, Adi Roiban <adiroiban@gmail.com> wrote:

On behalf of the Twisted contributors, I announce the release candidate
of Twisted 22.8.0

This is the first release after we migrated from Trac Tickets to GitHub Issues.

In this release, the API docs were updated to a newer pydoctor version
and the RTD theme is enabled.

The old Trac twistedmatrix.com website is now offline.
Redirections were implemented on a best effort basis :)
If you find any broken link you can report this via GitHub Issues

`trial --jobs=N --exitfirst` is now supported, for a much better
experience when running parallel tests.
If you haven't try trial parallel test run yet, give it a try.
On local dev computers with many CPUs you get a huge performance
improvement (assuming your tests are CPU bound )

Python 3.6 is no longer supported in this version.

The release and NEWS file is available for review at


Release candidate documentation is available at


Wheels for the release candidate are available on PyPI


  python -m pip install Twisted==22.8.0rc1

Please test it and report any issues.
If nothing comes up in one week,
22.8.0 will be released based on the latest release candidate.

Many thanks to everyone who had a part in Twisted development,
the supporters of the Twisted Software Foundation,
the developers, and all the people testing and building great things
with Twisted!

Slava Ukraini!
-- 
Adi Roiban
_______________________________________________
Twisted mailing list -- twisted@python.org
To unsubscribe send an email to twisted-leave@python.org
_______________________________________________
Twisted mailing list -- twisted@python.org
To unsubscribe send an email to twisted-leave@python.org
Code of Conduct: https://twisted.org/conduct


_______________________________________________
Twisted mailing list -- twisted@python.org
To unsubscribe send an email to twisted-leave@python.org
https://mail.python.org/mailman3/lists/twisted.python.org/
Message archived at https://mail.python.org/archives/list/twisted@python.org/message/GRSBK2TOQWH4WLOPHBBOTMCD4R5KXYMI/
Code of Conduct: https://twisted.org/conduct