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

Fix the basic cluster system tests that cannot be executed without modifications #4617

Closed
pro-akim opened this issue Oct 11, 2023 · 4 comments · Fixed by #4656
Closed

Fix the basic cluster system tests that cannot be executed without modifications #4617

pro-akim opened this issue Oct 11, 2023 · 4 comments · Fixed by #4656
Assignees
Labels

Comments

@pro-akim
Copy link
Member

pro-akim commented Oct 11, 2023

Description

The basic cluster tests need to be reviewed, as after the execution of certain tests, the environment becomes messy, causing subsequent failures

Current behavior

False failures occur when the tests are being executed.
Therefore it should be executed in the following order and steps:

part 1
test_cluster/test_agent_key_polling/test_agent_key_polling.py
test_multigroups/test_multigroups.py

part 2
test_cluster/test_agent_files_deletion/test_agent_files_deletion.py
test_cluster/test_agent_groups/test_agent_groups_forced_change.py

part 3
test_cluster/test_agent_info_sync/agent_info_sync

Expected behavior

The environment should be cleaned and restarted between tests in order to be able to execute them without additional maneuvers.

@juliamagan
Copy link
Member

Development branch
bug/4617-clean-system-env

Changes applied and tests passed, it remains to open the PR.

@juliamagan
Copy link
Member

Update

Finally I had to spend more time on this issue, because I wanted not only to be able to launch all tests at the same time without being affected, but also to be able to launch several rounds without having to destroy the environment in between.

@pro-akim
Copy link
Member Author

pro-akim commented Oct 25, 2023

Update

Evidence of the test will be awaited. Moved to on hold.

@pro-akim
Copy link
Member Author

Review Notes

Local tests worked OK.
LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants