search

inveniosoftware / invenio-cli

CLI module for Invenio
https://invenio-cli.readthedocs.io
MIT License
10 stars 43 forks source link

invenio-cli services status does not use invenio.cfg for opensearch host/port #337

Open mesemus opened 1 year ago

mesemus commented 1 year ago

Package version (if known): 1.0.13

Describe the bug

Invenio-cli checks connection to opensearch via the following lines:

source code

        return run_cmd(
            ["curl", "-f", "localhost:9200/_cluster/health?wait_for_status=yellow"]
        )

If elasticsearch runs on different host or port, this check fails.

Steps to Reproduce

  1. Install RDM site
  2. Change port mapping in docker-services.yaml and set opensearch local port to a different value
  3. Run invenio-cli services status
  4. This will fail

Expected behavior

The host, port and authorization info should be read from the invenio.cfg.

I can make a pull request (using the requests module which is already in the dependencies) if it helps.