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

FIM and Syscollector IT fails for Windows agent due to error deleting old db #22496

Closed
Rebits opened this issue Mar 13, 2024 · 7 comments
Closed
Assignees
Labels
level/task qa_known Issues that are already known by the QA team type/bug Something isn't working

Comments

@Rebits
Copy link
Member

Rebits commented Mar 13, 2024

Wazuh version Component Install type Install method Platform
4.8.0-beta4 FIM/Sycollector Agent Packages Windows Server 2016

Description

FIM and syscollector integration tests fail for Windows agents due to an unexpected error deleting old db.

2024/03/12 18:19:48 wazuh-agent[2908] logging_helper.c:53 at loggingFunction(): CRITICAL: Error, id: dbEngine: Error deleting old db.

The roster of IT failures includes:

  • test_file_limit_full
  • test_file_limit_no_limit
  • test_syscollector_configuration

Related Issues

Evidences

@Rebits Rebits added type/bug Something isn't working level/task labels Mar 13, 2024
@wazuhci wazuhci moved this to Triage in Release 4.8.0 Mar 13, 2024
@wazuhci wazuhci moved this from Triage to Backlog in Release 4.8.0 Mar 13, 2024
@wazuhci wazuhci moved this from Backlog to Triage in Release 4.8.0 Mar 13, 2024
@wazuhci wazuhci moved this from Triage to Backlog in Release 4.8.0 Mar 15, 2024
@Dwordcito Dwordcito removed their assignment Mar 15, 2024
@Dwordcito
Copy link
Member

TIMEBOX RCA 19/03/2024

@MiguelazoDS MiguelazoDS self-assigned this Apr 3, 2024
@wazuhci wazuhci moved this from Backlog to In progress in Release 4.8.0 Apr 3, 2024
@MiguelazoDS
Copy link
Member

The issue may be related to this

https://en.cppreference.com/w/cpp/io/c/remove

If the file is currently open by any process, the behavior of this function is implementation-defined. POSIX systems unlink the file name (directory entry), but the filesystem space used by the file is not reclaimed while it is open in any process and while other hard links to the file exist. Windows does not allow the file to be deleted in such cases.

@Dwordcito
Copy link
Member

@MiguelazoDS this is a flaky test ??? Is reproducible in previous versions?

@MiguelazoDS
Copy link
Member

@Dwordcito This was executed with Jenkins, I'm trying to reproduce it locally to discard that possibility, but if I have to guess, I don't think it's a flaky test. I have to take a deeper look at these tests but some handle must be preventing the delete operation for Windows

@MiguelazoDS
Copy link
Member

MiguelazoDS commented Apr 4, 2024

Update 04/04

image

Unable to reproduce the Syscollector errors

syscollector.tar.gz

image

Unable to reproduce the FIM errors

fim.tar.gz

image

Conclusion

I was not able to reproduce this behavior by running the tests several times, and this seems to be an old issue addressed here for 4.6.0 #18968

@Rebits are you able to reproduce this locally?

A new run on the environment that triggered the failing test must be performed.

@MiguelazoDS MiguelazoDS linked a pull request Apr 4, 2024 that will close this issue
5 tasks
@wazuhci wazuhci moved this from In progress to Pending review in Release 4.8.0 Apr 4, 2024
@wazuhci wazuhci moved this from Pending review to Blocked in Release 4.8.0 Apr 4, 2024
@wazuhci wazuhci removed this from Release 4.8.0 Apr 4, 2024
@Dwordcito
Copy link
Member

Moved to 4.8.2, this part of the code has not changed in 4.8.0, is a pre-existing issue.

@Rebits Rebits added the qa_known Issues that are already known by the QA team label Apr 5, 2024
@sebasfalcone
Copy link
Contributor

We conclude that this issue has to do with the test itself and not with the product, as stated here:

And also concluded here:

The issue may be addressed here:

@sebasfalcone sebasfalcone closed this as not planned Won't fix, can't repro, duplicate, stale Apr 5, 2024
@wazuhci wazuhci moved this to Done in Release 4.9.0 Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level/task qa_known Issues that are already known by the QA team type/bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants