host_metrics date fix to make summary dates (datetime.datetime) comparable 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 <arestlel@arestlel-thinkpadx1carbongen9.rht.csb>
This commit is contained in:
Andrea Restle-Lay 2024-12-16 12:10:59 -05:00 committed by GitHub
parent c1f0a831ff
commit 86d202456a
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -221,7 +221,7 @@ class HostMetricSummaryMonthlyTask:
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
@ -229,6 +229,8 @@ class HostMetricSummaryMonthlyTask:
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: