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

Increasing wait time until socket monitor is fully initialized in test_multi_agent_protocols_communication.py #1625

Conversation

DProvinciani
Copy link
Contributor

Related issue
Closes: #1618

Description

This PR modifies the test_multi_agent_protocols_communication.py of remoted tests to increase the time that the test waits until having the socket monitor is fully initialized.

In addition, this PR updates the documentation to reflect the increase of time duration.

Configuration options

All the tests are run with the default test configuration and the following options in local_internal_options.conf

remoted.debug=2
wazuh_database.interval=1
wazuh_db.commit_time=2
wazuh_db.commit_time_max=3
monitord.rotate_log=0

Tests

The comments will have the description for every test run

  • Proven that tests pass when they have to pass.

@DProvinciani DProvinciani self-assigned this Jul 21, 2021
@DProvinciani
Copy link
Contributor Author

Test Execution - Results

Description

Test results with the configuration specified by the test in tests/integration/test_remoted/test_agent_communication/data/wazuh_multi_agent_protocols_communication.yaml:

Test Executions Date By Status
test_multi_agent_protocols_communication-r1.log 2021-07-21 Diego 🟡
test_multi_agent_protocols_communication-r2.log 2021-07-21 Diego 🟡
test_multi_agent_protocols_communication-r3.log 2021-07-21 Diego 🟡

@mdengra
Copy link
Contributor

mdengra commented Jul 22, 2021

2021-07-22

Test results

Test Executions Date By Status
test_multi_agent_protocols_communication_local_r1.log 2021-07-22 Miguel 🟡
test_multi_agent_protocols_communication_local_r2.log 2021-07-22 Miguel 🟡
test_multi_agent_protocols_communication_local_r3.log 2021-07-22 Miguel 🟡

Copy link
Contributor

@mdengra mdengra left a comment

Choose a reason for hiding this comment

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

Great job!

@damarisg
Copy link
Member

Test Results

Test Executions Date By Status
RemotedMultiAgentProtocolCom.log 2021-07-22 Seyla 🟡
RemotedMultiAgentProtocolCom2.log 2021-07-22 Seyla 🟡
RemotedMultiAgentProtocolCom3.log 2021-07-22 Seyla 🟡

@DProvinciani DProvinciani changed the base branch from 1516-4.2.0-full-green to 1530-full-yellow-remoted July 22, 2021 15:10
@damarisg damarisg merged commit 8f73b1d into 1530-full-yellow-remoted Jul 26, 2021
@damarisg damarisg deleted the 1618-test-multi-agent-protocols-communication branch July 26, 2021 12:24
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.

None yet

3 participants