fix: assign carpool_agency as default role

amarillo/services/agencyconf.py

@@ -30,7 +30,7 @@ def __init__(self):
 
                 # Default: if no role was defined, the default role is
                 # carpool_agency.
-                if agency_conf.roles is None:
+                if agency_conf.roles is None or len(agency_conf.roles) == 0:
                     agency_conf.roles = [AgencyRole.carpool_agency]
 
                 agency_id = agency_conf.agency_id

amarillo/services/sync.py

@@ -25,8 +25,9 @@ def __init__(self, store, agencyconf_service):
         self.agencyconf_service = agencyconf_service
 
     def perform_full_sync(self) -> None:
-        agencies = self.agencyconf_service.get_all_agencies()
-        for agency in [agency for agency in agencies if self.should_sync(agency)]:
+        agencies_to_sync = [agency for agency in self.agencyconf_service.get_all_agencies() if self.should_sync(agency)]
+        logger.info(f"Perform full sync for agencies {agencies_to_sync}")
+        for agency in agencies_to_sync:
             try:
                 asyncio.run(self.sync(agency.agency_id, agency.offers_download_url, agency.offers_download_http_headers))
             except Exception as e:
@@ -57,9 +58,12 @@ async def sync(self, agency_id: str, offers_download_url=None, offers_download_h
             importer = AmarilloImporter(agency_id, offers_download_url, offers_download_http_headers)
 
         sync_start_time = time.time()
+        logger.info(f"Start sync for agency {agency_id} at {sync_start_time}")
+
         carpools = importer.load_carpools()
 
         result = [await self.store.store_carpool(cp) for cp in carpools]
-
+        logger.info(f"Synced {len(result)} offers for agency {agency_id}")
+
         await self.store.delete_agency_carpools_older_than(agency_id, sync_start_time)
         return result