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

Inventory harvester - Technical debt - Fix QA test #28347

Open
2 of 3 tasks
pereyra-m opened this issue Feb 21, 2025 · 8 comments · May be fixed by #28392
Open
2 of 3 tasks

Inventory harvester - Technical debt - Fix QA test #28347

pereyra-m opened this issue Feb 21, 2025 · 8 comments · May be fixed by #28392
Assignees
Labels
level/task type/enhancement New feature or request

Comments

@pereyra-m
Copy link
Member

pereyra-m commented Feb 21, 2025

Description

After creating the new module workflow, the component tests that verifies the indexation to Opensearch doesn't work.

DoD

  • Find the cause and fix the QA test
  • Add more tests
  • CI Pass
@wazuhci wazuhci moved this to Triage in XDR+SIEM/Release 4.13.0 Feb 21, 2025
@wazuhci wazuhci moved this from Triage to Backlog in XDR+SIEM/Release 4.13.0 Feb 21, 2025
@fdalmaup fdalmaup added type/enhancement New feature or request level/task labels Feb 21, 2025
@pereyra-m pereyra-m self-assigned this Feb 24, 2025
@pereyra-m pereyra-m changed the title Inventory harvester - Technical debt - Fix component test Inventory harvester - Technical debt - Fix QA test Feb 24, 2025
@pereyra-m
Copy link
Member Author

Update

I start to work in the QA test, trying to get at least the first successful case.

@wazuhci wazuhci moved this from Backlog to In progress in XDR+SIEM/Release 4.13.0 Feb 25, 2025
@pereyra-m pereyra-m linked a pull request Feb 26, 2025 that will close this issue
@wazuhci wazuhci moved this from In progress to Pending review in XDR+SIEM/Release 4.13.0 Feb 26, 2025
@GabrielEValenzuela GabrielEValenzuela linked a pull request Feb 26, 2025 that will close this issue
@wazuhci wazuhci moved this from Pending review to In review in XDR+SIEM/Release 4.13.0 Feb 26, 2025
@pereyra-m
Copy link
Member Author

pereyra-m commented Feb 26, 2025

We've decided to keep adding tests in this branch. Missing tests:

  • Delete agent @pereyra-m
  • Insert package (delta/rsync) @pereyra-m
  • Insert and delete package (delta/rsync) @pereyra-m
  • Insert OS (delta/rsync) @pereyra-m
  • Insert and delete OS (delta) @pereyra-m
  • Integrity clear system (packages, processes, os, etc.) @pereyra-m
  • Integrity clear FIM (files, registry, etc.)
  • Integrity check global (we must update the tests for this particular test) @pereyra-m
  • Insert process (delta/rsync) @pereyra-m
  • Insert and delete process (delta/rsync) @pereyra-m
  • Insert file (delta/rsync)
  • Insert and delete file (delta/rsync)
  • Insert registry (delta/rsync)
  • Insert and delete registry (delta/rsync)

Sorry, something went wrong.

@pereyra-m
Copy link
Member Author

Update

I keep adding tests according to the list shared above.

@wazuhci wazuhci moved this from In review to On hold in XDR+SIEM/Release 4.13.0 Feb 26, 2025
@wazuhci wazuhci moved this from On hold to In progress in XDR+SIEM/Release 4.13.0 Feb 26, 2025
@GabrielEValenzuela
Copy link
Member

Update

I added cases for insert and delete, delta, and rsync—pending FIM integrity_clear.

@pereyra-m
Copy link
Member Author

Update

I've added more cases (clear, global, processes, etc.) and fixed some bugs found.

@pereyra-m
Copy link
Member Author

I will move the ETA so the PR can be reviewed and any suggested changes can be applied.

@GabrielEValenzuela
Copy link
Member

Update

I added the integrity check and cleared form FIM. Also, updated the script and readme co-authored-by: @pereyra-m

@wazuhci wazuhci moved this from In progress to Pending review in XDR+SIEM/Release 4.13.0 Feb 28, 2025
@wazuhci wazuhci moved this from Pending review to In review in XDR+SIEM/Release 4.13.0 Mar 5, 2025
@wazuhci wazuhci moved this from In review to Pending final review in XDR+SIEM/Release 4.13.0 Mar 6, 2025
@pereyra-m
Copy link
Member Author

Update

PR rebase and conflict solving

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level/task type/enhancement New feature or request
Projects
Status: Pending final review
Development

Successfully merging a pull request may close this issue.

3 participants