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

Wazuh Handler: Stop agent/s ungracefully #61

Closed
pro-akim opened this issue Jun 16, 2023 · 11 comments · Fixed by #72
Closed

Wazuh Handler: Stop agent/s ungracefully #61

pro-akim opened this issue Jun 16, 2023 · 11 comments · Fixed by #72

Comments

@pro-akim
Copy link
Member

pro-akim commented Jun 16, 2023

Description

Develop for qa-system-framework in WazuhHandler

  • Add to stop_agent a parameter that allows ungracefully stop an agent.
  • Add to stop_agents a parameter that allows ungracefully stop agents through a list of agents.
    The latter should consider the use of multithreading methods.
@pro-akim
Copy link
Member Author

28/06/2023

Methods stop_agent and stop_agents were changed adding a parameter "type" where default option is "gracefully", but it is possible to do it ungracefully

@pro-akim pro-akim linked a pull request Jun 28, 2023 that will close this issue
@pro-akim
Copy link
Member Author

pro-akim commented Jul 6, 2023

06/07/2023

Retesting was initiated but put on hold due to higher priority tasks (workload testing in release).

@pro-akim
Copy link
Member Author

pro-akim commented Jul 13, 2023

Update

Review checked, Starting fixes.

@pro-akim
Copy link
Member Author

pro-akim commented Jul 13, 2023

Update

Changes done.
Starting with tests before pending review.

pro-akim added a commit that referenced this issue Jul 13, 2023
@pro-akim
Copy link
Member Author

pro-akim commented Jul 13, 2023

Update

Test performed OK.
Changing to pending review.

@pro-akim
Copy link
Member Author

Update

Starting fixes after Review

pro-akim added a commit that referenced this issue Jul 28, 2023
@pro-akim
Copy link
Member Author

Update

Moved to pending review.
Changes done

pro-akim added a commit that referenced this issue Nov 15, 2023
@pro-akim
Copy link
Member Author

Update

Conflicts were solved.

@Rebits
Copy link
Member

Rebits commented Nov 21, 2023

Delayed ETA to 22, in order to allow a review of the applied changes

@pro-akim
Copy link
Member Author

Update

Required changes done

@davidjiglesias
Copy link
Member

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants