-
Notifications
You must be signed in to change notification settings - Fork 24.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Watcher: Switch to pagerduty API V2 #32243
Comments
Pinging @elastic/es-core-infra |
If I read this correctly:
it implies that new users would not be able to use our PagerDuty integration today. |
The PagerDuty v1 API is EOL and will stop accepting new accounts shortly. This commit swaps out the watcher use of the v1 API with the new v2 API. It does not change anything about the existing watcher API. Closes elastic#32243
So after finishing the PR, I got to reading the actual docs here. Look at the gem I found.
This comes directly from the linked doc in the issue description. The only place we use the PagerDuty API is here, and for brevity ill include the 2 lines below.
This means we are only using the Events API, and yes we are still using the V1 Events API, which will not be affected by this. The good news is that we can still migrate to the V2 Events API. PagerDuty has a V2 Events API already. The work is done in the PR attached to this issue. I am not sure this warrants a blocker / critical status anymore though. |
Agreed on this one. Events API seems safe. I think it makes sense to switch to the newer API then on master only, once verified that old tokens can be reused. But this is not a blocker anymore. |
I think its worth backporting to 6.5 so future backports (which there will at least be one I know of) will be clean. Sound good @spinscale ? |
The PagerDuty v1 API is EOL and will stop accepting new accounts shortly. This commit swaps out the watcher use of the v1 API with the new v2 API. It does not change anything about the existing watcher API. Closes #32243
The PagerDuty v1 API is EOL and will stop accepting new accounts shortly. This commit swaps out the watcher use of the v1 API with the new v2 API. It does not change anything about the existing watcher API. Closes #32243
I am running ES v6.4.3. Just tried creating a new API key. PagerDuty only creates v2 keys now, and it assigned a 20 character alphanumeric key. Plugged API key into elasticsearch.yml and created a watcher to use it. Checked .watcher-history index and found the call had failed with a 400 error and the following body. "body": """{"status":"invalid event","message":"Event object is invalid","errors":["Service key is the wrong length (should be 32 characters)","Service key cannot contain non-alphanumeric characters."]}""" |
Got it working, but had to create the API key in PagerDuty via Configuration/Services, rather than Configuration/API Access as is documented at https://www.elastic.co/guide/en/watcher/current/configuring-pagerduty.html N.B.: During key creation, I selected "Events API v2" key, and this yielded a 32-character key. |
@hub-cap can you take a look in the docs and fix if needed? |
@hub-cap my pagerduty trial still has 12 days remaining. let me know if I can provide screenshots or my trial credentials for you to take screenshots. |
heh, i missed this ping somehow. grr. Ill update the docs here, thanks for the notificaiton. |
Watcher is currently using the v1 API events endpoint, available under https://v1.developer.pagerduty.com/documentation/events
This is however decommissioned at the 19th of October, so we have to switch to V2 API as soon as possible.
More info under https://v2.developer.pagerduty.com/
The text was updated successfully, but these errors were encountered: