[GitHub] [airflow] danielpoonwj edited a comment on issue #5731: [AIRFLOW-5117] support refreshing EKS api tokens

[GitBox <gi...@apache.org>]

danielpoonwj edited a comment on issue #5731: [AIRFLOW-5117] support refreshing EKS api tokens
URL: https://github.com/apache/airflow/pull/5731#issuecomment-570859204
 
 
   hi thank you for this fix!
   
   I've upgraded to 1.10.7 specifically for this fix but I realized that the datetime format for my expirationTimestamp is `%Y-%m-%dT%H:%M:%SZ` (eg. `2020-01-05T10:00:00Z`) instead of `%Y-%m-%dT%H:%M:%S%z`.
   
   it fails to parse and swallows the error with logging and essentially reverts to the previous behaviour.
   
   was just wondering if its a percularity in my setup where others are actually receiving the timestamp in that format, or if it's a bug.
   
   I can submit a fix depending if we need to support both datetime formats, or if we need to change it.
   
   for reference, I am running airflow out of cluster on EC2 (Ubuntu 16.04) interacting with kubernetes v1.14.9 on EKS. awscli v1.16.310

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services