-
Notifications
You must be signed in to change notification settings - Fork 32
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
Unexpected errors and warnings in the vulnerability detection E2E tests #5698
Comments
We can check that all these warnings/errors can be splitter to: Unexpected connection error of
|
|
On hold due to no macOS are available: wazuh/wazuh#25345 |
In order to test this development along with #5699 it was created the branch I am currently testing over 4.9.0-rc2. |
A new build was launched to include newer changes in #5699 (comment) |
ConclusionThe current approach results in constant updates of non-critical warnings in the expected list. To address this, we've decided to remove warning messages from the unexpected lists, reducing false positive errors related to manager connections, networking, and similar issues. Additionally, a delay was introduced between the startup of the manager and the agents. This ensures the manager has time to fully initialize before the agents start, thereby preventing potential connection errors. |
LGTM |
Waiting until #5715 to tests changes in the same build |
Report analysisThe evidence can be found at #5698 (comment) Cluster connection error before the test startedWe can notice in the manager2 errors in authd and db daemons related to the registrations of some agents:
This behavior has led to agent2 not being registered, making failing the initial tests. However, we can see that the agent in the rest of the tests registers with the manager and its vulnerabilities are being detected correctly. These errors appear before the tests start, during the provision stages. A similar error was reported in wazuh/wazuh#20915, wazuh/wazuh#22855 and wazuh/wazuh#16464. This race condition was [discarded](see wazuh/wazuh#22855 (comment)) due to the low impact (agent finally registered). Regarding Vulnerability E2E, it was created an issue to mitigate this issue #5746 Missing vulnerabilitiesKnown issues specified in wazuh/wazuh#25506 |
#5746 should not be necessary, since what is intended to be fixed in the new issue are the same connection errors reported in the main comment, so this should be fixed in this issue itself. |
In the provisioning trace, we can see the following error that could specifically explain the errors regarding the cluster
It seems that the retry in the provisioning could lead to an unstable state where is more probable that unexpected errors or warnings like the specified in the issue occur. I suggest including a retry logic in this task. |
Build after applying changes in provisioning methods: https://ci.wazuh.info/job/Test_e2e_system/384 |
Description
Unexpected errors and warnings were reported in wazuh/wazuh#25363. It's necessary to perform a thorough research of them and include in case of they are expected in the errors to ignore dictionary in the vulnerability tests.
Evidences
Report: wazuh/wazuh#25363 (comment)
Tasks
Report unexpected errors to developer teams with a replication guideNot needed, those warnings were expectedThe text was updated successfully, but these errors were encountered: