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

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

Open
mesemus opened this issue Jan 6, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@mesemus
Copy link

mesemus commented Jan 6, 2023

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.

@mesemus mesemus added the bug Something isn't working label Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant