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

Add GCP Service Account rule #11230

Merged
merged 4 commits into from
Oct 9, 2020
Merged

Conversation

pcandoalmeida
Copy link
Contributor

Adds GCP Service Account rule check for Kubernetes configuration.

Related: #8765
Closes: #11047

@pcandoalmeida pcandoalmeida marked this pull request as ready for review October 2, 2020 12:15
@turbaszek turbaszek added the area:upgrade Facilitating migration to a newer version of Airflow label Oct 3, 2020
Copy link
Member

@turbaszek turbaszek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

GCP service account key deprecation
-----------------------------------
Option has been removed because it is no longer supported by the Google Kubernetes Engine.

Problems:
  1.  This option has been removed because it is no longer supported by the Google Kubernetes Engine. The new recommended service account keys for the Google Cloud management method is Workload Identity.

@pcandoalmeida
Copy link
Contributor Author

Thanks for the feedback as always @turbaszek 🚀

@potiuk
Copy link
Member

potiuk commented Oct 3, 2020

@pcandoalmeida Can you please rebase to latest v1-10-test ? It should work now.

@turbaszek
Copy link
Member

@pcandoalmeida can you clean up your PR, there's a lot of unrelated commits

@pcandoalmeida
Copy link
Contributor Author

Hey @turbaszek sure, will do!

@potiuk
Copy link
Member

potiuk commented Oct 9, 2020

cool :)

@pcandoalmeida pcandoalmeida deleted the v1-10-test branch October 9, 2020 23:49
kaxil pushed a commit that referenced this pull request Oct 11, 2020
turbaszek pushed a commit to PolideaInternal/airflow that referenced this pull request Oct 21, 2020
turbaszek pushed a commit to PolideaInternal/airflow that referenced this pull request Oct 23, 2020
@potiuk potiuk added this to the Airflow 1.10.13 milestone Nov 14, 2020
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:upgrade Facilitating migration to a newer version of Airflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create NoGCPServiceAccountKeyInConfigRule to ease upgrade to Airflow 2.0
3 participants