-
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
Integrity sync test flaky behavior #4368
Comments
UpdateMoved to framework team. |
UpdateThis issue could be a duplicate of wazuh/wazuh#18123. There is one extra test failing in the artifacts uploaded here ( |
Issue UpdateAfter running the test multiple times, it was found that the error is consistent and the message is always the same:
The error occurs on the test_zip_size_limit. After reviewing the cluster.log of all components no error messages are found and after debugging the test it was found that the error occurs on the HostMonitor class when executing the run method. And After debugging the Run method it was found that the Timeout occurs because it can't find an available handler for the file monitor, the loop gets stuck there until a timeout occurs. The next step would be to replicate the behavior of the test locally to ensure where is the error if it's an error on the test or the cluster. |
Issue UpdateAfter further debugging the tests it was confirmed that the Default config found inside
During the tests:
As defined in the test config:
And trying to find the expected message
The cancel_task is nowhere to be found meaning the signal is not being properly sent. |
Issue UpdateAfter further testing and debugging, it was found that the test fails due to the sync succeding instead of failing as the test expects, this can happen because of the re Increasing the configuration max size limit from 50 MB --> 100 MB and increasing the number of files created from 5 --> 10. the timeout occurs and the
|
* 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
During the Pre-Alpha 1 system tests, it was found that the
test_cluster/test_integrity_sync/test_integrity_sync.py
system test for theagentless_cluster
environment has a flaky behavior.Sometimes a timeout error appears, and in other cases, a file that should be created is missing.
Evidences
integrity_sync_reports.zip
The text was updated successfully, but these errors were encountered: