feat(incident-io): add account_config.json with API key auth - #24431
Conversation
There was a problem hiding this comment.
More details
Configuration file adds incident.io account authentication schema with API key bearer token support. Validation confirms valid JSON structure, correct field types, proper naming conventions, and full downstream consumer compatibility. No regressions identified.
📊 Validated against 12 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit fb2de8c · What is Autotest? · Any feedback? Reach out in #autotest
| "bearer_token": { | ||
| "password_setting": { | ||
| "key": "incident_io_api_key", | ||
| "label": "Incident.io API key", |
There was a problem hiding this comment.
nit: not sure how this renders but would Incident.io API Key be a bit long in the UI? Should we do label API Key and then set help to be incident.io API Key?
Review from yfangechiu is dismissed. Related teams and files:
- collaboration-integrations
- incident_io/assets/account_config.json
Validation ReportAll 21 validations passed. Show details
|
leandroalmeida1997
left a comment
There was a problem hiding this comment.
dont forget to promote the field once the asset is published.
you can read more in https://datadoghq.atlassian.net/wiki/spaces/WEBINT/pages/6877184047/Integration+Lifecycle+AMS+crawler-sdk+Schema+Validations#Adding-Fields-to-Existing-Integration-or-New-crawler-that-needs-new-field/dataflow
What does this PR do?
Add incident.io account config schema
Motivation
Add support for incident.io resources
Review checklist (to be filled by reviewers)
qa/requiredif this PR needs QA validation, orqa/skip-qaif it does not. Exactly one of the two is required.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged