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

[CommonServerPython] Add offset support to lookback #30300

Merged
merged 59 commits into from
Oct 23, 2023

Conversation

ilaner
Copy link
Contributor

@ilaner ilaner commented Oct 18, 2023

related: https://jira-hq.paloaltonetworks.local/browse/XSUP-24966

Adds support for offset in the lookback.
In case of offset, we will use the old time, and set the offset parameter to the new offset.

daryakoval and others added 30 commits September 3, 2023 16:34
@ilaner ilaner requested review from yaakovpraisler and yuvalbenshalom and removed request for yaakovpraisler October 18, 2023 14:23
@ilaner
Copy link
Contributor Author

ilaner commented Oct 19, 2023

The commonserverpython changes is on this PR:
#30300

The changes here will be reverted once this PR is merged

Copy link
Contributor

@yaakovpraisler yaakovpraisler left a comment

Choose a reason for hiding this comment

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

Nice!

Packs/Base/ReleaseNotes/1_32_38.md Outdated Show resolved Hide resolved
@yuvalbenshalom yuvalbenshalom merged commit d29ee11 into master Oct 23, 2023
13 checks passed
@yuvalbenshalom yuvalbenshalom deleted the add_offset_support_lookback branch October 23, 2023 15:17
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* fixed the parameter that send as a limit

* update rn

* update test playbook

* Update Packs/CrowdStrikeFalcon/ReleaseNotes/1_11_7.md

Co-authored-by: Shelly Tzohar <[email protected]>

* fixing test playbook

* adding sort incidents by the ids order; fix time field issue

* rename rn

* bump version

* added unitest

* fix rn

* save unitest fix

* save format

* save unitest fix

* update docker

* use created timestamp

* start_time

* RN

* Update Packs/CrowdStrikeFalcon/Integrations/CrowdStrikeFalcon/CrowdStrikeFalcon.py

Co-authored-by: yuvalbenshalom <[email protected]>

* CR

* typo

* fixes

* fixes

* fixes

* fixes

* sort by created

* fixes

* fixes

* simplify

* back to offset

* fix offset

* remove sort

* fixes

* RN

* RN

* fixes

* adding tests

* update

* fixes

* fix

* fix

* fix

* add test with offset

* add freeze time

---------

Co-authored-by: daryakoval <[email protected]>
Co-authored-by: Darya Koval <[email protected]>
Co-authored-by: Shelly Tzohar <[email protected]>
Co-authored-by: yuvalbenshalom <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants