Reminder: Please elaborate commit messages
Hi, In https://bugs.python.org/issue33531, Andrew Svetlov wrote "Fixed failed sendfile tests on Windows (at least I hope so)." without giving any bpo number or a commit number. So I looked at latest commits and I found: --- commit e2537521916c5bf88fcf54d4654ff1bcd332be4a Author: Andrew Svetlov <andrew.svetlov@gmail.com> Date: Mon May 21 12:03:45 2018 +0300 Fix asyncio flaky tests (#7023) --- Please try to write better error messages for people who will dig into the Git history in 1, 5 or 10 years: * Usually, it's better to open a bug. Here you could give the full error message, mention on which platform the test fails, etc. * Mention which tests are affected * Maybe even give an extract of the error message of the fixed test in the commit message I know that it's more effort and fixing flaky tests is annoying and may require multiple iterations, but again, please think to people who will have to read the Git history later... I was able able to rebuild the context of this commit from a comment of https://bugs.python.org/issue33531 Victor
Sorry for that. I thought that the bpo issue can be skipped because it is tests-only change, no asyncio code was affected. Will be more accurate next time. On Tue, May 22, 2018 at 3:26 PM Victor Stinner <vstinner@redhat.com> wrote:
Hi,
In https://bugs.python.org/issue33531, Andrew Svetlov wrote "Fixed failed sendfile tests on Windows (at least I hope so)." without giving any bpo number or a commit number. So I looked at latest commits and I found:
--- commit e2537521916c5bf88fcf54d4654ff1bcd332be4a Author: Andrew Svetlov <andrew.svetlov@gmail.com> Date: Mon May 21 12:03:45 2018 +0300
Fix asyncio flaky tests (#7023) ---
Please try to write better error messages for people who will dig into the Git history in 1, 5 or 10 years:
* Usually, it's better to open a bug. Here you could give the full error message, mention on which platform the test fails, etc. * Mention which tests are affected * Maybe even give an extract of the error message of the fixed test in the commit message
I know that it's more effort and fixing flaky tests is annoying and may require multiple iterations, but again, please think to people who will have to read the Git history later...
I was able able to rebuild the context of this commit from a comment of https://bugs.python.org/issue33531
Victor
-- Thanks, Andrew Svetlov
Usually, I don't open a new bug to fix or enhance a test. So I wouldn't say that it's mandatory. It's really on a case by case basis. It seems like test_asyncio failures are a hot topic these days :-) It's one of the reasons why Python 3.7rc1 has been delayed by 2 days, no? :-) Victor 2018-05-22 14:37 GMT+02:00 Andrew Svetlov <andrew.svetlov@gmail.com>:
Sorry for that. I thought that the bpo issue can be skipped because it is tests-only change, no asyncio code was affected. Will be more accurate next time.
On Tue, May 22, 2018 at 3:26 PM Victor Stinner <vstinner@redhat.com> wrote:
Hi,
In https://bugs.python.org/issue33531, Andrew Svetlov wrote "Fixed failed sendfile tests on Windows (at least I hope so)." without giving any bpo number or a commit number. So I looked at latest commits and I found:
--- commit e2537521916c5bf88fcf54d4654ff1bcd332be4a Author: Andrew Svetlov <andrew.svetlov@gmail.com> Date: Mon May 21 12:03:45 2018 +0300
Fix asyncio flaky tests (#7023) ---
Please try to write better error messages for people who will dig into the Git history in 1, 5 or 10 years:
* Usually, it's better to open a bug. Here you could give the full error message, mention on which platform the test fails, etc. * Mention which tests are affected * Maybe even give an extract of the error message of the fixed test in the commit message
I know that it's more effort and fixing flaky tests is annoying and may require multiple iterations, but again, please think to people who will have to read the Git history later...
I was able able to rebuild the context of this commit from a comment of https://bugs.python.org/issue33531
Victor
-- Thanks, Andrew Svetlov
On 5/22/2018 8:37 AM, Andrew Svetlov wrote:
Sorry for that. I thought that the bpo issue can be skipped because it is tests-only change, no asyncio code was affected. Will be more accurate next time.
A new issue was not needed. Adding 'bpo-33531' would have been fine, automatically linking the issue and the PR in both directions. -- Terry Jan Reedy
On Tue, May 22, 2018 at 8:52 AM Victor Stinner <vstinner@redhat.com> wrote:
Usually, I don't open a new bug to fix or enhance a test. So I wouldn't say that it's mandatory. It's really on a case by case basis.
It seems like test_asyncio failures are a hot topic these days :-) It's one of the reasons why Python 3.7rc1 has been delayed by 2 days, no? :-)
Yes, getting Windows tests stable wasn't easy. I think it's solved now (thanks to Andrew), but we always welcome any help from other core devs :) Yury
On 5/22/2018 2:26 PM, Yury Selivanov wrote:
On Tue, May 22, 2018 at 8:52 AM Victor Stinner <vstinner@redhat.com> wrote:
Usually, I don't open a new bug to fix or enhance a test. So I wouldn't say that it's mandatory. It's really on a case by case basis.
It seems like test_asyncio failures are a hot topic these days :-) It's one of the reasons why Python 3.7rc1 has been delayed by 2 days, no? :-)
Yes, getting Windows tests stable wasn't easy. I think it's solved now (thanks to Andrew), but we always welcome any help from other core devs :)
The AppVeyor (Windows) failure have been gone for a day. The Travis failures are rarer. -- Terry Jan Reedy
participants (4)
-
Andrew Svetlov -
Terry Reedy -
Victor Stinner -
Yury Selivanov