Thanks!

lgtm

lotm (looks ok to me ;)

@cmatrix Thanks and congrats on your landed diff ;)

Couple of remarks/questions inline.

SWH migration to GitLab is incoming and we are not using elastic workers
yet in production so we still need to build debian packages for swh modules.

I guess this is the fix for the current master red build [1]

I don't know how to address the shortcomings nico mentioned.
Let's figure this out along the way.

one question inline (again ;)

one question inline.

one question inline.

(i had to look at the associated commits as the diff, since "it landed" only shows the first commit now).

Rebase

Fix

Adapt according to discussion

Thanks for looking at this (fwiw, this warning is pretty innocuous for now,
and is not what prevents the Debian build; the issue seems to be in pytest).

• Is it possible to also list this query I often use to check the status of the ingestion:

for example:
select last_visit_status, count(ovs.url) from origin_visit_stats ovs join listed_origins lo on lo.url = ovs.url and lo.visit_type = ovs.visit_type where lister_id='8d1ebad5-a2dc-482c-9b9b-188665274377' group by last_visit_status;

Adapt according to suggestion and rework query and query sample presentation

Rebase

Closed by 8e125f1

Adapt according to last irc discussion -> use --preset

Expose the new subcommand

Amend according to discussion and fix tests

• cli.add_forge_now: Open register-lister with sensible defaults
• cli.add_forge_now: Rework according to review

Fix

Fix

Adapt somewhat according to review (let jenkins builds stuff) [1]

Neat simplification indeed, congrats \m/.