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

Skip core pack installation in XSIAM #27692

Merged
merged 10 commits into from Jun 26, 2023
Merged

Skip core pack installation in XSIAM #27692

merged 10 commits into from Jun 26, 2023

Conversation

adi88d
Copy link
Contributor

@adi88d adi88d commented Jun 25, 2023

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

fixes: link to the issue

Description

  • Added the UNREMOVABLE_PACKS list.
  • Updated reset_base_pack_version method name to ** reset_core_pack_version**.
  • Use the search_and_install_packs_and_their_dependencies method to install the UNREMOVABLE_PACKS.

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

@adi88d adi88d requested a review from daryakoval as a code owner June 25, 2023 08:32
Copy link
Contributor

@daryakoval daryakoval left a comment

Choose a reason for hiding this comment

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

Please handle all "unremoable packs" like the base pack in the function:

def reset_base_pack_version(client: demisto_client):

Tests/Marketplace/search_and_uninstall_pack.py Outdated Show resolved Hide resolved
@@ -11,6 +11,8 @@
from Tests.Marketplace.search_and_install_packs import install_packs
from time import sleep

UNREMOVABLE_PACKS = ['Base', 'CoreAlertFields', 'Core']
Copy link
Contributor

Choose a reason for hiding this comment

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

Are there mandatory dependencies for these packs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, for Core:
image
and for CoreAlertFields:
image

@adi88d adi88d requested a review from daryakoval June 25, 2023 14:19
@adi88d adi88d requested a review from ostolero as a code owner June 26, 2023 06:32
@DeanArbel DeanArbel merged commit 41f44fd into master Jun 26, 2023
8 of 13 checks passed
@DeanArbel DeanArbel deleted the skip-core-packs branch June 26, 2023 08:04
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants