schema: correct required fields for pagerduty notifier (#17277)

This commit is contained in:
Robert Lin 2021-01-15 08:44:06 +08:00 committed by GitHub
parent 4497b71c82
commit 4428dfdd0d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 4 additions and 3 deletions

View File

@ -37,6 +37,7 @@ All notable changes to Sourcegraph are documented in this file.
- Fixed search highlighting the wrong line. [#10468](https://github.com/sourcegraph/sourcegraph/issues/10468)
- Fixed an issue where searches of the form `foo type:file` returned results of type `path` too. [#17076](https://github.com/sourcegraph/sourcegraph/issues/17076)
- Fixed queries like `(type:commit or type:diff)` so that if the query matches both the commit message and the diff, both are returned as results. [#16899](https://github.com/sourcegraph/sourcegraph/issues/16899)
- Fixed a nonexistent field in site configuration being marked as "required" when configuring PagerDuty alert notifications. [#17277](https://github.com/sourcegraph/sourcegraph/pull/17277)
### Removed

View File

@ -820,7 +820,7 @@ type NotifierOpsGenie struct {
type NotifierPagerduty struct {
ApiUrl string `json:"apiUrl,omitempty"`
// IntegrationKey description: Integration key for the PagerDuty Events API v2 - see https://developer.pagerduty.com/docs/events-api-v2/overview
IntegrationKey string `json:"integrationKey,omitempty"`
IntegrationKey string `json:"integrationKey"`
// Severity description: Severity level for PagerDuty alert
Severity string `json:"severity,omitempty"`
Type string `json:"type"`

View File

@ -1155,7 +1155,7 @@
"NotifierPagerduty": {
"description": "PagerDuty notifier",
"type": "object",
"required": ["type", "routingKey"],
"required": ["type", "integrationKey"],
"properties": {
"type": {
"type": "string",

View File

@ -1160,7 +1160,7 @@ const SiteSchemaJSON = `{
"NotifierPagerduty": {
"description": "PagerDuty notifier",
"type": "object",
"required": ["type", "routingKey"],
"required": ["type", "integrationKey"],
"properties": {
"type": {
"type": "string",