Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configurable AWS Session Factory #21778

Merged
merged 1 commit into from
Feb 28, 2022
Merged

Conversation

alexchen8
Copy link
Contributor

Configurable AWS SessionFactory for users to have full control of AWS boto3 session instantiation. This will also allow custom federated AWS access.

closes: #21672

@boring-cyborg boring-cyborg bot added area:providers provider:amazon-aws AWS/Amazon - related issues labels Feb 24, 2022
@alexchen8 alexchen8 force-pushed the aws-session-factory branch 2 times, most recently from 5755c73 to 05ea2f6 Compare February 24, 2022 05:13
@potiuk
Copy link
Member

potiuk commented Feb 27, 2022

The change is fine but it has zero user-facing documentation. Please explain it in the amazon providers howtos and docstrings. Some examples of usage where it can be used should be added.

@alexchen8 alexchen8 changed the title Configurable AWS SessionFactory Configurable AWS Session Factory Feb 28, 2022
@alexchen8
Copy link
Contributor Author

Added documentation. Not sure if it's in the right format.

@potiuk
Copy link
Member

potiuk commented Feb 28, 2022

Added documentation. Not sure if it's in the right format.

This looks super cool!

@potiuk potiuk merged commit c819b4f into apache:main Feb 28, 2022
@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Feb 28, 2022
@github-actions
Copy link

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers okay to merge It's ok to merge this PR as it does not require more tests provider:amazon-aws AWS/Amazon - related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[AWS] Configurable AWS SessionFactory for AwsBaseHook
2 participants