provider-aws
provider-aws copied to clipboard
crossplane-contrib
Time parse issue in compareTimeRanges (maintenanceWindow)
Hi,
i guess i have discovered a bug in the comparison logic of the compareTimeRanges code for checking if the maintenanceWindow needs to be updated. https://github.com/crossplane-contrib/provider-aws/blob/4b536aa882f6e4b4914c556bbfc4854ea5461a74/pkg/controller/rds/dbinstance/setup.go#L590-L616
What happened?
The current implementation to check the timeRanges rely on the idea that the weekday in the string ("mon") is parsed into a time object. But according to the docs this is not the case https://pkg.go.dev/time#Parse
The day of the week is checked for syntax but it is otherwise ignored.
So its only compared to the hour and minute and not weekday.
This results in a wrong comparison result to update the maintenanceWindow and thus could cause a skipped update if only the weekday of a maintenanceWindow would be changed.
How can we reproduce it?
The following snippet will show the issue with a simplified function of the compareTimeRanges code.
https://go.dev/play/p/n2jRovBJBj5
What environment did it happen in?
Bug in the code
Crossplane does not currently have enough maintainers to address every issue and pull request. This issue has been automatically marked as stale because it has had no activity in the last 90 days. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Crossplane does not currently have enough maintainers to address every issue and pull request. This issue has been automatically marked as stale because it has had no activity in the last 90 days. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.