Re: [PR] Consolidate the warning stacklevel in KubernetesPodTrigger [airflow]

2023-10-23 Thread via GitHub
hussein-awala merged PR #35079: URL: https://github.com/apache/airflow/pull/35079 -- 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...@ai

Re: [PR] Consolidate the warning stacklevel in KubernetesPodTrigger [airflow]

2023-10-20 Thread via GitHub
hussein-awala commented on code in PR #35079: URL: https://github.com/apache/airflow/pull/35079#discussion_r1367472697 ## airflow/providers/cncf/kubernetes/triggers/pod.py: ## @@ -102,7 +102,8 @@ def __init__( if should_delete_pod is not None: warnings.warn

Re: [PR] Consolidate the warning stacklevel in KubernetesPodTrigger [airflow]

2023-10-20 Thread via GitHub
amoghrajesh commented on code in PR #35079: URL: https://github.com/apache/airflow/pull/35079#discussion_r1366750841 ## airflow/providers/cncf/kubernetes/triggers/pod.py: ## @@ -103,6 +103,7 @@ def __init__( warnings.warn( "`should_delete_pod` param

[PR] Consolidate the warning stacklevel in KubernetesPodTrigger [airflow]

2023-10-20 Thread via GitHub
anukrati1507 opened a new pull request, #35079: URL: https://github.com/apache/airflow/pull/35079 Used stacklevel 2 instead of the default 1 in Closes issue: #34424 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https