aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)Author
2016-07-19parsemail: Increase the lock timeout to a minuteDamien Lespiau
We do reach that timeout on production for large series. Give a bit more leeway. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-07-19parsemail: Limit the search for previous patches to the current projectDamien Lespiau
Patches are sometimes cross posted on mailing-lists handled by the same patchwork instance, so msgids aren't unique across projects anymore. This fixes: Traceback (most recent call last): File "patchwork/bin/parsemail.py", line 883, in main return parse_mail(mail) File "patchwork/bin/parsemail.py", line 761, in parse_mail content = find_content(project, mail) File "patchwork/bin/parsemail.py", line 423, in find_content ret.patch_order, n, refs) File "patchwork/bin/parsemail.py", line 533, in find_series_for_mail previous_patch = find_previous_patch(revision, order, refs) File "patchwork/bin/parsemail.py", line 468, in find_previous_patch patch = Patch.objects.get(msgid=ref) File "django/db/models/manager.py", line 127, in manager_method return getattr(self.get_queryset(), name)(*args, **kwargs) File "django/db/models/query.py", line 338, in get (self.model._meta.object_name, num) MultipleObjectsReturned: get() returned more than one Patch -- it returned 2! Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-07-06flake8: Ignore warning only leading to false positivesDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-07-06Merge pull request #189 from moto-timo/moto-timo-git-pw-fedoraDamien Lespiau
Fedora package is GitPython, not python-GitPython
2016-07-02Fedora package is GitPython, not python-GitPythonmoto-timo
2016-05-16api: Add filtering to /patches/Damien Lespiau
Just like in the series case, we'll use the filtering from the web page and git-pw. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-16docs: Fix unexpected indentation rst errorDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-16api: Add a per-project list of patches entry pointDamien Lespiau
To be consistent with what has been done for series, even it's not entirely pure. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-05api: Extract a PatchListMixinDamien Lespiau
We're going to have more than one list of patches (just like series). Extract common properties into a mixin. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-05api: Expose 'last_updated' for patchesDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-05patch: Add a last_updated fieldDamien Lespiau
Just like with series, it's handy to be able to query patches that have changed since the last time we looked. The migration is a bit complicated as we need to populate the initial value of that field, ensuring the invariant that this field cannot be NULL. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-05api: Put the permission classes on the SeriesListMixinDamien Lespiau
This field is also shared between the SeriesList viewsets. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-05api: Factor out the queryset on SeriesViewSetDamien Lespiau
SeriesListMixin already has that field, no need to override it. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03admin: Add SeriesRevision to the admin siteDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03revision: Allows the test_state to be blank as wellDamien Lespiau
We'll need that for admin support. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03tests: Make sure we don't have any pending migrationsDamien Lespiau
We've been caught a few times with pushing commits where the state of manage.py makemigrations wasn't clean and had pending migrations. Make sure it doesn't happen again. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03db: Fix test state orderDamien Lespiau
The migration was created with a different order of the TestState fields. Everything worked as intented, but makemigrations was yelling that we had a pending migration because of a different order in the choice field. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03docs: Fix errors found by an aspell passDamien Lespiau
Unfortunately we cannot apply patches from the getpatchwork branch, so I needed to redo that one. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03pwclient: Fix typosEric Engestrom
Signed-off-by: Eric Engestrom <eric@engestrom.ch> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03tests: Fix spelling mistakes and typosEric Engestrom
v2: Port to fdo's patchwork (Damien) Signed-off-by: Eric Engestrom <eric@engestrom.ch> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03templates: Fix spelling mistakes and typosEric Engestrom
Signed-off-by: Eric Engestrom <eric@engestrom.ch> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03sql: Fix spelling mistakes and typosEric Engestrom
Signed-off-by: Eric Engestrom <eric@engestrom.ch>
2016-05-03post-receive.hook: rename detectionJan Remmet
patches which descripes renamed files show different hashes with git show and pwclient. Use git show -C (Detect copies as well as renames.) Signed-off-by: Jan Remmet <j.remmet@phytec.de> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03ui: Redirect to '/' after logoutStephen Finucane
It's already obvious that the user has logged out, so don't bother displaying a message saying as much. Redirect to the home page instead. v2: Port to fdo's patchwork (Damien) Signed-off-by: Stephen Finucane <stephen.finucane@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03ui: Add user bundle link into the dropdown menuStephen Finucane
Since the link to TODOs (reviews pending) can already be seen there, it makes sense to link to bundles also. A divider is added to logically group these elements. v2: Port to fdo's patchwork (Damien) Signed-off-by: Stephen Finucane <stephen.finucane@intel.com> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03revisions: Handle updating the test state when deleting a resultDamien Lespiau
While we don't have an API to delete a test result at the moment, it's future proof to fix this straight away. That means we can also use it when tinkering with the db from manage.py shell for instance. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03testresult: Adopt a neutral marker for info emailsDamien Lespiau
We don't really want a big cross for info test results being sent out as emails, a circle seems neutral enough to me. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03git-pw: Add support for the new 'info' test stateDamien Lespiau
Nothing major, the most annoying thing was the argument parsing rejecting the 'info' state for filtering. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-05-03docs: Add 'info' to the list of valid test statesMichael Wood
Signed-off-by: Michael Wood <michael.g.wood@intel.com>
2016-05-03series-list: Add 'info' test result state frontend featureMichael Wood
Adds the front end parts to display a test which has a result as 'info'. Signed-off-by: Michael Wood <michael.g.wood@intel.com>
2016-05-03api: Add test for the 'info' test result stateMichael Wood
Signed-off-by: Michael Wood <michael.g.wood@intel.com>
2016-05-03models: Add 'info' state for test resultsMichael Wood
Sometimes tests return information which is neither a fail or success for example statistics or linters which are useful to manually review the output. Signed-off-by: Michael Wood <michael.g.wood@intel.com>
2016-04-29series: Add a new action to re-trigger a testing cycleDamien Lespiau
People are asking our CI team to re-trigger tests when something goes wrong (for instance when a test slave was online during the previous test run) or to double check that the results are stable. This action allows maintainers to re-trigger test runs by inserting series-new-revision events into the event stream. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-29js: Introduce a post_data() functionDamien Lespiau
Depending on what we want to do, we need POST and PATCH requests. Introduce a POST one, rewriting the patch_data() as a function of post_data() (we use a header to override the POST method to a PATCH). Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-29js: Rename post_data() to patch_data()Damien Lespiau
We're doing a PATCH request, not a POST one! Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-29js: Also give the revision to do_action()Damien Lespiau
For some API calls, we not only need the series id but also the latest revision. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-29js: Factor out a function to retrieve the column data by nameDamien Lespiau
From a checkbox (that's the element we receive the event from), it's handy to be able to retrieve the row data. That function does it, given the column header we want. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-29js: Encode the order in the columns mapDamien Lespiau
It's useful to have more metadata about the table columns in one place so we can query various things about them. This time, I need the rank of the column so I can query data from the name of column header. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-28series: Add a hidden version columnDamien Lespiau
We'll need the version for API calls on the correct revision, so store it in the table. We'll also be able, later, to let people select which column to show/hide to personalize which date one would like to see. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-28js: Factor out the code that run the actionDamien Lespiau
We're going to have another type of action that doesn't use a menu/form, but we want to reuse that code. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-28api: Add a newrevision method on seriesDamien Lespiau
People would like to be able to re-trigger testing through the web interface. Create a new series-new-revision event should be enough to trick the testing systems to re-trigger a test run. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-28docs: Use --name to only select series-new-revision eventsDamien Lespiau
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-28Merge pull request #186 from llandwerlin-intel/masterDamien Lespiau
git-pw: fix issue with python-requests
2016-04-28git-pw: fix issue with python-requestsLionel Landwerlin
2016-04-27Merge pull request #184 from avagin/git-pw-stderrDamien Lespiau
git-pw: print error messages on stderr
2016-04-27Merge pull request #185 from michaelgwood/masterDamien Lespiau
docs: Update default fixtures list s/default_actions/default_events
2016-04-26git-pw: print error messages on stderrAndrei Vagin
$ git pw list -s `date +%F -d '-1 day'` --json No series found! Non-json message is unexpected in this case.
2016-04-26docs: Update default fixtures list s/default_actions/default_eventsMichael Wood
Signed-off-by: Michael Wood <michael.g.wood@intel.com>
2016-04-13git-pw: Recurse with newer versions of GitPythonDamien Lespiau
Ander noticed we didn't actually recurse, and that commit 74039877a755711bea370e200984acbb2334b07c Author: Damien Lespiau <damien.lespiau@intel.com> Date: Wed Nov 11 12:42:05 2015 +0000 git-pw: Make git-pw work with older versions of GitPython was too hopeful in its expectations (I swear it was true at the time? but may have been fooled). v2: Use git.__version__ to decide when to include the search_parent_directories named argument. Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
2016-04-05git-pw: Add link optionsDamien Lespiau
We can now add Patchwork links to commit messages! $ git am -s -l $series_id $ git am -s -l --link-name Link $series_id Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> Fixes: https://github.com/dlespiau/patchwork/issues/176