You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/12/05 18:56:14 UTC
[GitHub] [airflow] Taragolis opened a new pull request, #28129: Reset AWS settings (env var) to mock values for each amazon unit test
Taragolis opened a new pull request, #28129:
URL: https://github.com/apache/airflow/pull/28129
Follow up: https://github.com/apache/airflow/pull/27823#issuecomment-1336217469
This PR add two changes in Amazon Provider tests
1. Move check no cred exception from `test_s3` to `test_base_aws`
2. Clear AWS environment variables to mocked values
- If developer set in `files/airflow-breeze-config/variables.env` or in IDE some [AWS specific variables](https://boto3.amazonaws.com/v1/documentation/api/latest/guide/configuration.html#using-environment-variables) it might affect local tests execution
- By default if credentials not set then `botocore`/`boto3` tried to use credentials/configs from `default` profile from `~/.aws/config` and `~/.aws/credentials` as well as legacy boto configuration. For prevent this behaviour create temporary empty configurations files (on session level) and set appropriate ENV variables
- Set fake AWS Credentials for prevent usage of actual credentials. In this case some tests might failed with `An error occurred (UnrecognizedClientException) when calling the FooBar operation: The security token included in the request is invalid.`
--
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
[GitHub] [airflow] potiuk merged pull request #28129: Reset AWS settings (env var) to mock values for each amazon unit test
Posted by GitBox <gi...@apache.org>.
potiuk merged PR #28129:
URL: https://github.com/apache/airflow/pull/28129
--
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
[GitHub] [airflow] ferruzzi commented on pull request #28129: Reset AWS settings (env var) to mock values for each amazon unit test
Posted by GitBox <gi...@apache.org>.
ferruzzi commented on PR #28129:
URL: https://github.com/apache/airflow/pull/28129#issuecomment-1338290505
Heh. Jarek approved and merged while I was looking at it, but a posthumous LGTM :P
--
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