-
Notifications
You must be signed in to change notification settings - Fork 30
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
Full Yellow Local - Fix Remoted tests in 4.2 #1717
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
refac: Delete six redundant use cases Closes: #1579
refac: Reduce code using the connect method of the Agent class Closes: #1571
Add timeout for remoted socket to be ready in `test_request_agent_info.py`
…ommunication Increasing wait time until socket monitor is fully initialized in `test_multi_agent_protocols_communication.py`
Add timeout for the manager services to be ready in `test_active_response_send_ar.py`
…logs in remoted tests
Avoid updating the position of FileMonitor when monitoring multiples logs in remoted tests
…e case of having a pending status
Incrementing the time to wait for active status in order to handle the case of having a pending status
Shutting down the agent socket before closing to avoid malformed messages errors
…ion tests by the archives.log monitor
…onitor Replacing the MITM usage by the `archives.log` monitor in the `test_multi_agent_protocols_communication` tests
Disable inconsitent remoted tests
Remove warnings with unverified HTTPs requests
14 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Packages details
Environment
Setup local_internal_opcion.conf
Description
Find full details about all the progress and tests rounds in the main Remoted issue: #1530
Many problems related to Authd and Remoted simulator came up in the progress of fixing Remoted integration tests.
We decide to disable two tests in order to achieve the full yellow, those are:
Issue #1698 must be corrected before activating again the two tests mentioned above.
Remoted tests were one of the hardest ones to stabilize and still, much work is pending to make them work properly.
With more than 30 comments and 128 tests rounds, we will close this PR,
Tests results
@snaow