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

Deprecate ossec-init.conf #1013

Merged
merged 5 commits into from
Feb 5, 2021
Merged

Deprecate ossec-init.conf #1013

merged 5 commits into from
Feb 5, 2021

Conversation

spothound
Copy link
Contributor

@spothound spothound commented Jan 28, 2021

Hello team,

This issue introduced required changes in our tests to remove ossec-init.conf deprecated file.

Closes #1007
Closes https://github.com/wazuh/wazuh-jenkins/issues/2243

BraulioV
BraulioV previously approved these changes Jan 29, 2021
Copy link
Contributor

@BraulioV BraulioV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the requested changes. Also, the function get_current_version is basically the same both for manager and agents but, the agent's version will fail for Solaris and macOS systems.

What I mean by this is I would love to see both functions merged in a single one and imported by the test.

tests/integration/test_wpk/test_wpk_agent.py Outdated Show resolved Hide resolved
tests/integration/test_wpk/test_wpk_manager.py Outdated Show resolved Hide resolved
@spothound spothound force-pushed the 1007-init-conf branch 5 times, most recently from 8bcdf00 to 67496df Compare February 1, 2021 10:35
deps/wazuh_testing/wazuh_testing/wazuh.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/wazuh.py Outdated Show resolved Hide resolved
deps/wazuh_testing/wazuh_testing/wazuh.py Outdated Show resolved Hide resolved
@spothound spothound force-pushed the 1007-init-conf branch 10 times, most recently from 57c90d5 to a755d0d Compare February 3, 2021 14:36
@BraulioV BraulioV self-assigned this Feb 3, 2021
BraulioV
BraulioV previously approved these changes Feb 3, 2021
@BraulioV BraulioV mentioned this pull request Feb 5, 2021
3 tasks
Copy link
Contributor

@palaciosjeremias palaciosjeremias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vikman90 vikman90 merged commit d7b03e5 into master Feb 5, 2021
@vikman90 vikman90 deleted the 1007-init-conf branch February 5, 2021 15:55
spothound pushed a commit that referenced this pull request Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate the usage of the ossec-init.conf file from the whole qa repository
4 participants