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.

Sign up for GitHub

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

Jump to bottom

Fix syscollector deltas integration tests #3695

Merged
merged 3 commits into from
Jan 10, 2023
Merged

Fix syscollector deltas integration tests #3695

merged 3 commits into from
Jan 10, 2023

Conversation

Dwordcito
Copy link
Member

@Dwordcito Dwordcito commented Dec 21, 2022 •
edited by jnasselle
Loading

Related issue
wazuh/wazuh#15639

Description

This PR aims to update the integration tests related to syscollector deltas, which are

Both integration tests were originally developed here #2146

Updated

  • test_analysisd/test_syscollector/test_syscollector_events.py
  • test_wazuh_db/data/agent/syscollector_deltas_messages.yaml

Package used (outdated)

Package used:

Testing performed (outdated)

Tester Test path Jenkins Local OS Commit Notes
@jnasselle (Developer) test_analysisd 🟢 🟢 🟢 🟢 🟢 🟢 Centos c6f66ae Manager
@jnasselle (Developer) test_wazuh_db 🟢 🟢 🟢 🟢 🟢 🟢 Centos c6f66ae Manager
@damarisg (Reviewer) test_analysisd 🟢 🚫 Centos c6f66ae Manager
@damarisg (Reviewer) test_wazuh_db 🟢 🚫 Centos c6f66ae Manager

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@jnasselle (Developer) test_analysisd 🟢 🟢 🟢 🔵🔵🔵 Centos c12f2fd5f69138fbe7fded9dabf721c1515f2c5c Manager
@jnasselle (Developer) test_wazuh_db 🟢 🟢 🟢 🟢 🟢 🟢 Centos c12f2fd5f69138fbe7fded9dabf721c1515f2c5c Manager

NOTE: wazuh_db is having issues on Ubuntu environments #3716

@Dwordcito Dwordcito force-pushed the 15639-syscollector-deltas-on-worker-change branch from f5aefba to ca05979 Compare December 21, 2022 04:53
@Dwordcito Dwordcito changed the title Fix analysisd delta tests. Modify analysisd syscollector deltas integration tests Dec 21, 2022
@Dwordcito Dwordcito changed the title Modify analysisd syscollector deltas integration tests Fix analysisd syscollector deltas integration tests Dec 21, 2022
@Dwordcito Dwordcito changed the title Fix analysisd syscollector deltas integration tests Fix syscollector deltas integration tests Dec 21, 2022
@Dwordcito Dwordcito force-pushed the 15639-syscollector-deltas-on-worker-change branch 2 times, most recently from fd7350d to e8d2f5a Compare December 21, 2022 05:01
@jnasselle jnasselle linked an issue Dec 21, 2022 that may be closed by this pull request
Update automated tests related to Syscollector deltas to new format #3673
Closed
@Dwordcito Dwordcito force-pushed the 15639-syscollector-deltas-on-worker-change branch from e8d2f5a to bfeead3 Compare December 21, 2022 20:16
@jnasselle jnasselle force-pushed the 15639-syscollector-deltas-on-worker-change branch 3 times, most recently from 1350711 to 8c58f8f Compare December 22, 2022 04:49
@jnasselle jnasselle force-pushed the 15639-syscollector-deltas-on-worker-change branch from 8c58f8f to c6f66ae Compare December 22, 2022 21:41
@jnasselle jnasselle mentioned this pull request Dec 29, 2022
Closed
@Dwordcito Dwordcito merged commit 5cb5c54 into 4.4 Jan 10, 2023
@Dwordcito Dwordcito deleted the 15639-syscollector-deltas-on-worker-change branch January 10, 2023 14:19
@jnasselle jnasselle mentioned this pull request Jan 10, 2023
Closed
Dwordcito added a commit that referenced this pull request Jan 12, 2023
@Dwordcito @jnasselle
Fix syscollector deltas integration tests (#3695)
92e1657 
* fix(#3673): modify analysisd delta tests.

* fix(#3673): modify wazuh-db delta tests.

* fix(#3673): Adapt test to netaddr proto

Co-authored-by: jnasselle <jnasselle@gmail.com>
Dwordcito added a commit that referenced this pull request Jan 13, 2023
@Dwordcito @jnasselle
Fix syscollector deltas integration tests (#3695) (#3748)
487a1eb 
* fix(#3673): modify analysisd delta tests.

* fix(#3673): modify wazuh-db delta tests.

* fix(#3673): Adapt test to netaddr proto

Co-authored-by: jnasselle <jnasselle@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jnasselle jnasselle jnasselle approved these changes

@damarisg damarisg damarisg approved these changes

Assignees

@jnasselle jnasselle

@Dwordcito Dwordcito

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update automated tests related to Syscollector deltas to new format
3 participants
@Dwordcito @jnasselle @damarisg