Skip to content
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

[3.7] bpo-36670: Multiple regrtest bugfixes (GH-16511) #16514

Merged
merged 1 commit into from
Oct 1, 2019

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 1, 2019

  • Windows: Fix counter name in WindowsLoadTracker. Counter names are
    localized: use the registry to get the counter name. Original
    change written by Lorenz Mende.
  • Regrtest.main() now ensures that the Windows load tracker is also
    killed if an exception is raised
  • TestWorkerProcess now ensures that worker processes are no longer
    running before exiting: kill also worker processes when an
    exception is raised.
  • Enhance regrtest messages and warnings: include test name,
    duration, add a worker identifier, etc.
  • Rename MultiprocessRunner to TestWorkerProcess
  • Use print_warning() to display warnings.

Co-Authored-By: Lorenz Mende Lorenz.mende@gmail.com
(cherry picked from commit 982bfa4)

Co-authored-by: Victor Stinner vstinner@redhat.com

https://bugs.python.org/issue36670

* Windows: Fix counter name in WindowsLoadTracker. Counter names are
  localized: use the registry to get the counter name. Original
  change written by Lorenz Mende.
* Regrtest.main() now ensures that the Windows load tracker is also
  killed if an exception is raised
* TestWorkerProcess now ensures that worker processes are no longer
  running before exiting: kill also worker processes when an
  exception is raised.
* Enhance regrtest messages and warnings: include test name,
  duration, add a worker identifier, etc.
* Rename MultiprocessRunner to TestWorkerProcess
* Use print_warning() to display warnings.

Co-Authored-By: Lorenz Mende <Lorenz.mende@gmail.com>
(cherry picked from commit 982bfa4)

Co-authored-by: Victor Stinner <vstinner@redhat.com>
Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington
Copy link
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit d6a92b5 into python:3.7 Oct 1, 2019
@miss-islington miss-islington deleted the backport-982bfa4-3.7 branch October 1, 2019 10:58
@miss-islington
Copy link
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants