Skip to content

Flashpoint - Removed the call to get_pack_version function #38524

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

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

ShacharKidor
Copy link
Contributor

@ShacharKidor ShacharKidor commented Feb 6, 2025

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: XSUP-46473.

Description

It turns out that the call to the get_pack_version() function that occurs every run of the main function causes performance issues, therefore we temporarily replace this call with a hardcoded string of the current pack version.

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Feb 6, 2025

Your contributed Flashpoint pack has been modified on files:

Packs/Flashpoint/ReleaseNotes/2_0_4.md
Packs/Flashpoint/pack_metadata.json
Packs/Flashpoint/Integrations/Ignite/Ignite.py
Please review the changes here

Copy link

github-actions bot commented Feb 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Flashpoint/Integrations/Ignite
   Ignite.py10264395%216, 263, 281–287, 363, 530, 533, 622, 814, 1031–1032, 1067, 1202, 1286, 1608, 1850–1855, 1857, 2030, 2036–2037, 2045, 2080–2081, 2405–2408, 2418, 2430, 2435–2436, 2439, 2443
Packs/FlashpointFeed/Integrations/IgniteFeed
   IgniteFeed.py2462291%97, 218–219, 399–400, 503–505, 507, 566, 572, 578–587, 589
TOTAL12726594% 

Tests Skipped Failures Errors Time
108 0 💤 0 ❌ 0 🔥 5.624s ⏱️

@ShacharKidor ShacharKidor marked this pull request as ready for review February 6, 2025 14:58
@yuvalbenshalom yuvalbenshalom merged commit eac6897 into master Feb 6, 2025
25 of 31 checks passed
@yuvalbenshalom yuvalbenshalom deleted the flashpoint_remove_get_pack_version_call branch February 6, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants