[GitHub] [airflow] vincbeck commented on pull request #31197: Add missing DynamoDBToS3Operator system test

["vincbeck (via GitHub)" <gi...@apache.org>]

vincbeck commented on PR #31197:
URL: https://github.com/apache/airflow/pull/31197#issuecomment-1548018392

   Our environment is using Breeze in order to run system tests. We are building the last image with `breeze ci-image build` and then un the test using the command `breeze testing tests <filename> --system amazon`.
   
   But given the exception I dont think it is question of breeze, the service Dynamodb is complaining that `export_time` is invalid. My guess is `datetime.now()` returns in some cases (depending on the timezone) a date in the future and in some cases a date in the present/past. I might recommend to use `DateTime.UtcNow()` instead with some buffer (e.g. exclude one week maybe?)


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org