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

Move APIs from Elasticsearch index to the config.yml file. #1645

Closed
wants to merge 53 commits into from

Conversation

adri9valle
Copy link
Contributor

@adri9valle adri9valle commented Jul 31, 2019

Hi team,

This PR implements the migration of the APIs from the .wazuh index to the config.yml and the wazuh-registry.json.

Now in the config.yml file, we'll store the APIs entries:

wazuh.hosts: 
  - api1:
      url: http://localhost
      port: 55000
      user: foo
      password: bar
  - api2:
      url: http://localhost
      port: 55000
      user: foo
      password: bar

On the other hand, the wazuh-version.json becomes into wazuh-registry.json and now contents the cluster information associated to the APIs in the config.yml in addition to the Wazuh version and the last restart, etc.

The procedure consists of several steps:

  • Migrate from the .wazuh index to the config.yml.
  • Clean the documents of the .wazuh index.
  • Set the cluster information in the wazuh-registry.json and update in this file instead of the index.
  • Read the APIs and cluster information from the config.yml and wazuh-registry.json files.

Also, all the methods which we used to save, edit, remove or check the API have been modified in order to do them work properly.

Solves: #1465

NOTE: All the progress can be seen in the issue #1465

Regards,

Adri,

@adri9valle adri9valle added type/enhancement Enhancement issue back-end Thins related to the server side labels Jul 31, 2019
@adri9valle adri9valle self-assigned this Jul 31, 2019
@adri9valle
Copy link
Contributor Author

This PR is replaced by this other one #1811

@adri9valle adri9valle closed this Sep 20, 2019
@juankaromo juankaromo deleted the apis-in-config branch January 4, 2020 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end Thins related to the server side type/enhancement Enhancement issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants