Migrate to modern datetime library API#5840
Migrate to modern datetime library API#5840emmanuel-ferdman wants to merge 1 commit intoemissary-ingress:masterfrom
Conversation
Signed-off-by: Emmanuel Ferdman <emmanuelferdman@gmail.com>
|
@the-wondersmith could you please review this small PR when you get the chance? thanks 😄 |
|
@emmanuel-ferdman Just for the sake of due diligence, do you happen to know where the deprecation schedule associated with the warning is? Just want to make sure the change is compatible across the (admittedly narrow) range of Python versions emissary needs to comply with. |
|
@the-wondersmith thanks for the review. The |
|
@the-wondersmith anything else is needed from my side? thanks 😄 |
|
@emmanuel-ferdman No, I don't think so. But it looks like CI isn't being cooperative. @kflynn do you have a lever to pull here? |
|
@the-wondersmith @kflynn how can we progress from here? 🙌 |
|
^ bump @kflynn |
|
Hi @kflynn, how can we progress with this PR? thanks 😄 |
|
Hey @emmanuel-ferdman, so sorry for the silence here! I think the way forward here is going to be the We're in the middle of getting CI to properly run there, so hang tight for a bit longer and we'll get this moved over. Thanks! |
|
Hi @kflynn, any update? 😄 |


Description
This small PR resolves the
datetimelibrary warnings:Related Issues
Testing
Tests.
Checklist
Does my change need to be backported to a previous release?
I made sure to update
CHANGELOG.md.Remember, the CHANGELOG needs to mention:
This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:
My change is adequately tested.
Remember when considering testing:
I updated
CONTRIBUTING.mdwith any special dev tricks I had to use to work on this code efficiently.The changes in this PR have been reviewed for security concerns and adherence to security best practices.