◐ Shell
reader mode source ↗
Skip to content

bpo-45052: Unskips a failing test_shared_memory_basics test#28182

Merged
zooba merged 1 commit into
python:mainfrom
sobolevn:issue-45052-1
Sep 6, 2021
Merged

bpo-45052: Unskips a failing test_shared_memory_basics test#28182
zooba merged 1 commit into
python:mainfrom
sobolevn:issue-45052-1

Conversation

@sobolevn

@sobolevn sobolevn commented Sep 6, 2021 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

@zooba zooba merged commit 19871fc into python:main Sep 6, 2021
@miss-islington

miss-islington commented Sep 6, 2021

Copy link
Copy Markdown
Contributor

Thanks @sobolevn for the PR, and @zooba for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 6, 2021
@sobolevn @miss-islington
bpo-45052: Unskips a failing test_shared_memory_basics test (python… …
056cf6c 
…GH-28182)

(cherry picked from commit 19871fc)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@miss-islington

miss-islington commented Sep 6, 2021

Copy link
Copy Markdown
Contributor

Sorry, @sobolevn and @zooba, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 19871fce3b74fc3f37e334a999e00d0ef65a8f1e 3.9

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Sep 6, 2021
@bedevere-bot

bedevere-bot commented Sep 6, 2021

Copy link
Copy Markdown

GH-28185 is a backport of this pull request to the 3.10 branch.

@zooba

zooba commented Sep 6, 2021

Copy link
Copy Markdown
Member

This change doesn't seem to apply to 3.9

miss-islington added a commit that referenced this pull request Sep 6, 2021
@miss-islington @sobolevn
bpo-45052: Unskips a failing test_shared_memory_basics test (GH-28182) …
6b5aea2 
(cherry picked from commit 19871fc)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
@pablogsal

pablogsal commented Sep 7, 2021 •
edited
Loading

Copy link
Copy Markdown
Member

This seem to have caused several errors that have manifested on the release of 3.10.0rc2:

test test_multiprocessing_fork failed -- Traceback (most recent call last):
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1239, in _dot_lookup
    return getattr(thing, comp)
AttributeError: module 'multiprocessing' has no attribute 'shared_memory'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/test/_test_multiprocessing.py", line 3818, in test_shared_memory_basics
    with unittest.mock.patch(
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1422, in __enter__
    self.target = self.getter()
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1609, in <lambda>
    getter = lambda: _importer(target)
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1252, in _importer
    thing = _dot_lookup(thing, comp, import_path)
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1242, in _dot_lookup
    return getattr(thing, comp)
AttributeError: module 'multiprocessing' has no attribute 'shared_memory'

0:09:11 load avg: 0.71 [231/427/1] test_multiprocessing_forkserver -- test_multiprocessing_fork failed (1 error) in 1 min 11 sec
test test_multiprocessing_forkserver failed -- Traceback (most recent call last):
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1239, in _dot_lookup
    return getattr(thing, comp)
AttributeError: module 'multiprocessing' has no attribute 'shared_memory'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/test/_test_multiprocessing.py", line 3818, in test_shared_memory_basics
    with unittest.mock.patch(
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1422, in __enter__
    self.target = self.getter()
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1609, in <lambda>
    getter = lambda: _importer(target)
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1252, in _importer
    thing = _dot_lookup(thing, comp, import_path)
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1242, in _dot_lookup
    return getattr(thing, comp)
AttributeError: module 'multiprocessing' has no attribute 'shared_memory'

0:11:11 load avg: 0.93 [232/427/2] test_multiprocessing_main_handling -- test_multiprocessing_forkserver failed (1 error) in 2 min
0:11:18 load avg: 1.09 [233/427/2] test_multiprocessing_spawn
test test_multiprocessing_spawn failed -- Traceback (most recent call last):
  File "/tmp/tmpu30qfjpr/installation/lib/python3.10/unittest/mock.py", line 1239, in _dot_lookup
    return getattr(thing, comp)
AttributeError: module 'multiprocessing' has no attribute 'shared_memory'

@sobolevn

sobolevn commented Sep 7, 2021

Copy link
Copy Markdown
Member Author

@pablogsal is there a bpo entry for this bug? This test can contain additional problems, this is not a surprise 😞

@pablogsal

pablogsal commented Sep 7, 2021

Copy link
Copy Markdown
Member

@pablogsal is there a bpo entry for this bug? This test can contain additional problems, this is not a surprise

Not for now, I am trying to debug it, but this is literally blocking the rc2 release of Python 3.10

@sobolevn

sobolevn commented Sep 7, 2021

Copy link
Copy Markdown
Member Author

I am sorry! 🙏
Too bad that all checks have passed both for main and 3.10

The easiest (and the fastest) thing is to ignore this test back, but on all platforms, not just windows.
It was ignored for several years (~2), before my change. I will fix it later.

Does this solution seem ok?

@sobolevn sobolevn deleted the issue-45052-1 branch September 7, 2021 14:15
@pablogsal

pablogsal commented Sep 7, 2021

Copy link
Copy Markdown
Member

Does this solution seem ok?

Hummm, it has to be something extra because this is failing currently on Linux. I still prefer to investigate.

I am sorry!

Don't worry, is not your fault :) Nobody knows what's happening, I just know that I cannot release the new rc with these problems.

@pablogsal

pablogsal commented Sep 7, 2021 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Maybe this is actually due to bpo-45042, who knows

@pablogsal

pablogsal commented Sep 7, 2021

Copy link
Copy Markdown
Member

Opened https://bugs.python.org/issue45128

This was referenced Sep 13, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@zooba zooba

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@sobolevn @miss-islington @bedevere-bot @zooba @pablogsal @the-knights-who-say-ni