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

Disable debug evidences for Vulnerability Detection E2E tests by default #5331

Merged
merged 6 commits into from
May 9, 2024

Conversation

Rebits
Copy link
Member

@Rebits Rebits commented May 3, 2024

Description

Following #4914, the vulnerability detection tests now gather substantial evidence across all test cases. These adjustments were made to address issues in unstable environments. However, the current abundance of evidence may lead to instability during test execution in Jenkins environments. Thus, it is advisable to disable these options by default and refrain from utilizing them when tests are launched automatically.


Testing performed

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

Validation Jenkins Local OS Commit Notes
🔴 🚫 Nothing to highlight

@Rebits Rebits self-assigned this May 3, 2024
@Rebits Rebits marked this pull request as ready for review May 3, 2024 17:20
Copy link
Member

@MARCOSD4 MARCOSD4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GJ, but remember to resolve conflicts in the changelog

MARCOSD4
MARCOSD4 previously approved these changes May 7, 2024
Copy link
Member

@MARCOSD4 MARCOSD4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@juliamagan juliamagan merged commit 5cf75fd into 4.8.0 May 9, 2024
1 of 2 checks passed
@juliamagan juliamagan deleted the fix/6445-e2e-pipeline branch May 9, 2024 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants