You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2021/08/26 02:14:54 UTC
[GitHub] [skywalking] wu-sheng opened a new issue #7572: Support PagerDuty immigration for alarm
wu-sheng opened a new issue #7572:
URL: https://github.com/apache/skywalking/issues/7572
PagerDuty(https://www.pagerduty.com/) is widely used for on-call globally. We are welcome the contributions to support it as alarm notification channel.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923494242
@Superskyyy Good to know they are listing this. I read the list page, which seems fine from policy perspective.
Before that, we need this feature ready and released, then we at least have a version mentioned on their page.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] harry-dev98 commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
harry-dev98 commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-912727640
Okay this is what i understand:
So we do have couple of channels already like webhook, gRPChook, slackhook,.. etc. And now we need to add pagerduty as another new channel supported by skywalking, right?
also, is pagerduty opensource? I looked through their website its kinda paid service (yes free for some caped usage)
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923490170
Hey there, just curious about the progress. Back when this issue was first created, I actually set up a PagerDuty developer account and a mock integration to try the implementation. Now PagerDuty people actually reached out to me asking about if SkyWalking would like to list the integration officially on their [website](https://www.pagerduty.com/integrations/). If there's no Apache regulations against it, I think we should do it when this merges. FYI @wu-sheng @harry-dev98
From my intern experience, PagerDuty is extremely popular and widely used among companies worldwide, I believe this is a good step to expand our community globally.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-912862461
It is not open source. But once it has open HTTP API, we could try to integrate.
This is totally optional, and rarely changed API.
Please check whether this is correct.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923605232
Also PagerDuty is a very interesting platform that differs from previous platforms that we support, it can do bi-directional interaction, maybe interesting to have some side projects in the future. For this one, I will only implement basic alarms.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng removed a comment on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng removed a comment on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923502684
Yes, that is the current plan.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy edited a comment on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy edited a comment on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923490170
Hey there, just curious about the progress. Back when this issue was first created, I actually set up a PagerDuty developer account and a mock integration to try the implementation. Now PagerDuty people actually reached out to me asking about if SkyWalking would like to list the integration officially on their [website](https://www.pagerduty.com/integrations/). If there's no Apache regulations against it, I think we should do it when this merges. FYI @wu-sheng @harry-dev98
From my experience, PagerDuty is extremely popular and widely used among companies worldwide, I believe this is a good step to expand our community globally.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923494919
Or @Superskyyy Do you already have codes of this feature ready to submit?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng edited a comment on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng edited a comment on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923494919
Or @Superskyyy Do you already have codes of this feature ready to submit? 8.8.0 release is closing, 8.9.0 could be on Nov. or Dec. So you any of you have this feature ready, I would love to include this in the 8.8.0 release.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-929762840
Sure, we will be on China National Day vacation, and we are doing 8.8.1 release as a hot fix release.
Feel free to send pull request to 8.9.0
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923596964
Fyi @wu-sheng ^^
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] harry-dev98 commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
harry-dev98 commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-907781571
hey, I am kinda new to this community. I'd like to head with this issue to start contributing, can you please share bit of more information what is the requirement of this issue?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-929760743
Update, I implemented it over last weekend. Been busy at school reading papers, it might take another few days before opening PR.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-1046007156
@Superskyyy Is this PR ready to go? Or we should find someone else to cover this?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] harry-dev98 commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
harry-dev98 commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-912936955
regarding APIs, yes pagerduty provides REST APIs. Basically two types of APIs one for managing the account other for events generated by some tool like skywalking,etc. So should i try integrating events api for now, and lateron can be extended to other type of API i.e. to manage account.
We just need to make sure the alarm notifications reach pagerduty account of the user, right?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923598170
> Fyi @wu-sheng ^^
Then, @Superskyyy If you want and have time, it is open for you to take over responsibility.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923611298
> Also PagerDuty is a very interesting platform that differs from previous platforms that we support, it can do bi-directional interaction, maybe interesting to have some side projects in the future. For this one, I will only implement basic alarms.
Make sense. Feel free to open a discussion and involve their team in.
Let's see what we could build.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-907789190
@harry-dev98 Thanks. We have various notifications, from webhook to slack. PageDuty seems a very important channel to forward the alarm.
https://skywalking.apache.org/docs/main/latest/en/setup/backend/backend-alarm/#webhook
The source codes about the existing notification implementations are here, https://github.com/apache/skywalking/tree/master/oap-server/server-alarm-plugin/src/main/java/org/apache/skywalking/oap/server/core/alarm/provider.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy edited a comment on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy edited a comment on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923496779
> Or @Superskyyy Do you already have codes of this feature ready to submit?
@wu-sheng Nah, I simply tested with its APIs on the mock integration, I didn't add anything to SW codebase. I'll say lets wait a day or two for Harry's response, if no response I can do it and if goes smoothly we can release it next month. We plan to release 8.8.0 after Oct.10th right?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] harry-dev98 commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
harry-dev98 commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923559758
Hey there, i don't have solid advancement, i just used pager duty apis just to understand how they work, and then got busy with my own stuff.. it may take some time to integrate 🥺
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923502599
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923494395
@harry-dev98 Do you have any update to share?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-929760743
Update, I implemented it over last weekend. Been busy at school reading papers, it might take another few days before opening PR.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923601327
> > Fyi @wu-sheng ^^
>
> Then, @Superskyyy If you want and have time, it is open for you to take over responsibility.
I can figure something out this weekend, I think it doesn't require too much changes for only a alarm.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-912937385
> So should i try integrating events api for now, and lateron can be extended to other type of API i.e. to manage account.
Yes, try event APIs. And we may not need to spend time on account management APIs. As we don't have UI or feature about this. Only if you want to provide some scripts to create or set up account, then those APIs help.
> We just need to make sure the alarm notifications reach pagerduty account of the user, right?
Yes, you could create a trial account, and make sure the alarm messages landed there.
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] Superskyyy commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
Superskyyy commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-923496779
> Or @Superskyyy Do you already have codes of this feature ready to submit?
@wu-sheng
Nah, I simply tested with its APIs on the mock integration, I didn't add anything to SW codebase. I'll say lets wait a day or two for Harry's response, if no response I can do it and if goes smoothly we can release it next month. We plan to release 8.8.0 after Oct.10th right?
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [skywalking] wu-sheng commented on issue #7572: Support PagerDuty immigration for alarm
Posted by GitBox <gi...@apache.org>.
wu-sheng commented on issue #7572:
URL: https://github.com/apache/skywalking/issues/7572#issuecomment-929762840
Sure, we will be on China National Day vacation, and we are doing 8.8.1 release as a hot fix release.
Feel free to send pull request to 8.9.0
--
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: notifications-unsubscribe@skywalking.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org