-
Notifications
You must be signed in to change notification settings - Fork 5
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
Improve asistant test completion status #188
Merged
c-bordon
merged 21 commits into
main
from
enhancement/47-improve-asistant-test-completion-states
Dec 31, 2024
Merged
Improve asistant test completion status #188
c-bordon
merged 21 commits into
main
from
enhancement/47-improve-asistant-test-completion-states
Dec 31, 2024
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
3 tasks
c-bordon
approved these changes
Dec 31, 2024
c-bordon
deleted the
enhancement/47-improve-asistant-test-completion-states
branch
December 31, 2024 11:19
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.
Closes #47
Description
The objective of this PR is to improve the test completion status.
The solution was implemented where if the test of the Wazuh installation tools fails, the entire test is reported as failed. On the other hand, if only the test of the Wazuh components fails, the test is reported as successful (since the objective of the test is to validate the installation tools and not the functionality of Wazuh). Still, warnings are left in the component testing step, for later information to the responsible team.
Tests
Installation Wizard (Distributed)
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12518212911/job/34920285045
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12518291928/job/34920498369
Bug fixed in the installation wizard
All test works successfully
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12518596246/job/34921311838
Component test fail - Test installation passed
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12518849129/job/34921959360
Test AIO
Stop execution of steps on failures:
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12519625965/job/34923983400
Bug fixed in the installation wizard
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12520456213/job/34926149965
All test works successfully
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12520456213/job/34926149965
Component test fail - Test installation passed
https://github.com/wazuh/wazuh-installation-assistant/actions/runs/12548207736/job/34987047896