Build is green

Build has FAILED

Build is green

rebase

Fix revision -> release typo in release_add flush call

lgtm, providing the typo is fixed ;)

And i assume they were helpful ;)

Build is green

Build has FAILED

Diff for log routes

WIP: Many URLs defined in the origin view (urlconf) are redundant.
This fix redirects /origin/branches?origin_url=<url> and
/origin/<url>/branches to /snapshot-branches/

Build is green

It still appears in sentry.
Although it seems specific to staging (which is a hint of a divergence of some sort).

Better docstrings and kill a few map()

Build is green

Build is green

Rebase

Fix typo

Build is green

Build is green

Rebase

Adapt according to suggestion

In T3625#71799, @olasd wrote:

While we're at it, we should probably be adding some thresholds in the buffer proxy for:

• cumulated length of messages for revisions and releases

Since i don't really see how to check that, I was in a mood to just mark it as
resolve in sentry until sentry shout "regression" (or not hopefully)

Thanks.

Thx

be a bit more consistent...

In D6410#167121, @douardda wrote:

In D6410#166398, @ardumont wrote:

as @vlorentz pointed out [1], this change should be irrelevant though...

[1] https://github.com/celery/kombu/blob/master/kombu/serialization.py#L369-L372

does not seem to be the proper fix.

FTR, using the celery cli tool directly from a development venv to interact with the celery server running in the docker compose test setup (as described there ) used to work ok, but not any more.

One have to specify the app, like:

celery --app=swh.scheduler.celery_backend.config.app status

[edit] I use celery 5.1.2 in my venv.

Unless I'm mistaken, this error does not appear in sentry any more, right?

In D6410#166398, @ardumont wrote:

as @vlorentz pointed out [1], this change should be irrelevant though...

[1] https://github.com/celery/kombu/blob/master/kombu/serialization.py#L369-L372

looks fine to me

Ok but see my 2 (nitpicky) comments

LGTM thx

[FTR] See https://forge.softwareheritage.org/source/swh-perfecthash/

Build is green

Build was aborted

(this task is a dependency of T3638, because author != committer in revisions created by opam, and releases don't have a "committer" field, so switching to releases would lose this data)

I'm curious to know where the libcmph-dev dependency was added?

rSENV646f62805ef564bceed4d3a4d84d8fb6890f2d19 declares the new certificate for the vagrant tests (wrong task on the commit message)

I'll split off the new buffer thresholds in a new diff. This diff now only contains the (small) improvements to the buffer/filter proxies

Build is green