-
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
Test cluster worker logs order is detecting old format of logs in cluster.log #4365
Comments
Issue UpdateProbably related to #4364, Once the issue is resolved this will be worked on. |
UpdateAfter running the test again over the same artifacts, some changes were detected. From: FristE - Log type: Integrity check
E Expected logs: ['Sending zip file to master.*']
E Found log: 2023/07/25 09:03:05 DEBUG: [Worker CLUSTER-Workload_benchmarks_metrics_B274_manager_21] [Integrity check] Sending zip file. And Second:E - Log type: Agent-groups recv
E Expected logs: ['The checksum of both databases match.*Reset the attempt counter.', 'The checksum of both databases match.*', 'Checksum comparison failed. Attempt 10/10.*', 'Checksum comparison failed. Attempt .*', "The master's checksum and the worker's checksum are different. Local checksum:.*| Master checksum:.*."]
E Found log: 2023/07/25 09:25:34 DEBUG: [Worker CLUSTER-Workload_benchmarks_metrics_B274_manager_21] [Agent-groups recv] The checksum of master (0107654add85786339eba7c5fa72cee25783dd6c) and worker (4ba9e4d09c32baef1d763fd81e3cf804aa06b607) are different. Only the second error is still visible |
UpdateReplicated the error locally using the artifacts of build 274:
After changing the format of the log to match the new format, the following error occur:
This is caused by the dependency created in Lines 52 to 61 in b184c9e
In this case, the following log should only appear if the I'm still debugging the code to validate the behavior described above. |
UpdateAdded new nodes to the tree depending on the Lines 52 to 71 in 9a0d894
Results:
|
Update
Moved state to final review since the first one was moved to 'on hold' just for the checks |
* fix(#4365): Adds new logs validations for Agent-groups_recv.yaml * fix(#4635): Remove single quotes * fix(#4635): Updates log messages * fix(#4635): Adds new line at end of Agent-groups_recv.yaml * fix(#4635): Adds PR to changelog. * fix(#4365): Update to changelog * fix(#4365): Update changelog. --------- Co-authored-by: GGP1 <[email protected]> Co-authored-by: mauromalara <[email protected]>
* docs(#3786): update changelog.md * feat(#3786): new event_monitors * fix(#3786): recursive_directory_creation perms * feat(#3786): new fixture * fix(#3786): configuration imports * feat(#3786): new test module * docs(#3786): update changelog.md * style(#3786): fix indentation and whitelines * style(#3786): fix indentation * feat(#3693): add cases and configuration files * feat(#3693): add test_registry_wildcards module * feat(#3693): add new callbacks and event_monitor * docs(#3693): update changelog.md * style(#3693): fix whitelines * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * fix(#4275): modified year field in test_update_from_year * fix(#4275): update custom feeds to NVD 2.0 structure * fix(#4275): deprecate NVD update_from_year option and related changes * fix(#4275): NVD feed must be in one line * fix(#4275): more NVD feed one line fix * style(#4275): fix quoted errors in YAML file * style(#4275): added changelog entry and fixed indexing problems * Merge 4.5.2 into 4.6.0 (#4348) * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * refactor(#4344): Add space to version json * feat(#4344): add Release section --------- Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Move 4.5.0 `CHANGELOG.md` changes to 4.6.0 (#4331) * Fix registry wildcards path (#4357) * fix(#4356): fix configuration_templates path * docs(#4356): update test wazuh_min_version * fix(#3786): imports and paths * fix: delete update_from_year for nvd * Added new test to verify every check tag in configuration * feat(#3723): Adds custom AlmaLinux OVAL feed * feat(#3723): Adds AlmaLinux to test_providers vd tests * feat(#3723): Adds AlmaLinux to test_scan_results vd tests * feat(#3723): Adds AlmaLinux to test_feeds vd tests * feat(#3723): Adds AlmaLinux to the remaining vd tests description * feat(#3723): Adds AlmaLinux init configurations * style(#3723): minor fixes * style(#3723): Formatting .yaml files according to linting test * fix: renamed syscollector wmodules prefix * fix(#4336): fix flaky test. * style(#4336): add missing line * fix(#4336): fix test logic * docs: include 4382 to changelog * fix(#4231): fix canonical tests * style(#3723): Fixing formatting for AlmaLinux .yaml config file * docs: include affected component to changelog Co-Authored-By: Juan Nicolas Asselle <[email protected]> * Fix FIM framework to validate path in event correctly * docs: update changelog * docs: update changelog * refactor: rename discard cases files * feat: add cloudwatch and inspector discard regex tests and cases * docs: add changelog entry * fix(#4368): Change test and config file * docs(#4368): update changelog * Fixed error related to logs format in reliability test (#4387) * fix(#4365): Adds new logs validations for Agent-groups_recv.yaml * fix(#4635): Remove single quotes * fix(#4635): Updates log messages * fix(#4635): Adds new line at end of Agent-groups_recv.yaml * fix(#4635): Adds PR to changelog. * fix(#4365): Update to changelog * fix(#4365): Update changelog. --------- Co-authored-by: GGP1 <[email protected]> Co-authored-by: mauromalara <[email protected]> * docs: modify changelog and test cases descriptions * fix(#4423): fix NVD custom feed * Merge 4.5.2 into 4.6.0 (#4458) * refactor: bump revision * Fix package name in one_manager_agent system test environment * Add fix to changelog * Update CHANGELOG.md Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Update CHANGELOG.md Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Merge 4.5.1 into 4.5.2 (#4457) * fix: update VD validate xml test RHEL url * docs: include 4424 in changelog * fix(#4231): fix canonical tests * docs: change changelog line to include all changes * fix(#4411): Upgrading integration test dependencies for python in Mac (#4427) * docs: update changelog * docs: update changelog * docs: delete extra number sign * refactor: bump revision * Change revision to 4.5.1-rc2 (#4435) * Update Changelog --------- Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Jorge Marino <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: Julia <[email protected]> --------- Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: Miguel Verdaguer Velázquez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Jorge Marino <[email protected]> --------- Co-authored-by: Deblintrake09 <[email protected]> Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Leonardo Quiceno <[email protected]> Co-authored-by: Mateo Cervilla <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Marcel Kemp <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: Octavio Valle <[email protected]> Co-authored-by: Jose Luis Carreras Marin <[email protected]> Co-authored-by: Matias Pereyra <[email protected]> Co-authored-by: mauromalara <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Facundo Dalmau <[email protected]> Co-authored-by: Selutario <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Javier Castro <[email protected]> Co-authored-by: Federico Ramos <[email protected]> Co-authored-by: GGP1 <[email protected]> Co-authored-by: Miguel Verdaguer Velázquez <[email protected]> Co-authored-by: Jorge Marino <[email protected]>
* Merge 4.6.0 into 4.7.0 (#4349) * feat(#3693): add cases and configuration files * feat(#3693): add test_registry_wildcards module * feat(#3693): add new callbacks and event_monitor * docs(#3693): update changelog.md * style(#3693): fix whitelines * Merge 4.5.2 into 4.6.0 (#4348) * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * refactor(#4344): Add space to version json * feat(#4344): add Release section --------- Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Move 4.5.0 `CHANGELOG.md` changes to 4.6.0 (#4331) * Move changes of 4.5.0 in CHANGELOG.md --------- Co-authored-by: Deblintrake09 <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> * feat(#4045): add custom feed * feat(#4045): add test cases and configuration * feat(#4045): add test module * docs(#4045): case name and description * style(#4045): rename feed file name * style(#4045): fix description indentation * fix(#4356): fix configuration_templates path * docs(#4356): update test wazuh_min_version * refactor(#4404): update database version * refactor(#4404): updated changelog * Merge 4.6.0 into 4.7.0 (#4421) * docs(#3786): update changelog.md * feat(#3786): new event_monitors * fix(#3786): recursive_directory_creation perms * feat(#3786): new fixture * fix(#3786): configuration imports * feat(#3786): new test module * docs(#3786): update changelog.md * style(#3786): fix indentation and whitelines * style(#3786): fix indentation * feat(#3693): add cases and configuration files * feat(#3693): add test_registry_wildcards module * feat(#3693): add new callbacks and event_monitor * docs(#3693): update changelog.md * style(#3693): fix whitelines * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * fix(#4275): modified year field in test_update_from_year * fix(#4275): update custom feeds to NVD 2.0 structure * fix(#4275): deprecate NVD update_from_year option and related changes * fix(#4275): NVD feed must be in one line * fix(#4275): more NVD feed one line fix * style(#4275): fix quoted errors in YAML file * style(#4275): added changelog entry and fixed indexing problems * Merge 4.5.2 into 4.6.0 (#4348) * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * refactor(#4344): Add space to version json * feat(#4344): add Release section --------- Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Move 4.5.0 `CHANGELOG.md` changes to 4.6.0 (#4331) * Fix registry wildcards path (#4357) * fix(#4356): fix configuration_templates path * docs(#4356): update test wazuh_min_version * fix(#3786): imports and paths * fix: delete update_from_year for nvd * Added new test to verify every check tag in configuration * feat(#3723): Adds custom AlmaLinux OVAL feed * feat(#3723): Adds AlmaLinux to test_providers vd tests * feat(#3723): Adds AlmaLinux to test_scan_results vd tests * feat(#3723): Adds AlmaLinux to test_feeds vd tests * feat(#3723): Adds AlmaLinux to the remaining vd tests description * feat(#3723): Adds AlmaLinux init configurations * style(#3723): minor fixes * style(#3723): Formatting .yaml files according to linting test * fix: renamed syscollector wmodules prefix * fix(#4336): fix flaky test. * style(#4336): add missing line * fix(#4336): fix test logic * docs: include 4382 to changelog * fix(#4231): fix canonical tests * style(#3723): Fixing formatting for AlmaLinux .yaml config file * docs: include affected component to changelog Co-Authored-By: Juan Nicolas Asselle <[email protected]> * Fix FIM framework to validate path in event correctly * docs: update changelog * docs: update changelog --------- Co-authored-by: Deblintrake09 <[email protected]> Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: damarisg <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Leonardo Quiceno <[email protected]> Co-authored-by: Mateo Cervilla <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Marcel Kemp <[email protected]> Co-authored-by: Seyla Dámaris Gomez <[email protected]> Co-authored-by: Octavio Valle <[email protected]> Co-authored-by: Jose Luis Carreras Marin <[email protected]> Co-authored-by: Matias Pereyra <[email protected]> Co-authored-by: mauromalara <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Javier Castro <[email protected]> * Merge 4.6.0 into 4.7.0 (#4459) * docs(#3786): update changelog.md * feat(#3786): new event_monitors * fix(#3786): recursive_directory_creation perms * feat(#3786): new fixture * fix(#3786): configuration imports * feat(#3786): new test module * docs(#3786): update changelog.md * style(#3786): fix indentation and whitelines * style(#3786): fix indentation * feat(#3693): add cases and configuration files * feat(#3693): add test_registry_wildcards module * feat(#3693): add new callbacks and event_monitor * docs(#3693): update changelog.md * style(#3693): fix whitelines * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * fix(#4275): modified year field in test_update_from_year * fix(#4275): update custom feeds to NVD 2.0 structure * fix(#4275): deprecate NVD update_from_year option and related changes * fix(#4275): NVD feed must be in one line * fix(#4275): more NVD feed one line fix * style(#4275): fix quoted errors in YAML file * style(#4275): added changelog entry and fixed indexing problems * Merge 4.5.2 into 4.6.0 (#4348) * feat(#4281): New invalid decoder test case for wazuh-logtest * fix(#4281): Fix invalid_decoder_syntax.yaml file line lengths * feat(#4325): upgrade pyyaml to 6.0.1 * feat: bump version 4.5.2 * refactor(#4344): Add space to version json * feat(#4344): add Release section --------- Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Move 4.5.0 `CHANGELOG.md` changes to 4.6.0 (#4331) * Fix registry wildcards path (#4357) * fix(#4356): fix configuration_templates path * docs(#4356): update test wazuh_min_version * fix(#3786): imports and paths * fix: delete update_from_year for nvd * Added new test to verify every check tag in configuration * feat(#3723): Adds custom AlmaLinux OVAL feed * feat(#3723): Adds AlmaLinux to test_providers vd tests * feat(#3723): Adds AlmaLinux to test_scan_results vd tests * feat(#3723): Adds AlmaLinux to test_feeds vd tests * feat(#3723): Adds AlmaLinux to the remaining vd tests description * feat(#3723): Adds AlmaLinux init configurations * style(#3723): minor fixes * style(#3723): Formatting .yaml files according to linting test * fix: renamed syscollector wmodules prefix * fix(#4336): fix flaky test. * style(#4336): add missing line * fix(#4336): fix test logic * docs: include 4382 to changelog * fix(#4231): fix canonical tests * style(#3723): Fixing formatting for AlmaLinux .yaml config file * docs: include affected component to changelog Co-Authored-By: Juan Nicolas Asselle <[email protected]> * Fix FIM framework to validate path in event correctly * docs: update changelog * docs: update changelog * refactor: rename discard cases files * feat: add cloudwatch and inspector discard regex tests and cases * docs: add changelog entry * fix(#4368): Change test and config file * docs(#4368): update changelog * Fixed error related to logs format in reliability test (#4387) * fix(#4365): Adds new logs validations for Agent-groups_recv.yaml * fix(#4635): Remove single quotes * fix(#4635): Updates log messages * fix(#4635): Adds new line at end of Agent-groups_recv.yaml * fix(#4635): Adds PR to changelog. * fix(#4365): Update to changelog * fix(#4365): Update changelog. --------- Co-authored-by: GGP1 <[email protected]> Co-authored-by: mauromalara <[email protected]> * docs: modify changelog and test cases descriptions * fix(#4423): fix NVD custom feed * Merge 4.5.2 into 4.6.0 (#4458) * refactor: bump revision * Fix package name in one_manager_agent system test environment * Add fix to changelog * Update CHANGELOG.md Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Update CHANGELOG.md Co-authored-by: Víctor Rebollo Pérez <[email protected]> * Merge 4.5.1 into 4.5.2 (#4457) * fix: update VD validate xml test RHEL url * docs: include 4424 in changelog * fix(#4231): fix canonical tests * docs: change changelog line to include all changes * fix(#4411): Upgrading integration test dependencies for python in Mac (#4427) * docs: update changelog * docs: update changelog * docs: delete extra number sign * refactor: bump revision * Change revision to 4.5.1-rc2 (#4435) * Update Changelog --------- Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Jorge Marino <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: Julia <[email protected]> --------- Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: Miguel Verdaguer Velázquez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Jorge Marino <[email protected]> --------- Co-authored-by: Deblintrake09 <[email protected]> Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Leonardo Quiceno <[email protected]> Co-authored-by: Mateo Cervilla <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Marcel Kemp <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: Octavio Valle <[email protected]> Co-authored-by: Jose Luis Carreras Marin <[email protected]> Co-authored-by: Matias Pereyra <[email protected]> Co-authored-by: mauromalara <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Facundo Dalmau <[email protected]> Co-authored-by: Selutario <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Javier Castro <[email protected]> Co-authored-by: Federico Ramos <[email protected]> Co-authored-by: GGP1 <[email protected]> Co-authored-by: Miguel Verdaguer Velázquez <[email protected]> Co-authored-by: Jorge Marino <[email protected]> * Update Changelog --------- Co-authored-by: Deblintrake09 <[email protected]> Co-authored-by: Victor M. Fernandez-Castro <[email protected]> Co-authored-by: Vikman Fernandez-Castro <[email protected]> Co-authored-by: jnasselle <[email protected]> Co-authored-by: Julia <[email protected]> Co-authored-by: Julia Magán <[email protected]> Co-authored-by: David Jose Iglesias Lopez <[email protected]> Co-authored-by: Víctor Rebollo Pérez <[email protected]> Co-authored-by: Dwordcito <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Leonardo Quiceno <[email protected]> Co-authored-by: Mateo Cervilla <[email protected]> Co-authored-by: lsayanes <[email protected]> Co-authored-by: Marcel Kemp <[email protected]> Co-authored-by: Jose Luis Carreras Marin <[email protected]> Co-authored-by: Matias Pereyra <[email protected]> Co-authored-by: mauromalara <[email protected]> Co-authored-by: BelenValdivia <[email protected]> Co-authored-by: Javier Castro <[email protected]> Co-authored-by: Facundo Dalmau <[email protected]> Co-authored-by: Selutario <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Federico Ramos <[email protected]> Co-authored-by: GGP1 <[email protected]> Co-authored-by: Miguel Verdaguer Velázquez <[email protected]> Co-authored-by: Jorge Marino <[email protected]>
Description
Workload benchmark testing were performed for Release of v4.6.0-pre-alpha1
Running manually reliability/test_cluster/test_cluster_logs/test_cluster_worker_logs_order
/test_cluster_worker_logs_order.py::test_check_logs_order_workers over artifacts created by workload benchmark pipeline (It should be fixed #4299) it is possible to find that the log messages provided by Wazuh are different than what the test is expecting. Small format changes could be detected, however, the meaning of the messages are still the same as the old format.
Current behavior
Detected errors in the test:
Expected behavior
No errors should be detected.
The text was updated successfully, but these errors were encountered: