Fix re-trigger bug with tags

[Patchew-devel] [PATCH 0/2] Fix re-trigger bug with tags

Fam Zheng posted 2 patches 5 years, 11 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/patchew-ci tags/patchew/20181116060948.25592-1-famz@redhat.com

api/models.py                                |   2 +-
mods/testing.py                              |  17 +++++----
tests/data/0025-foo-patch-review.mbox.gz     | Bin 0 -> 1417 bytes
tests/data/0026-bar-patch-standalone.mbox.gz | Bin 0 -> 1558 bytes
tests/data/0027-foo-patch-based-on.mbox.gz   | Bin 0 -> 1421 bytes
tests/patchewtest.py                         |   5 ++-
tests/test_testing.py                        |  38 +++++++++++++------
7 files changed, 41 insertions(+), 21 deletions(-)
create mode 100644 tests/data/0025-foo-patch-review.mbox.gz
create mode 100644 tests/data/0026-bar-patch-standalone.mbox.gz
create mode 100644 tests/data/0027-foo-patch-based-on.mbox.gz

Expand all Fold all

[Patchew-devel] [PATCH 0/2] Fix re-trigger bug with tags

Posted by Fam Zheng 5 years, 11 months ago

Reviewed-by tags etc. from replies now trigge retesting and in turn resending
the same error reportings (though on refreshed HEAD).

Fix this by adding more result update handler logic in the testing module.

Fam Zheng (2):
  models: Add project name to Message __str__
  testing: Fix bugs that new reviewed-by tags trigger retest

 api/models.py                                |   2 +-
 mods/testing.py                              |  17 +++++----
 tests/data/0025-foo-patch-review.mbox.gz     | Bin 0 -> 1417 bytes
 tests/data/0026-bar-patch-standalone.mbox.gz | Bin 0 -> 1558 bytes
 tests/data/0027-foo-patch-based-on.mbox.gz   | Bin 0 -> 1421 bytes
 tests/patchewtest.py                         |   5 ++-
 tests/test_testing.py                        |  38 +++++++++++++------
 7 files changed, 41 insertions(+), 21 deletions(-)
 create mode 100644 tests/data/0025-foo-patch-review.mbox.gz
 create mode 100644 tests/data/0026-bar-patch-standalone.mbox.gz
 create mode 100644 tests/data/0027-foo-patch-based-on.mbox.gz

-- 
2.17.2

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel

Re: [Patchew-devel] [PATCH 0/2] Fix re-trigger bug with tags

Posted by Paolo Bonzini 5 years, 11 months ago

On 16/11/18 07:09, Fam Zheng wrote:
> Reviewed-by tags etc. from replies now trigge retesting and in turn resending
> the same error reportings (though on refreshed HEAD).
> 
> Fix this by adding more result update handler logic in the testing module.
> 
> Fam Zheng (2):
>   models: Add project name to Message __str__
>   testing: Fix bugs that new reviewed-by tags trigger retest

Looks good, please apply!

Paolo

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel