-
Notifications
You must be signed in to change notification settings - Fork 188
Servicenow Integration with AI SRE and update redirects from incident response #10865
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
Conversation
Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://686eb95aada70f990059002e--harness-developer.netlify.app |
This is a big PR, I'll wait until the SRE codeowner approves before reviewing. |
42ac58f
to
1469b12
Compare
Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6870fff0bda2dabbc37ebc97--harness-developer.netlify.app |
1469b12
to
4ce320e
Compare
Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://68763818742ace1506541713--harness-developer.netlify.app |
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.
Although not a result from this PR, /docs/ai-sre/runbooks/integrations/servicenow
page has an overuse of lists and bullet points (I counted 20+). Since this PR has been approved by the SME, my only feedback is for the https://github.com/harness/developer-hub/pull/10865/files#diff-7f0fe58b910f36a57698c78bef5362eba9bf25d976f0b3b461968e945fae0bcc image size. Please consider updating /docs/ai-sre/runbooks/integrations/servicenow
page via a future PR to swap some of the list items with paragraph.
Signed-off-by: Vasundhara Shukla <[email protected]>
Signed-off-by: Vasundhara Shukla <[email protected]>
4ce320e
to
15ddcfd
Compare
Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://687a47c39cf9cc3ada3d1964--harness-developer.netlify.app |
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.
Thanks for addressing the previous review comment. This will also need an SME review.
Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description
PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged: