wazuh / qa-system-framework

GNU General Public License v2.0
1 stars 3 forks source link

Use inventory OS information instead of parameters it in the HostManager methods #40

Closed pro-akim closed 1 year ago

pro-akim commented 1 year ago
Related issue
#38

Description

This PR aims to add change HostManager methods using Os information from inventory.yml instead of paraters.

Updated

Methods that have changes in the parameterization of Os and/or become:

Methods that have changes in the parameterization of become:

Testing performed

Tester Test path Jenkins Local OS Commit Notes
@pro-akim (Developer) :green_circle: Windows - Agent https://github.com/wazuh/qa-system-framework/pull/40/commits/9a4eb261691f5db512fd43b589d3fc44e8564959
pro-akim commented 1 year ago

All methods have been reviewed, and those that receive the 'become' parameter have been configured to default to None. In the case that a value is provided, that value will be taken as the parameter. If no value is provided, the method will fetch that value from the inventory using the method get_host_variables(host), taking the value with the index ['become']. If the value is null or None, it will be set to the default value of False.

Report has been updated

pro-akim commented 1 year ago

After Changes in https://github.com/wazuh/qa-system-framework/commit/23bf01df9520cf48f6b0c35baf6bcdb108c2b559, test were performed. Tests report updated.