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

Updated the poll interval to reduce delay and handle throttle #375

Merged
merged 2 commits into from
Jul 5, 2024

Conversation

rakhimundhada15
Copy link
Contributor

Problem Description

The customer is experiencing two delays in collecting Cisco Duo logs.

Solution Description

Reduce the poll interval from 4 minutes (previously based on the number of streams) to 1 minute.
Change the condition to catch up if the collection delay is greater than or equal to 1 hour. pr

Acceptance Criteria for Contributors

According to the Cisco duo api doc it is recommended to make only one API call per minute. Therefore, we cannot catch up quickly, and there might be a delay of up to 1 hour, but not more than that.
image

@al-cibot
Copy link

al-cibot commented Jul 3, 2024

CI Service Job Number 1021 Logs - SUCCESS

alps ci job-logs --organization alertlogic --repository paws-collector --job-num 1021 -g

Stages Run:

  • PR Test

@al-cibot
Copy link

al-cibot commented Jul 3, 2024

CI Service Job Number 1022 Logs - SUCCESS

alps ci job-logs --organization alertlogic --repository paws-collector --job-num 1022 -g

Stages Run:

  • PR Test

@al-cibot
Copy link

al-cibot commented Jul 5, 2024

CI Service Job Number 1023 Logs - SUCCESS

alps ci job-logs --organization alertlogic --repository paws-collector --job-num 1023 -g

Stages Run:

  • PR Test

@rakhimundhada15
Copy link
Contributor Author

build-collector-ciscoduo

@al-cibot
Copy link

al-cibot commented Jul 5, 2024

CI Service Job Number 1024 Logs - SUCCESS

alps ci job-logs --organization alertlogic --repository paws-collector --job-num 1024 -g

Stages Run:

  • build-collector-ciscoduo
  • file_packager_eec087

@rakhimundhada15 rakhimundhada15 merged commit f6c9a7b into alertlogic:master Jul 5, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants