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

Change Ensure that the manager version is warning version agnostic in VD E2E tests #5609

Closed
Rebits opened this issue Jul 24, 2024 · 4 comments · Fixed by #5630
Closed

Change Ensure that the manager version is warning version agnostic in VD E2E tests #5609

Rebits opened this issue Jul 24, 2024 · 4 comments · Fixed by #5630
Assignees
Labels

Comments

@Rebits
Copy link
Member

Rebits commented Jul 24, 2024

Description

It has been detected in Release 4.9.0 - Alpha 3 - Vulnerability Detection E2E tests that
Ensure that the manager version is expected warning it only expects hardcoded v4.8.0.
It's necessary to change this into an agnostic version of regex.

@hossam1522
Copy link
Member

The Ensure that the manager version is expected warning has been changed to work with any version:

It remains to check that the changes have been correctly applied and that everything works perfectly.

@hossam1522
Copy link
Member

On hold until the test is finished:

@hossam1522
Copy link
Member

The build finished without a report because of:

21:27:08  hudson.remoting.ProxyException: org.wazuh.JenkinsException: Invalid testingPytest did not launch correctly [end_to_end/test_vulnerability_detector] tests modules. Report was not created

This failure was mentioned in https://github.com/wazuh/wazuh-jenkins/issues/6246#issuecomment-1961021711 without reaching any clear conclusion.

The pipeline will be re-executed. In case the error is persistent, a new issue will be opened concerning this problem:

On hold until the test is finished.

@hossam1522
Copy link
Member

The pipeline was successfully executed and you can see from the following evidence that the problem was fixed correctly:

Summary

Before

Before the code was fixed, the results obtained in wazuh/wazuh#24809 (comment) were:
image
image

After

Now, after the code was fixed:
image
image

Errors obtained in the report other than the fixed one were reported in wazuh/wazuh#24809 (comment).

A pull request has been created:

@damarisg damarisg closed this as completed Aug 1, 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