You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "bshea5 (via GitHub)" <gi...@apache.org> on 2025/01/13 20:59:04 UTC

[I] PagerDuty PagerdutyNotifier Class Uses a Removed Method and Fails to Send Notifications [airflow]

bshea5 opened a new issue, #45626:
URL: https://github.com/apache/airflow/issues/45626

   ### Apache Airflow Provider(s)
   
   pagerduty
   
   ### Versions of Apache Airflow Providers
   
   apache-airflow-providers-pagerduty **4.0**
   
   ### Apache Airflow version
   
   2.10.4
   
   ### Operating System
   
   Ubuntu
   
   ### Deployment
   
   Official Apache Airflow Helm Chart
   
   ### Deployment details
   
   _No response_
   
   ### What happened
   
   PagerDuty alerts sent via `PagerdutyNotifier` are not sent. Looking at the source code, it looks like the `notify` method is using the `create_event` method which no longer exists.
   
   
   ```
       def notify(self, context):
           """Send a alert to a pagerduty event v2 API."""
           self.hook.**create_event**(
               summary=self.summary,
               severity=self.severity,
               source=self.source,
               action=self.action,
               dedup_key=self.dedup_key,
               custom_details=self.custom_details,
               group=self.group,
               component=self.component,
               class_type=self.class_type,
               images=self.images,
               links=self.links,
           )
   ```
   
   ### What you think should happen instead
   
   As noted in the most recent change log, https://airflow.apache.org/docs/apache-airflow-providers-pagerduty/stable/changelog.html, switch the `create_event` method to `send_event`.
   
   ### How to reproduce
   
   Trigger an alert with `PagerdutyNotifier` 
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [x] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] PagerDuty PagerdutyNotifier Class Uses a Removed Method and Fails to Send Notifications [airflow]

Posted by "eladkal (via GitHub)" <gi...@apache.org>.
eladkal closed issue #45626: PagerDuty PagerdutyNotifier Class Uses a Removed Method and Fails to Send Notifications
URL: https://github.com/apache/airflow/issues/45626


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] PagerDuty PagerdutyNotifier Class Uses a Removed Method and Fails to Send Notifications [airflow]

Posted by "bshea5 (via GitHub)" <gi...@apache.org>.
bshea5 commented on issue #45626:
URL: https://github.com/apache/airflow/issues/45626#issuecomment-2588510273

   Submitted a PR, https://github.com/apache/airflow/pull/45630


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] PagerDuty PagerdutyNotifier Class Uses a Removed Method and Fails to Send Notifications [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on issue #45626:
URL: https://github.com/apache/airflow/issues/45626#issuecomment-2588186241

   Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org