Change Details

This will allow to list the foss.heptapod.net instance for example. Related to T3581 gitlab: Allow ingestion of hg_git origins as hg ones Related to T3581#70593 Allow gitlab lister's name to be overriden by task arguments This will allow to dedicate the heptapod instances into its their own stats. Related to T3581 opam: Move the state initialization into the get_pages method We should avoid side-effects in the constructor as much as possible. That avoids surprising behavior at object instantiation time. The state if needed must be initialized into the `swh.lister.pattern.Lister.get_pages` method, as preconized in the class docstring. This also fixes the current test that actually bootstrap a real opam local "clone" in /tmp. Related to T3590 opam: Make the instance optional and derived from the url This matches how it's done for all other multi instances listers. Related to T3590 opam: Allow defining where to actually install the opam_root folder Related to T3590 gnu: Respect the pattern docstring about state initialization Any extra state initialization (outside the scheduler scope) is to happen in the get_pages method. opam: Share opam root directory even on multiple instances That avoids having multiple distinct opam root directories per opam lister instance. The current opam commands used by the lister are actually listing specifically per instance. Related to P1171 maven-lister: Add maven lister with review D6133.

Would you mind deleting this revision? This is another (failed) try at cleaning up the commits after review D6133. The new revision https://forge.softwareheritage.org/D6395 seems to be successful, though. Thanks for your patience.

This will allow to list the foss.heptapod.net instance for example. Related to T3581 gitlab: Allow ingestion of hg_git origins as hg ones Related to T3581#70593 Allow gitlab lister's name to be overriden by task arguments This will allow to dedicate the heptapod instances into its their own stats. Related to T3581 opam: Move the state initialization into the get_pages method We should avoid side-effects in the constructor as much as possible.Would you mind deleting this revision? This is another (failed) try at cleaning up the commits after review D6133. The new revision https://forge.softwareheritage.org/D6395 seems to be successful, That avoidsthough. surprising behavior at object instantiation time. The state if needed must be initialized into the `swh.lister.pattern.Lister.get_pages` method, as preconized in the class docstring. This also fixes the current test that actually bootstrap a real opam local "clone" in /tmp. Related to T3590 opam: Make the instance optional and derived from the url This matches how it's done for all other multi instances listers. Related to T3590 opam: Allow defining where to actually install the opam_root folder Related to T3590 gnu: Respect the pattern docstring about state initialization Any extra state initialization (outside the scheduler scope) is to happen in the get_pages method. opam: Share opam root directory even on multiple instances That avoids having multiple distinct opam root directories per opam lister instance. The current opam commands used by the lister are actually listing specifically per instance. Related to P1171 maven-lister: Add maven lister with review D6133Thanks for your patience.