forgot the renaming of host->deposit_host

Update/fixes according vlorentz' comments

did only a quick review, lgtm. Note however my comment on using path.basename() instead of path.split()[-1]

In D2491#59106, @ardumont wrote:

that looks quite awesomly promising.

will that replace the current shell scripts?

oops forgot a commit

rebase

rebase

rebase + small fix in client/__init__.py

rebase

rebase

same as D2421

rebase

rebase

rebase

rebase

rebase

Forgot a revision in this diff

don't remember what I did change in here...

I am still not sure how to understand this.

I'm not completely sure I get this ticket; I understand that HAL may (always?) produce invalid zip files when it uploads a deposit in SWH. So my questions are:

LGTM

rebase

Nice, thx

sorry for the glitch

In D2427#57372, @ardumont wrote:

tests should pass ok however no test exists for the interaction with the
scheduler, so...

By adding the swh_scheduler fixture in one test (which adds a deposit), you could either:

• check the check_task_id, load_task_id are set

• introspect the swh_scheduler.search_task('check-deposit') tasks to ensure there is something [1]

[1] https://forge.softwareheritage.org/source/swh-lister/browse/master/swh/lister/cgit/tests/test_lister.py$53-55

add X-Forwarded-Host header and configure allowed_hosts as '*'

prometheus_client looks quite low-level to me. We already have statsd for this kind of purpose, which does not imply having to deal with one more dedicated http server or so...

Sorry I'm commenting this a bit late, but wouldn't it make more sense to have something like:

rebased

move statsd emv var declarations in an env/statsd.env file and use it on swh-web

In D2416#57093, @anlambert wrote:

Great ! Could you also add that configuration to the swh-web service ? I am currently working on T2124 and I have added a couple of statsd metric to the webapp.

rebase

rebase