-
Notifications
You must be signed in to change notification settings - Fork 26
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
Updated Pagerduty Self Service GitHub Actions #1215
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
@@ -191,8 +192,9 @@ Create a new self service action using the following JSON configuration. | |||
"{{if (.inputs | has(\"from\")) then \"from\" else null end}}": "{{.inputs.\"from\"}}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"{{if (.inputs | has(\"from\")) then \"from\" else null end}}": "{{.inputs.\"from\"}}", | |
"from": "{{.inputs.\"from\"}}", | |
@@ -250,16 +251,17 @@ Create a new self service action using the following JSON configuration. | |||
"type": "GITHUB", | |||
"org": "<GITHUB_ORG>", | |||
"repo": "<GITHUB_REPO>", | |||
"workflow": "change-incident-owner.yaml", | |||
"workflow": "change-on-call-user.yaml", | |||
"workflowInputs": { | |||
"{{if (.inputs | has(\"ref\")) then \"ref\" else null end}}": "{{.inputs.\"ref\"}}", | |||
"{{if (.inputs | has(\"start_time\")) then \"start_time\" else null end}}": "{{.inputs.\"start_time\"}}", | |||
"{{if (.inputs | has(\"end_time\")) then \"end_time\" else null end}}": "{{.inputs.\"end_time\"}}", | |||
"{{if (.inputs | has(\"new_on_call_user\")) then \"new_on_call_user\" else null end}}": "{{.inputs.\"new_on_call_user\"}}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all the inputs here too could go through this change:
"{{if (.inputs | has(\"new_on_call_user\")) then \"new_on_call_user\" else null end}}": "{{.inputs.\"new_on_call_user\"}}", | |
"new_on_call_user": "{{.inputs.\"new_on_call_user\"}}", | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thats relevant for the other files in this PR as well
...ice-experiences/setup-backend/github-workflow/examples/PagerDuty/create-pagerduty-service.md
Show resolved
Hide resolved
...e-experiences/setup-backend/github-workflow/examples/PagerDuty/trigger-pagerduty-incident.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go over @pazhersh comments
Description
Improved existing pagerduty guides and fixed bug in obtaining the entity ids
Updated docs pages
/create-self-service-experiences/setup-backend/github-workflow/examples/PagerDuty/
BugFix
.entity.identifier
Improvement
relations
field to all DAY-2 actionscreate-pagerduty-service
andtrigger incident
actions to report updated runs back to port