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

feat(eda): add ability to start stream from the latest event #552

Merged
merged 6 commits into from
Sep 1, 2024

Conversation

carlosmmatos
Copy link
Contributor

Fixes #526
Fixes #549

This option is mutually exclusive with offset, and if selected, will allow the event stream to always start from the latest event. This is useful for not having to manage the offset.

This PR also fixes some issues with handling timeout due to refresh interval.

Added additional buffer time to the aiohttp.ClientTimeout to not collide
with the same timeout interval set for refreshing the stream.

Also added some better exception handling surrounding potential timeout
issues.
Fixes CrowdStrike#526
Fixes CrowdStrike#549

This option is mutually exclusive with offset, and if selected, will
allow the event stream to always start from the latest event. This is
useful for not having to manage the offset.
@carlosmmatos carlosmmatos added bugfixes Bug fixes or minor enhancements minor_changes New features, like plugin or module options eda Event driven ansible work labels Aug 31, 2024
@carlosmmatos carlosmmatos added this to the 4.6.0 milestone Aug 31, 2024
@carlosmmatos carlosmmatos self-assigned this Aug 31, 2024
redhatrises
redhatrises previously approved these changes Aug 31, 2024
redhatrises
redhatrises previously approved these changes Aug 31, 2024
@carlosmmatos carlosmmatos marked this pull request as ready for review September 1, 2024 03:06
redhatrises
redhatrises previously approved these changes Sep 1, 2024
@carlosmmatos carlosmmatos merged commit e895fc1 into CrowdStrike:main Sep 1, 2024
3 checks passed
@carlosmmatos carlosmmatos deleted the eda-updates branch September 1, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfixes Bug fixes or minor enhancements eda Event driven ansible work minor_changes New features, like plugin or module options
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EDA: support starting at latest Event Streams EDA possible enhancement
2 participants