Push Notifications

Open
#2 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
20/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
django, python
Domain
api, backend, devops

Research direction

The issue names no files, tests, or entry points. Start by reviewing the existing polling flow and Django's conditional view processing, then compare the proposed HTTP notification approaches; done means CI systems can receive timely patchwork updates or use a clearly defined efficient fallback.

Written by the indexing model from the issue text.

Description

enhancement

From @stephenfin on April 4, 2015 17:16

It should be possible to hook CI systems (like Jenkins) into patchwork. While the patchwork->CI element of this is currently possible with polling, this is inefficient. CI->patchwork communication, on the other hand, is not currently possible.

There are multiple ways to resolve the former of these issues. Gerrit does so by using an SSH service, but we could probably use a HTTP-based approach like Websockets.

If this is not possible, we should at least (a) minimise the amount of processing needed by the client and (b) cache as much as possible. The former can be solved through use of something like Activity Stream (via the django-notifications plugin). The latter can be resolved using ETags (provided by Django) or similar.

Copied from original issue: stephenfin/patchwork#13

Dominant language
Python
Stars
317
Forks
91
PR merge metrics
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from getpatchwork/patchwork

All issues in getpatchwork/patchwork

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.