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.

Sign up for GitHub

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

Jump to bottom

Optimized manager role #320

Merged
merged 6 commits into from
Nov 11, 2019
Merged

Optimized manager role #320

merged 6 commits into from
Nov 11, 2019
+69 −70

Conversation

xr09
Copy link
Contributor

@xr09 xr09 commented Nov 11, 2019 •
edited
Loading

This PR fixes #317, the changes were:

  • OS specific tasks were moved to its own file
  • Switched import_tasks to include_tasks to evaluate os-conditionals only once per include. This also reduces output noise. Ansible docs on this subject.
  • Removed deprecation warnings on apt module

@xr09 xr09 requested a review from manuasir November 11, 2019 15:03
@manuasir manuasir changed the title Feature 317 optimize manager role Optimized manager role Nov 11, 2019
@manuasir manuasir merged commit ad54aa4 into devel Nov 11, 2019
@manuasir manuasir deleted the feature-317-optimize-manager-role branch November 11, 2019 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manuasir manuasir manuasir approved these changes

Assignees
No one assigned
Labels
wazuhci/runtests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@xr09 @manuasir