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

Remove identity center auth manager cli #42481

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

o-nikolas
Copy link
Contributor

The CLI command to setup identity center could only setup part of the required resources, since adding an application must be done from the console. As of November 15, 2023 it is now required to have an AWS Organization setup to create the required type of Identity Center Instance. The script would have to be change majorly to achieve this but it is also something that should be done with great care and intention since creating an organization in your AWS account has implications. If we automate it, many users won't know it's being created. Instead have users run through the wizard provided in the AWS console.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

The CLI command to setup identity center could only setup part of the
required resources, since adding an application must be done from the
console. As of November 15, 2023 it is now required to have an
AWS Organization setup to create the required type of Identity Center
Instance. The script would have to be change majorly to achieve this but
it is also something that should be done with great care and intention
since creating an organization in your AWS account has implications. If
we automate it, many users won't know it's being created. Instead have
users run through the wizard provided in the AWS console.
@o-nikolas o-nikolas merged commit f6852c2 into apache:main Sep 25, 2024
54 checks passed
@o-nikolas o-nikolas deleted the onikolas/aws_auth_cli_deprecate branch September 25, 2024 22:09
@apache apache deleted a comment from supmcgee2 Sep 26, 2024
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
* Remove identity center auth manager cli

The CLI command to setup identity center could only setup part of the
required resources, since adding an application must be done from the
console. As of November 15, 2023 it is now required to have an
AWS Organization setup to create the required type of Identity Center
Instance. The script would have to be change majorly to achieve this but
it is also something that should be done with great care and intention
since creating an organization in your AWS account has implications. If
we automate it, many users won't know it's being created. Instead have
users run through the wizard provided in the AWS console.

* Missing test change
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* Remove identity center auth manager cli

The CLI command to setup identity center could only setup part of the
required resources, since adding an application must be done from the
console. As of November 15, 2023 it is now required to have an
AWS Organization setup to create the required type of Identity Center
Instance. The script would have to be change majorly to achieve this but
it is also something that should be done with great care and intention
since creating an organization in your AWS account has implications. If
we automate it, many users won't know it's being created. Instead have
users run through the wizard provided in the AWS console.

* Missing test change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants