classification
Title: test_start_new_session for posix_spawnp fails
Type: Stage: patch review
Components: Tests Versions: Python 3.8
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: nanjekyejoannah, pablogsal, vstinner
Priority: normal Keywords: patch

Created on 2019-02-01 10:53 by pablogsal, last changed 2019-02-01 11:17 by vstinner.

Pull Requests
URL Status Linked Edit
PR 11721 open vstinner, 2019-02-01 11:17
Messages (3)
msg334664 - (view) Author: Pablo Galindo Salgado (pablogsal) * (Python committer) Date: 2019-02-01 10:53
Example failure:

https://buildbot.python.org/all/#/builders/141/builds/1142/steps/4/logs/stdio

=====================================================================
ERROR: test_start_new_session (test.test_posix.TestPosixSpawn)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/srv/buildbot/buildarea/3.x.bolen-ubuntu/build/Lib/test/test_posix.py", line 1640, in test_start_new_session
    child_pgid = int(output)
NameError: name 'output' is not defined
======================================================================
ERROR: test_start_new_session (test.test_posix.TestPosixSpawnP)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/srv/buildbot/buildarea/3.x.bolen-ubuntu/build/Lib/test/test_posix.py", line 1640, in test_start_new_session
    child_pgid = int(output)
NameError: name 'output' is not defined
----------------------------------------------------------------------


Victor made a PR to ignore the test for now:

https://github.com/python/cpython/pull/11718

This WIP PR intends to fix this bug:

https://github.com/python/cpython/pull/11719
msg334666 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2019-02-01 10:56
I skipped the test with commit 1e39b83f24ffade3e0ca1a8004b461354f64ae08.

But the test still needs to be fixed.

Joannah Nanjekye wrote the new test and I'm mentoring her, so I would prefer that she fix the test.
msg334668 - (view) Author: STINNER Victor (vstinner) * (Python committer) Date: 2019-02-01 11:17
> Joannah Nanjekye wrote the new test and I'm mentoring her, so I would prefer that she fix the test.

Oh wait, when I looked at the test, it's way more complex to implement that what I expected! So I decided to rewrite the test myself.
History
Date User Action Args
2019-02-01 11:17:30vstinnersetmessages: + msg334668
2019-02-01 11:17:02vstinnersetkeywords: + patch
stage: patch review
pull_requests: + pull_request11597
2019-02-01 10:56:54vstinnersetnosy: + nanjekyejoannah, vstinner
messages: + msg334666
2019-02-01 10:53:19pablogsalcreate