GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — test_import fails on Refleak buildbots: assertion failed · Issue #119618 · python/cpython

test_import fails on Refleak buildbots: assertion failed #119618

New issue
Copy link
New issue
Copy link
Closed
Closed
test_import fails on Refleak buildbots: assertion failed#119618
Copy link
Assignees
ericsnowcurrently
Labels
testsTests in the Lib/test dirTests in the Lib/test dir

Description

@vstinner
vstinner
opened on May 27, 2024
Issue body actions

Example:

vstinner@mona$ ./python -m test test_import -R 3:3 -m test_check_state_first -v
(...)
test_check_state_first (test.test_import.SinglephaseInitTests.test_check_state_first) ...

python: ./Modules/_testsinglephase.c:714: PyInit__testsinglephase_with_reinit_check_cache_first: Assertion `_testsinglephase_with_reinit_check_cache_first.m_base.m_index == 0' failed.
Fatal Python error: Aborted

Current thread 0x00007f56b95a6740 (most recent call first):
  File "<frozen importlib._bootstrap>", line 488 in _call_with_frames_removed
  File "<frozen importlib._bootstrap_external>", line 1316 in create_module
  File "<frozen importlib._bootstrap>", line 813 in module_from_spec
  File "<frozen importlib._bootstrap>", line 921 in _load_unlocked
  File "<frozen importlib._bootstrap>", line 966 in _load
  File "/home/vstinner/python/main/Lib/test/test_import/__init__.py", line 2495 in _load_dynamic
  File "/home/vstinner/python/main/Lib/test/test_import/__init__.py", line 2894 in test_check_state_first
(...)

Regression: commit 0c5ebe1

commit 0c5ebe13e9937c446e9947c44f2570737ecca135
Author: Eric Snow <[email protected]>
Date:   Sat May 25 15:30:48 2024 -0400

    gh-119560: Drop an Invalid Assert in PyState_FindModule() (gh-119561)
    
    The assertion was added in gh-118532 but was based on the invalid assumption that PyState_FindModule() would only be called with an already-initialized module def.  I've added a test to make sure we don't make that assumption again.

@ericsnowcurrently: Can you please have a look?

Activity

  1. vstinner
    added
    testsTests in the Lib/test dirTests in the Lib/test dir
    on May 27, 2024
  2. ericsnowcurrently commented on May 27, 2024

    @ericsnowcurrently
    ericsnowcurrently
    on May 27, 2024
    Member
    More actions

    Yeah, I'll sort this out ASAP.

  3. AlexWaygood commented on May 27, 2024

    @AlexWaygood
    AlexWaygood
    on May 27, 2024
    Member
    More actions

    Duplicate of #119584, I think?

  4. ericsnowcurrently
    self-assigned this
    on May 27, 2024
  5. ericsnowcurrently commented on May 27, 2024

    @ericsnowcurrently
    ericsnowcurrently
    on May 27, 2024
    Member
    More actions

    Yeah, this has the same root cause.

  6. ericsnowcurrently commented on May 27, 2024

    @ericsnowcurrently
    ericsnowcurrently
    on May 27, 2024
    Member
    More actions

    I've posted a fix: gh-119623.

    Reacted by Victor Stinner
  7. ericsnowcurrently
    closed this as completedon May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @ericsnowcurrently
    ericsnowcurrently

Labels

testsTests in the Lib/test dirTests in the Lib/test dir

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.