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

Alerts are not being filtered by timestamp #5108

Closed
Tracked by #5183
juliamagan opened this issue Mar 14, 2024 · 4 comments · Fixed by #5157
Closed
Tracked by #5183

Alerts are not being filtered by timestamp #5108

juliamagan opened this issue Mar 14, 2024 · 4 comments · Fixed by #5157
Assignees
Labels

Comments

@juliamagan
Copy link
Member

Description

During the tests executed for 4.8.0 Beta 4, in wazuh/wazuh#22493, it has been observed that alerts are not being filtered by timestamp, since in different tests appear alerts from previous tests. We should adjust the code so that the alerts are filtered correctly with the timestamp, reducing test time and resources.

Report: Test_e2e_system_244_test_vulnerability_detector.zip

@wazuhci wazuhci moved this to Triage in Release 4.8.0 Mar 14, 2024
@wazuhci wazuhci moved this from Triage to Backlog in Release 4.8.0 Mar 18, 2024
@juliamagan juliamagan self-assigned this Mar 25, 2024
@wazuhci wazuhci moved this from Backlog to In progress in Release 4.8.0 Mar 25, 2024
@juliamagan
Copy link
Member Author

Development branch
bug/5108-filter-timestamp-alerts

Fix applied. It remains to open the PR.

@juliamagan juliamagan linked a pull request Mar 27, 2024 that will close this issue
@juliamagan
Copy link
Member Author

During the execution of the tests to attach the results, other problems have been encountered and we are solving them in this issue.

  • We have fixed a Grafana package wrong with the expected version.
  • Added not to ignore timeout errors in waiters.
  • Fixed a regex
  • Removed extra steps in the setup

With this last one, there are some problems when obtaining the vulnerability index of registered agents, it remains to fix this.

@wazuhci wazuhci moved this from In progress to Pending review in Release 4.8.0 Apr 3, 2024
@wazuhci wazuhci moved this from Pending review to In review in Release 4.8.0 Apr 3, 2024
@rafabailon
Copy link
Member

LGTM

@wazuhci wazuhci moved this from In review to Pending final review in Release 4.8.0 Apr 3, 2024
@wazuhci wazuhci moved this from Pending final review to Pending review in Release 4.8.0 Apr 3, 2024
@wazuhci wazuhci moved this from Pending review to In review in Release 4.8.0 Apr 4, 2024
@davidjiglesias
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants