Yaml format of the config.yml file to generate certificates is not correct

[jmv74211]

Reading the documentation of 4.3.0-rc5, I have seen that to create the necessary certificates for the communication between different Wazuh components, a config.yml file is used.

Reviewing the content of this file, I have seen that it contains the following:

nodes:
  # Wazuh indexer nodes
  indexer:
    name: node-1
    ip: <indexer-node-ip>
    name: node-2
    ip: <indexer-node-ip>
    name: node-3
    ip: <indexer-node-ip>
...

However, it does not comply with the actual yaml syntax, because if we parse this we would get the following:

nodes:
  # Wazuh indexer nodes
  indexer:
    name: node-3
    ip: <indexer-node-ip>
...

The correct syntax would be as follows:

nodes:
  # Wazuh indexer nodes
  indexer:
    - name: node-1
      ip: <indexer-node-ip>
    - name: node-2
      ip: <indexer-node-ip>
    - name: node-3
      ip: <indexer-node-ip>
...

Regardless of which syntax seems to be wrong, the certificates are generated correctly. This has to be because you are using a custom parser that does not take into account that syntax error.

It is requested to correct this error to comply with the standard yaml format, or directly change the extension to another, such as .cnf ... to indicate that it does not comply with any specific format.

[DFolchA]

Update

• Change templates for config.yml
• Modify yml parser to read new format
• Found some and solved issues with the variables generated by the new config.yml
• Start testing

To Do

• Finish tests
• Check if the change affects Jenkins automated tests

[alberpilot]

Tasks

• Wazuh installation assistant config.yml modification. (Improve config.yml #1443) (approved, pending to merge)
• config.yml must be adapted to the new format wazuh-ansible#742 (branch done, pending PR and test when Improve config.yml #1443 merged).
• config.yml must be adapted to the new format wazuh-puppet#475 (approved, pending to merge)
• config.yml must be adapted to the new format wazuh-docker#609 (approved, pending to merge)
• config.yml must be adapted to the new format wazuh-kubernetes#236 (closed as not affected)
• https://github.com/wazuh/wazuh-automation/issues/786 (closed as not affected)