Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

potiuk merged PR #35530: URL: https://github.com/apache/airflow/pull/35530 -- 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:

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

boring-cyborg[bot] commented on PR #35530: URL: https://github.com/apache/airflow/pull/35530#issuecomment-1819782883 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1397660023 ## tests/providers/opsgenie/typing/test_opsgenie.py: ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1397650965 ## tests/providers/opsgenie/typing/test_opsgenie.py: ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1397650221 ## tests/providers/opsgenie/typing/test_opsgenie.py: ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1397623702 ## tests/providers/opsgenie/typing/test_opsgenie.py: ## @@ -0,0 +1,106 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1392080905 ## docs/apache-airflow-providers-opsgenie/notifications/opsgenie_notifier.rst: ## @@ -0,0 +1,33 @@ + .. Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1391911465 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1391529962 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1389921840 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1389749318 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1389710558 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388671223 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388469452 ## tests/system/providers/opsgenie/example_opsgenie_notifier.py: ## @@ -0,0 +1,38 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388449127 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388359810 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388339827 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388339827 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

utkarsharma2 commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1388080760 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on PR #35530: URL: https://github.com/apache/airflow/pull/35530#issuecomment-1802421525 Need to fix [Static Checks](https://github.com/apache/airflow/blob/main/STATIC_CODE_CHECKS.rst), you could run locally `breeze static-checks {id-of-failed-check} --all-files` e.g.

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1387001825 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

Taragolis commented on code in PR #35530: URL: https://github.com/apache/airflow/pull/35530#discussion_r1386979341 ## airflow/providers/opsgenie/notifications/opsgenie.py: ## @@ -0,0 +1,150 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more

Re: [PR] feature(providers): added `OpsgenieNotifier` [airflow]

boring-cyborg[bot] commented on PR #35530: URL: https://github.com/apache/airflow/pull/35530#issuecomment-1802067040 Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution

[PR] feature(providers): added `OpsgenieNotifier` [airflow]

Eric-Mendes opened a new pull request, #35530: URL: https://github.com/apache/airflow/pull/35530 closes: #35516 Added an `OpsgenieNotifier`. It is almost the exact same implementation as `OpsgenieCreateAlertOperator`, but given that now notifiers exist, I