-
-
Notifications
You must be signed in to change notification settings - Fork 31k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gh-102151: Correctly fetch CONFIG_ARGS in Tools/freeze/test/freeze.py #102152
Conversation
erlend-aasland
commented
Feb 22, 2023
•
edited by bedevere-bot
Loading
edited by bedevere-bot
- Issue: test_freeze_simple_script in test_tools fails to fetch CONFIG_ARGS #102151
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome LGTM!
Thanks for the review, Dong-hee! I'll wait a day or two for one of the freeze experts to chime in, just in case. |
Thanks @erlend-aasland for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11. |
GH-102176 is a backport of this pull request to the 3.11 branch. |
…eze.py (pythonGH-102152) (cherry picked from commit c3a1783) Co-authored-by: Erlend E. Aasland <[email protected]>
…GH-102152) (cherry picked from commit c3a1783) Co-authored-by: Erlend E. Aasland <[email protected]>
This commit has caused https://bugs.gentoo.org/900429 (and https://bugs.gentoo.org/903890 because of the backport to Python 3.11) but it's unclear to me why yet. It leads to a PGO build hanging in test_tools (or running very very slowly) until disk is filled and then it'll move on. |
From my understanding which I've reported in https://bugs.gentoo.org/900429#c19 this change didn't directly cause the test_tools to hang. It just fixed another error which was preventing test_tools from starting properly. Before this it quickly failed with:
while with this fix it shows different error and then continues for hours and hours (until killed or runs out of disk space or something):
1:44:19 load avg: 1.61 [427/427] test_tools failed (1 error) --- STDERR --- ---- END ---- --- STDERR --- ---- END ---- Total duration: 1 hour 44 min
|
Thanks Martin. I'm sorry for the noise - I think there is an issue here, but it might be on our end, and it wasn't introduced by this commit (it just exposed some pre-existing issue). |