[PR] Make EksPodOperator exec config not rely on log level [airflow]

2023-11-20 Thread via GitHub
syedahsn opened a new pull request, #35771: URL: https://github.com/apache/airflow/pull/35771 This PR improves the mechanism for getting credentials to authenticate to the EKS pod by wrapping the `eks_get_token.py` script and printing the credentials out to `stdout`, without having to inter

Re: [PR] Make EksPodOperator exec config not rely on log level [airflow]

2023-11-21 Thread via GitHub
o-nikolas commented on code in PR #35771: URL: https://github.com/apache/airflow/pull/35771#discussion_r1401293160 ## airflow/providers/amazon/aws/hooks/eks.py: ## @@ -521,6 +521,33 @@ def generate_config_file( :param eks_cluster_name: The name of the cluster to generat

Re: [PR] Make EksPodOperator exec config not rely on log level [airflow]

2023-11-21 Thread via GitHub
syedahsn commented on code in PR #35771: URL: https://github.com/apache/airflow/pull/35771#discussion_r1401326916 ## airflow/providers/amazon/aws/hooks/eks.py: ## @@ -521,6 +521,33 @@ def generate_config_file( :param eks_cluster_name: The name of the cluster to generate

Re: [PR] Make EksPodOperator exec config not rely on log level [airflow]

2023-11-21 Thread via GitHub
syedahsn commented on code in PR #35771: URL: https://github.com/apache/airflow/pull/35771#discussion_r1401331844 ## airflow/providers/amazon/aws/utils/eks_get_token.py: ## @@ -58,13 +57,7 @@ def main(): eks_hook = EksHook(aws_conn_id=args.aws_conn_id, region_name=args.reg

Re: [PR] Make EksPodOperator exec config not rely on log level [airflow]

2023-11-22 Thread via GitHub
o-nikolas merged PR #35771: URL: https://github.com/apache/airflow/pull/35771 -- 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-unsubscr...@airflo