[Python-Dev] AIX, test_ssl in particular, but also AIX failed tests in general - getting to the 'STABLE' side

2018-08-27 Thread Michael
Dear all,

Last week I experimented with test_ssl. My expectation was that test
failures was caused by the openssl.base provided by IBM for AIX not
having a default certificate file (CApath). However, that is not the case.

The tests that fail are similar to:
self.assertRaisesRegex(ssl.SSLError, "PEM lib")

I started out by testing with something as:
if not AIX:
    with self.assertRaisesRegex(ssl.SSLError, "PEM lib"):
    ctx.load_cert_chain(BADCERT)
else:
    with self.assertRaises(ssl.SSLError):
    ctx.load_cert_chain(BADCERT)

This is after an analysis where I saw that calls too SSL were returning
an non-success status (!= 1) while ERR_peek_last_error() regularly
returned 0. Hence, the frequent 'AssertionError: "PEM lib" does not
match "unknown error ...' with "unknown error" the string Python provides.

While above might remove the 'fail messages' it did not satisfy me. So,
I downloaded openssl (1.0.2p) and compiled - with no optimization! And
now, even from Python3.6 I see:

test_ssl passed in 1 min 23 sec

== Tests result: SUCCESS ==

1 test OK.

In short, the failures of test_ssl may be ignored - as far as raising an
exception goes.

a) I am running a bot for Python, and once the argument
"-with-openssl=/opt/aixtools" is added my bot will stop showing these
errors. I mention this so that it is clear why they suddenly disappear
on my bot (but not elsewhere). Also to alert Python-Dev that the AIX
platform, regarding ssl.py, _ssl.c and test_ssl.py functions 'stable'
but is not as friendly when it comes to saying why WHEN (my guess) a
heavily optimized (I am thinking -O3 to -O5) library is used.

b) With this feedback - MAYBE - the team from IBM might review the way
they package openssl and make sure the messages are visible via
ERR_peek_last_error() et al. Ideally, IBM will notice and work on it
without prompting. One can dream :)

c) In the meantime - I am curious to know what this 'proof' means to
Python-Dev.

I have a simple goal - work through the tests that AIX has been failing
historically and figure out why they fail and fix the tests. To that end
I have submitted several PR's - starting back In January, then nothing
as noone ever seemed to notice, and the last weeks several additional
ones. Victor has been kind enough to say he will look at the tests as he
has time (and back from vacation). But we are all, or most, working on
our time. My goal, rephrased, is to see AIX in the 'stable' column so
that when a test fails it is because there is a regression that needs
addressing - either in the test or in the proposed code change. So I
would be grateful if others were also looking.

I am not trying to re-invent the wheel and will not be surprised if my
'test fix' is not done in the 'Python' way. I'll learn over time - but
this calls for instructive (and critical) comments. "bij voorbaat dank"
aka Thanks in Advance.

So, hoping this helps - I'll continue as I can. Time and resources are
limited. And, I am very curious re: point c) above.

Great Days! everyone,

Michael




signature.asc
Description: OpenPGP digital signature
___
Python-Dev mailing list
[email protected]
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com


Re: [Python-Dev] [Python-checkins] bpo-34171: Prevent creating Lib/trace.cover when run the trace module. (GH-8841)

2018-08-27 Thread Victor Stinner
Jemery: would you mind to revert this commit since nobody fixed the
buildbot since 2 days?
https://pythondev.readthedocs.io/ci.html#revert-on-fail

Victor
Le sam. 25 août 2018 à 22:50, Jeremy Kloth  a écrit :
>
> On Sat, Aug 25, 2018 at 1:28 AM Serhiy Storchaka
>  wrote:
> >
> > https://github.com/python/cpython/commit/c406d5cd74002964a64c3eb7d9e2445a7fd3a03f
> > commit: c406d5cd74002964a64c3eb7d9e2445a7fd3a03f
> > branch: master
> > author: Serhiy Storchaka 
> > committer: GitHub 
> > date: 2018-08-25T10:27:55+03:00
> > summary:
> >
> > bpo-34171: Prevent creating Lib/trace.cover when run the trace module. 
> > (GH-8841)
> >
> > files:
> > A Misc/NEWS.d/next/Library/2018-08-21-00-29-01.bpo-34171.6LkWav.rst
> > M Lib/test/test_trace.py
> > M Lib/trace.py
>
> This change seems to have caused most buildbots to go red.
> ___
> Python-Dev mailing list
> [email protected]
> https://mail.python.org/mailman/listinfo/python-dev
> Unsubscribe: 
> https://mail.python.org/mailman/options/python-dev/vstinner%40redhat.com
___
Python-Dev mailing list
[email protected]
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com


Re: [Python-Dev] [Python-checkins] bpo-34171: Prevent creating Lib/trace.cover when run the trace module. (GH-8841)

2018-08-27 Thread Jeremy Kloth
On Mon, Aug 27, 2018 at 7:26 AM Victor Stinner  wrote:
>
> Jemery: would you mind to revert this commit since nobody fixed the
> buildbot since 2 days?
> https://pythondev.readthedocs.io/ci.html#revert-on-fail

I think you meant Serhiy :)  Anyway, a commit has finally landed that
addresses the buildbot failures, at least for master.

-- 
Jeremy Kloth
___
Python-Dev mailing list
[email protected]
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com


[Python-Dev] Review of Pull Request #2078

2018-08-27 Thread Pierre Quentel
 Hi,

A few months ago I have submitted the Pull Request #2078 named "bpo-30576 :
Add HTTP compression support to http.server".

After a couple of iterations, discussions on the PR page (
https://github.com/python/cpython/pull/2078) and on the bug tracker (
https://bugs.python.org/issue30576), it seems to me that there are no
objection to adding this feature (the Github page even has recent messages
favorable to its inclusion) and that the implementation is not a complete
mess ;-)

It has been in the "waiting core review" state for many months now. As
indicated on the contributing page (
https://github.com/python/cpython/blob/master/.github/CONTRIBUTING.rst) I
send this message to python-dev to see if one of the core developers could
take a look at the Pull Request.

Thanks for your time,
Pierre
___
Python-Dev mailing list
[email protected]
https://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com