comments

deploy/clowdapp.yaml

@@ -5642,10 +5642,10 @@ parameters:
 - displayName: Provider count for batch polling
   name: POLLING_BATCH_SIZE
   value: "100"
-- displayName: Timer to prevent triggering tasks while still processing
+- displayName: Timer to prevent triggering tasks while still processing in days
   name: PROCESSING_WAIT_TIMER
   value: "3"
-- displayName: Timer to prevent triggering tasks while still processing
+- displayName: Timer to prevent triggering tasks while still processing in days
   name: LARGE_PROCESSING_WAIT_TIMER
   value: "7"
 - displayName: Enable Tags Limit

deploy/kustomize/base/base.yaml

@@ -515,10 +515,10 @@ parameters:
 - displayName: Provider count for batch polling
   name: POLLING_BATCH_SIZE
   value: "100"
-- displayName: Timer to prevent triggering tasks while still processing
+- displayName: Timer to prevent triggering tasks while still processing in days
   name: PROCESSING_WAIT_TIMER
   value: "3"
-- displayName: Timer to prevent triggering tasks while still processing
+- displayName: Timer to prevent triggering tasks while still processing in days
   name: LARGE_PROCESSING_WAIT_TIMER
   value: "7"
 - displayName: Enable Tags Limit

koku/api/provider/models.py

@@ -92,8 +92,6 @@ def get_polling_batch(self, limit=-1, offset=0, filters=None):
         if limit < 1:
             # Django can't do negative indexing, so just return all the Providers.
             # A limit of 0 doesn't make sense either. That would just return an empty QuerySet.
-            # Adding filter for data_updated_timestamp, prevent triggering new tasks if one is still in progress
-            # Additional 7 day check here if processing fails and data_updated_timestamp is not updated
             return self.filter(**filters).exclude(polling_timestamp__gt=polling_delta)
         return self.filter(**filters).exclude(polling_timestamp__gt=polling_delta)[offset : limit + offset]