diff --git a/swh/loader/core/loader.py b/swh/loader/core/loader.py --- a/swh/loader/core/loader.py +++ b/swh/loader/core/loader.py @@ -738,10 +738,6 @@ """ origin = self.origin.copy() - if 'id' in origin: - # TODO: remove the condition when we finished migrating away - # from origin ids - del origin['id'] self.send_origin(origin) if not self.visit_date: # now as default visit_date if not provided diff --git a/swh/loader/core/tests/test_loader.py b/swh/loader/core/tests/test_loader.py --- a/swh/loader/core/tests/test_loader.py +++ b/swh/loader/core/tests/test_loader.py @@ -35,7 +35,7 @@ self.origin_url = origin['url'] self.visit_date = datetime.datetime.utcnow() self.visit_type = 'git' - self.storage.origin_visit_add(origin['id'], self.visit_date, + self.storage.origin_visit_add(self.origin_url, self.visit_date, self.visit_type) def parse_config_file(self, *args, **kwargs):