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

Indexer v1 package SPECs #1226

Merged
merged 58 commits into from
Feb 3, 2022
Merged

Indexer v1 package SPECs #1226

merged 58 commits into from
Feb 3, 2022

Conversation

okynos
Copy link
Contributor

@okynos okynos commented Feb 1, 2022

Related issue
#1099

Description

Hello team,

We want to merge this branch into big-product branch
We have included:

  • Indexer Debian build scripts
  • Indexer RPM build scripts
  • Debian 4.3.0 indexer SPECs
  • RPM 4.3.0 indexer SPECs
  • Improved file permissions on both systems.
  • Improved installation procedure.
  • Added default roles and users to indexer package from unattended development.
  • Added wazuh-cert-tool and wazuh-passwords-tool
  • A lot more...

Tests

  • Build the package in any supported platform
    • Linux
    • Package installation
    • Package remove
  • Tests for Linux RPM
    • Build the package for x86_64
    • %files section is correctly updated if necessary
  • Tests for Linux deb
    • Build the package for x86_64
    • Check file permissions after installing the package

okynos and others added 30 commits December 16, 2021 17:56
@okynos okynos requested review from alberpilot and verdx February 1, 2022 11:32
@okynos okynos self-assigned this Feb 1, 2022
Copy link
Contributor

@verdx verdx left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@alberpilot alberpilot left a comment

Choose a reason for hiding this comment

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

  • Change the certificate names and configuration files to demo name, as Opendistro.
  • The config.yml by default must include (commented) 3 elasticsearch nodes in total. The node names must be node-1, node-2 and node-3. Like this:
nodes:
  # Elasticsearch server nodes
  elasticsearch:
    name: node-1
    ip: <elasticsearch-node-ip>
    # name: <elasticsearch-node-name>
    # ip: <elasticsearch-node-ip>
    # name: <elasticsearch-node-name>
    # ip: <elasticsearch-node-ip>

  # Wazuh server nodes
  # Use node_type only with more than one Wazuh manager
  wazuh_servers:
    name: wazuh-server
    ip: <wazuh-manager-ip>
    # node_type: master
    # name: <wazuh-node-name>
    # ip: <wazuh-manager-ip>
    # node_type: worker

  # Kibana node
  kibana:
    name: wazuh-dashboard
    ip: <kibana-node-ip>
  • Please take into account the inline comments.

indexer/rpm/wazuh-indexer.spec Outdated Show resolved Hide resolved
indexer/rpm/wazuh-indexer.spec Outdated Show resolved Hide resolved
stack/indexer/deb/debian/postinst Outdated Show resolved Hide resolved
stack/indexer/deb/debian/postinst Outdated Show resolved Hide resolved
indexer/rpm/wazuh-indexer.spec Outdated Show resolved Hide resolved
indexer/rpm/wazuh-indexer.spec Outdated Show resolved Hide resolved
stack/indexer/rpm/wazuh-indexer.spec Outdated Show resolved Hide resolved
stack/indexer/securityadmin_wrapper.sh Show resolved Hide resolved
stack/indexer/securityadmin_wrapper.sh Show resolved Hide resolved
stack/indexer/securityadmin_wrapper.sh Show resolved Hide resolved
@alberpilot alberpilot merged commit 42f394d into big-product Feb 3, 2022
@alberpilot alberpilot deleted the poc-indexer branch February 3, 2022 11:47
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.

4 participants