Skip to content
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

[ASM] - EXPANDER 4377 - ServiceNow ITSM Enrichment #30296

Conversation

BigEasyJ
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

EXPANDR-4377

Description

Create a new playbook to query ServiceNow ITSM for service owners.

Must have

  • Tests
  • Documentation

@content-bot content-bot added Contribution Thank you! Contributions are always welcome! External PR Xsoar Support Level Indicates that the contribution is for XSOAR supported pack labels Oct 18, 2023
@content-bot content-bot changed the base branch from master to contrib/PaloAltoNetworks_EXPANDR-4377 October 18, 2023 12:40
@content-bot
Copy link
Collaborator

Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @sapirshuker will know the proposed changes are ready to be reviewed.
For your convenience, here is a link to the contributions SLAs document.

@content-bot
Copy link
Collaborator

Hi @BigEasyJ, thanks for contributing to a Cortex XSOAR supported pack. To receive credit for your generous contribution please follow this link.

@content-bot content-bot added Community Contribution Form Filled Whether contribution form filled or not. labels Oct 18, 2023
@BigEasyJ BigEasyJ marked this pull request as draft October 18, 2023 13:46
quietmode: 0
isoversize: false
isautoswitchedtoquietmode: false
'27':
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BigEasyJ , in this task (or a separate task) we need to make sure SNOW-ITSM integration is enabled or there is no reason to continue.

Copy link
Contributor

@johnnywilkes johnnywilkes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BigEasyJ , thanks for the change.

@capanw
Copy link
Contributor

capanw commented Oct 18, 2023

LGTM

@BigEasyJ BigEasyJ marked this pull request as ready for review October 18, 2023 19:08
@sapirshuker
Copy link
Contributor

sapirshuker commented Oct 19, 2023

@ShirleyDenkberg , Can you please review?
I think that the RN format is incorrect, A hypen should be added to the RN format.

id: d3007e79-c33c-4f6f-83f3-e7ee98dfb3b2
version: -1
name: Build ServiceNow query - description
description: Set the beginning of a key used for querying ServiceNow using SN descriptions and a timestamp.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Set the beginning of a key used for querying ServiceNow using SN descriptions and a timestamp.
description: Set the beginning of a key used for querying ServiceNow using ServiceNow descriptions and a timestamp.

id: 962733bf-859d-4823-8d39-478d4fb092e8
version: -1
name: Build ServiceNow query - short description
description: Set the remaining query value with SN short descriptions for querying ServiceNow.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Set the remaining query value with SN short descriptions for querying ServiceNow.
description: Set the remaining query value with ServiceNow short descriptions for querying ServiceNow.

id: 449135ee-dbef-48c5-8e00-597ce6f55cf0
version: -1
name: Were search terms provided?
description: Conditional to check if values were provided to the search terms input or not.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Conditional to check if values were provided to the search terms input or not.
description: Check if values were provided to the search terms input or not.

id: c9c5b720-7cd3-406e-8a2e-3af77876dc8b
version: -1
name: Were any assignees found?
description: Conditional to check if any assignees were found in the assigned_to property of retrieved ServiceNow tickets.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Conditional to check if any assignees were found in the assigned_to property of retrieved ServiceNow tickets.
description: Check if any assignees were found in the assigned_to property of retrieved ServiceNow tickets.

- key: search_terms
value: {}
required: true
description: Search terms to be used in the ServiceNow ITSM query search
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Search terms to be used in the ServiceNow ITSM query search
description: Search terms to be used in the ServiceNow ITSM query search.


### Integrations

* ServiceNow v2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* ServiceNow v2
ServiceNow v2

@ShirleyDenkberg
Copy link
Contributor

@sapirshuker When there is only one item in a list, no hyphen is needed.

@johnnywilkes @capanw @melamedbn @sapirshuker Doc review completed.

@sapirshuker
Copy link
Contributor

Hey, @BigEasyJ
Please fix the doc review comments and let me know when the PR is ready for review.

@BigEasyJ
Copy link
Contributor Author

@sapirshuker updated and ready

@melamedbn melamedbn removed their request for review October 22, 2023 07:12
@sapirshuker sapirshuker added the ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. label Oct 22, 2023
@content-bot
Copy link
Collaborator

content-bot commented Oct 22, 2023

For the Reviewer: Successfully created a pipeline in Gitlab with url: https://code.pan.run/xsoar/content/-/pipelines/6690165

@sapirshuker
Copy link
Contributor

Build process failed due an known issue: CIAC-6419

sapirshuker
sapirshuker previously approved these changes Oct 22, 2023
@sapirshuker sapirshuker dismissed their stale review October 22, 2023 11:14

@BigEasyJ please resolve the conflicts

@sapirshuker
Copy link
Contributor

Hey, @BigEasyJ
please resolve the conflicts.

@sapirshuker sapirshuker added ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. and removed ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. labels Oct 24, 2023
@sapirshuker
Copy link
Contributor

Build process failed due an known issue: CIAC-6419

@sapirshuker sapirshuker self-requested a review October 24, 2023 12:16
@sapirshuker sapirshuker merged commit 5401f1f into demisto:contrib/PaloAltoNetworks_EXPANDR-4377 Oct 24, 2023
24 of 25 checks passed
sapirshuker pushed a commit that referenced this pull request Oct 25, 2023
* ReadMe and PNG

* Add ASM ServiceNow ITSM Enrichment playbook

- Fix format of ASM Enrichment playbook
- Add SN ITSM Enrichment to main Enrichment playbook

* Add release notes

* Update release notes

* Update README, formatting, pack ignore

* Validation fix

* Change fromversion

* Add integration enabled check

* Fix formatting

* Doc revisions

Co-authored-by: John <40349459+BigEasyJ@users.noreply.github.com>
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* ReadMe and PNG

* Add ASM ServiceNow ITSM Enrichment playbook

- Fix format of ASM Enrichment playbook
- Add SN ITSM Enrichment to main Enrichment playbook

* Add release notes

* Update release notes

* Update README, formatting, pack ignore

* Validation fix

* Change fromversion

* Add integration enabled check

* Fix formatting

* Doc revisions

Co-authored-by: John <40349459+BigEasyJ@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! docs-approved External PR ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. Security Review Xsoar Support Level Indicates that the contribution is for XSOAR supported pack
Projects
None yet
7 participants