Differential D1940 Diff 6559 swh/lister/npm/__init__.py

Changeset View

Standalone View

swh/lister/npm/init.py

	# Copyright (C) 2019 the Software Heritage developers			# Copyright (C) 2019 the Software Heritage developers
	# License: GNU General Public License version 3, or any later version			# License: GNU General Public License version 3, or any later version
	# See top-level LICENSE file for more information			# See top-level LICENSE file for more information


	def register():			def register():
	from .models import NpmVisitModel, NpmModel			from .models import NpmVisitModel, NpmModel
	from .lister import NpmLister			from .lister import NpmLister

	return {'models': [NpmVisitModel, NpmModel],			return {'models': [NpmVisitModel, NpmModel],
	'lister': NpmLister,			'lister': NpmLister,
	'task_modules': ['%s.tasks' % __name__],			'task_modules': ['%s.tasks' % __name__],
				'task_types': {
				'list-npm-full': {
				'default_interval': '7 days',
				'min_interval': '7 days',
				'max_interval': '7 days',
				},
				vlorentzUnsubmitted Not Done Inline Actions why a specific value for npm? There should be at least a comment about this. vlorentz: why a specific value for npm? There should be at least a comment about this.
				ardumontUnsubmitted Not Done Inline Actions Probably because that matches the current production setup. type \| description \| backend_name \| default_interval \| min_interval \| max_interval \| backoff_factor \| max_queue_length$ -----------------------------------+--------------------------------------------------------------+-------------------------------------------------------+------------------+--------------+--------------+----------------+-----------------$ list-npm-full \| Full npm lister \| swh.lister.npm.tasks.NpmListerTask \| 7 days \| 7 days \| 7 days \| 1 \| $ ardumont: Probably because that matches the current production setup. ``` type…
				vlorentzUnsubmitted Not Done Inline Actions And why was it done in production? vlorentz: And why was it done in production?
				ardumontUnsubmitted Not Done Inline Actions I don't really know (i must have kept the existing ones which we now find somehow consistently in the scheduling db). Same goes for the existing other setup, why 90 (list-full), 64 (loaders), 1 (list-incremental) day(s)? That probably sounded reasonable at some point in time. ardumont: I don't really know (i must have kept the existing ones which we now find somehow consistently…
				douarddaAuthorUnsubmitted Done Inline Actions Probably because that matches the current production setup. yep! douardda: > Probably because that matches the current production setup. yep!
				},
	}			}