[Patchew-devel] [RFC PATCH 00/12] Place results in the database

Paolo Bonzini posted 12 patches 5 years, 10 months ago
Failed in applying to current master (apply log)
api/blobs.py                                  |  38 +++
api/migrations/0027_auto_20180521_0152.py     |  38 +++
api/migrations/0028_populate_git_results.py   | 100 ++++++++
.../0029_populate_testing_results.py          | 205 ++++++++++++++++
api/migrations/0030_deblob_properties.py      |  34 +++
api/migrations/0031_auto_20180520_1654.py     |  34 +++
api/migrations/__init__.py                    |  47 ++++
api/models.py                                 | 174 +++++++-------
api/rest.py                                   |  75 +++---
mods/git.py                                   | 145 ++++++------
mods/testing.py                               | 220 ++++++++----------
tests/test_git.py                             |  34 +--
tests/test_testing.py                         |  63 +++--
www/views.py                                  |  17 +-
14 files changed, 846 insertions(+), 378 deletions(-)
create mode 100644 api/blobs.py
create mode 100644 api/migrations/0027_auto_20180521_0152.py
create mode 100644 api/migrations/0028_populate_git_results.py
create mode 100644 api/migrations/0029_populate_testing_results.py
create mode 100644 api/migrations/0030_deblob_properties.py
create mode 100644 api/migrations/0031_auto_20180520_1654.py
[Patchew-devel] [RFC PATCH 00/12] Place results in the database
Posted by Paolo Bonzini 5 years, 10 months ago
This seems daunting, but really most of the new code is in migrations
(by definition those cannot remove code ever!).  With those removed,
the net difference is slightly negative, and there are some nice
simplifications that the series makes too.

The migrations probably will take quite some time to run on the
patchew.org database.  For now I've only tested it (both
forwards and backwards) on the small sample that is downloaded by
scripts/developer-setup.

Paolo

Paolo Bonzini (12):
  testing: remove dead code
  testing: unify two ifs with the same condition
  testing: include "pending" testing results in REST API
  models: move Result log_url from constructor to renderer
  models: extract blobs functions to a separate file
  models: create Result model
  git: switch to Result model
  testing: track changes to the set of tests
  testing: switch to Result model
  complete switch to database-based Results
  do not load log when accessing api_result
  models: convert MessageProperty and ProjectProperty to JSONField

 api/blobs.py                                  |  38 +++
 api/migrations/0027_auto_20180521_0152.py     |  38 +++
 api/migrations/0028_populate_git_results.py   | 100 ++++++++
 .../0029_populate_testing_results.py          | 205 ++++++++++++++++
 api/migrations/0030_deblob_properties.py      |  34 +++
 api/migrations/0031_auto_20180520_1654.py     |  34 +++
 api/migrations/__init__.py                    |  47 ++++
 api/models.py                                 | 174 +++++++-------
 api/rest.py                                   |  75 +++---
 mods/git.py                                   | 145 ++++++------
 mods/testing.py                               | 220 ++++++++----------
 tests/test_git.py                             |  34 +--
 tests/test_testing.py                         |  63 +++--
 www/views.py                                  |  17 +-
 14 files changed, 846 insertions(+), 378 deletions(-)
 create mode 100644 api/blobs.py
 create mode 100644 api/migrations/0027_auto_20180521_0152.py
 create mode 100644 api/migrations/0028_populate_git_results.py
 create mode 100644 api/migrations/0029_populate_testing_results.py
 create mode 100644 api/migrations/0030_deblob_properties.py
 create mode 100644 api/migrations/0031_auto_20180520_1654.py

-- 
2.17.0

_______________________________________________
Patchew-devel mailing list
Patchew-devel@redhat.com
https://www.redhat.com/mailman/listinfo/patchew-devel
Re: [Patchew-devel] [RFC PATCH 00/12] Place results in the database
Posted by Fam Zheng 5 years, 10 months ago
On Tue, 05/22 08:57, Paolo Bonzini wrote:
> This seems daunting, but really most of the new code is in migrations
> (by definition those cannot remove code ever!).  With those removed,
> the net difference is slightly negative, and there are some nice
> simplifications that the series makes too.
> 
> The migrations probably will take quite some time to run on the
> patchew.org database.  For now I've only tested it (both
> forwards and backwards) on the small sample that is downloaded by
> scripts/developer-setup.

This is significant! Thanks! I've reviewed the first half (1-6) and will resume
later this week (unless you want to post v2. If I'm not missing anything, you
can drop RFC. :)

Fam

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