fix the test (op. priority...)

add tests + rebase

Shouldn't there be a migration script as well?

ok but the wording of the mark_production decorator

I'm not against this, but I have my doubts on the overall "architecture".

Don't like the zz-celery hack, but well, that's life. Did not check the new test files actually correspond to old test_metadata.py content. If this only test being moved or are there other (eg new tests) coming along? If so, please specify it in the commit message.

In D8066#209725, @ardumont wrote:

lgtm but it's missing the test case around it. It should not be too hard to add a scenario which lifts the flavor parameter, shouldn't it?

In T2309#87779, @douardda wrote:

do you have in mind to make the actual hash used as primary key in an objstorage a configuration of said storage instance? e.g. create a pathslicer or s3 objstorage using sha256 is just a matter of configuration of the objstorage?

do you have in mind to make the actual hash used as primary key in an objstorage a configuration of said storage instance? e.g. create a pathslicer or s3 objstorage using sha256 is just a matter of configuration of the objstorage?

please explain a bit what this refactor is about in the commit message.

Also, this will "break the API". I am not opposed to this change, but some care should be taken when changing public's API behavior like this. (attempt to check if there are users of the behavior to be modified, document the new behavior, display/return a warning or something, etc)

In D7980#207563, @anlambert wrote:

Ack, I will make the change to the /api/1/origin/ endpoint to remove the URL processing then.

In D7980#207469, @anlambert wrote:

In D7980#207468, @jayeshv wrote:

In D7980#207464, @vlorentz wrote:

Why should this be implemented by the API (as opposed to UIs)?

This is part of our current REST APIs.

rebase

set timeout to 5s on both configs, as requested by anlambert

I've added the DescribeConfigs and AlterConfigs permissions for the mirror-test-rw principal, now I can do all I need.

typos in the ci msg

maybe only show a warning if the grant query fails (rather than crashing)?

lgtm (probably needs tedious hand-managed-db-migration-in-docker test)

rebase

actual fix

rebase

fix

rebase

remove TValue typevar as suggested by vlorentz

rebase and update copyrights

Extract the type annotation part of this diff in a dedicated one

Add type annotation in swh.journal.writer and rename the 'stream' arg as 'file_like'

rebase