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

Allow using the hostname as node_name for managers #261

Merged
merged 3 commits into from
Oct 8, 2019
Merged

Allow using the hostname as node_name for managers #261

merged 3 commits into from
Oct 8, 2019

Conversation

JPLachance
Copy link
Contributor

Hello!

To simplify Wazuh's deployment in Kubernetes, we need the ability to set the node_name dynamically.

I tried to add the support for that in wazuh/wazuh#1611 and wazuh/wazuh#2006, but it was reverted and wazuh/wazuh#2249 is being pushed back at every release 🙂

So here is the idea:

  • Add a simple sed in the Wazuh manager configuration script to replace "to_be_replaced_by_hostname" by the hostname in ossec.conf

I'm running this in production. It works very well and I wanted to share it!

Have a nice day!

manuasir and others added 2 commits October 1, 2019 15:09
Copy link
Contributor

@jm404 jm404 left a comment

Choose a reason for hiding this comment

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

Hi @JPLachance,

I suggested a change, please have a look at it.

Best regards,

Jose

wazuh/config/00-wazuh.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@jm404 jm404 left a comment

Choose a reason for hiding this comment

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

LGTM!

@manuasir manuasir changed the base branch from master to devel October 8, 2019 10:05
@manuasir manuasir added this to To review in v3.11.x via automation Oct 8, 2019
@manuasir manuasir merged commit 35914c3 into wazuh:devel Oct 8, 2019
v3.11.x automation moved this from To review to Done Oct 8, 2019
@jm404 jm404 mentioned this pull request Dec 27, 2019
xr09 added a commit that referenced this pull request Oct 2, 2020
xr09 added a commit that referenced this pull request Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
v3.11.x
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants