host_metrics date fix to make summary dates (datetime.datetime) compa…

…rable to month: datetime.date (#15704) * host_metrics date fix * AAP-36839 Remove excess comments * fix extra date() conversion * actual fix * datetime is a library, use datetime.datetime --------- Co-authored-by: Andrea Restle-Lay <[email protected]>
ansible · Dec 16, 2024 · 86d2024 · 86d2024
1 parent c1f0a83
commit 86d2024
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/awx/main/tasks/host_metrics.py b/awx/main/tasks/host_metrics.py
@@ -221,14 +221,16 @@ def _find_or_create_summary(self, month):
             self.records_to_update.append(summary)
         return summary
 
-    def _find_summary(self, month):
+    def _find_summary(self, month: datetime.date):
         """
         Existing summaries are ordered by month ASC.
         This method is called with month in ascending order too => only 1 traversing is enough
         """
         summary = None
         while not summary and self.existing_summaries_idx < self.existing_summaries_cnt:
             tmp = self.existing_summaries[self.existing_summaries_idx]
+            if isinstance(tmp, datetime.datetime):
+                tmp = tmp.date()
             if tmp.date < month:
                 self.existing_summaries_idx += 1
             elif tmp.date == month: