Skip to content

Fix test_remote_logging_s3 when contents not found - #58492

Merged
amoghrajesh merged 1 commit into
apache:mainfrom
jason810496:fix/e2e/fix-test_remote_logging_s3-content-response
Nov 19, 2025
Merged

Fix test_remote_logging_s3 when contents not found#58492
amoghrajesh merged 1 commit into
apache:mainfrom
jason810496:fix/e2e/fix-test_remote_logging_s3-content-response

Conversation

@jason810496

Copy link
Copy Markdown
Member

related:

Why

  • The test crash when there isn't Contents field in response = s3_client.list_objects_v2(Bucket=bucket_name) response.

What

  • Get contents variable from retry instead of from response["Contents"].
  • Add logging for raw API response from list_objects_v2 as well.

@amoghrajesh
amoghrajesh merged commit a32f94e into apache:main Nov 19, 2025
53 checks passed
aaron-wolmutt pushed a commit to aaron-wolmutt/airflow that referenced this pull request Nov 20, 2025
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants