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

Include Agent key already in use warning message in expected E2E expected errors/warnings #5403

Closed
2 tasks
Rebits opened this issue May 20, 2024 · 3 comments · Fixed by #5409
Closed
2 tasks
Assignees
Labels

Comments

@Rebits
Copy link
Member

Rebits commented May 20, 2024

Description

It has been detected in #5397, that initial tests detect an unexpected warning in managers:

            "2024/05/17 18:12:03 wazuh-remoted: WARNING: Agent key already in use: agent ID '005'"

This warning is expected due to the logic of Vulnerability Detection E2E tests, were agents are removed before and after configuring the Vulnerability Detection module depending on the test case. This agent removal produces these expected warnings
We should include this warning in the expected error list.

Tasks

  • Include Agent key already in use warning into the expected E2E errors list.

Validation

  • E2E Vulnerability detection initial scans no longer detect this warning as unexpected.
@Rebits Rebits self-assigned this May 20, 2024
@wazuhci wazuhci moved this to Triage in Release 4.8.0 May 20, 2024
@wazuhci wazuhci moved this from Triage to In progress in Release 4.8.0 May 20, 2024
@Rebits
Copy link
Member Author

Rebits commented May 20, 2024

Currently testing among different E2E fixes. Check #5397 for more information

@Rebits
Copy link
Member Author

Rebits commented May 21, 2024

Created custom branch to test all E2E fixes tmp-merge-fixes-E2E-tests

Build: https://ci.wazuh.info/job/Test_e2e_system/289/

@wazuhci wazuhci moved this from In progress to On hold in Release 4.8.0 May 21, 2024
@wazuhci wazuhci moved this from On hold to In progress in Release 4.8.0 May 22, 2024
@wazuhci wazuhci moved this from In progress to Pending review in Release 4.8.0 May 22, 2024
@rafabailon
Copy link
Member

LGTM

@wazuhci wazuhci moved this from Pending review to Pending final review in Release 4.8.0 May 22, 2024
@wazuhci wazuhci moved this from Pending final review to Done in Release 4.8.0 May 23, 2024
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
3 participants