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

[Armis Event Collector] Update initial fetch handling #30030

Merged

Conversation

samuelFain
Copy link
Contributor

@samuelFain samuelFain commented Oct 4, 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

relates: XSUP-28767

Description

This PR sets the last run for the initial fetch, so that the original fetch time would remain constant until events are fetched (and not update the initial fetch time every fetch cycle).

Changes made:

  • Set last run to current time on initial fetch events run.
  • Add/Update UTs

Must have

  • Tests
  • Documentation

@samuelFain samuelFain self-assigned this Oct 4, 2023
Copy link
Contributor

@JasBeilin JasBeilin left a comment

Choose a reason for hiding this comment

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

Nice!

@samuelFain samuelFain merged commit 3f06470 into master Oct 4, 2023
18 checks passed
@samuelFain samuelFain deleted the bugfix/XSUP-28767/armis-collector-fix-initial-fetch branch October 4, 2023 12:52
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Oct 5, 2023
* Update initial fetch handling

* Update docstring

* Update Release Notes

* Update fetch logic

* Add test_set_last_run_with_current_time_initial UT

* Add debug log for initial fetch

* Add case to test_fetch_flow_cases UT

* Update UT

* Update UT
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* Update initial fetch handling

* Update docstring

* Update Release Notes

* Update fetch logic

* Add test_set_last_run_with_current_time_initial UT

* Add debug log for initial fetch

* Add case to test_fetch_flow_cases UT

* Update UT

* Update UT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants