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.

Sign up for GitHub

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

Jump to bottom

pelias elastic stats has the default index hardcoded #371

Open
arnesetzer opened this issue Mar 10, 2025 · 2 comments · May be fixed by #372 or pelias/schema#507
Open

pelias elastic stats has the default index hardcoded #371

arnesetzer opened this issue Mar 10, 2025 · 2 comments · May be fixed by #372 or pelias/schema#507
Labels
bug Something isn't working

Comments

@arnesetzer
Copy link

arnesetzer commented Mar 10, 2025 •
edited
Loading

Describe the bug
Can be interpreted as bug or as feature request. In

docker/cmd/elastic.sh

Line 65 in d219ed3

curl -s "http://${ELASTIC_HOST:-localhost:9200}/pelias/_search?request_cache=true&timeout=10s&pretty=true" \
the indexName is hardcoded which can lead to problems when you use a different indexName

Steps to Reproduce

  1. Modify the pelias.json like this:
"schema": {
    "indexName": "everything except pelias"
},
"api": {
    "indexName": "everything except pelias"
}
  1. Import some data
  2. Run pelias elastic stats
    => index_not_found_exception

Expected behavior
Use the index which is used to serve the API or the index which is filled from the import scripts.

Environment (please complete the following information):

OS: Windows 11 with WSL 2.4.10.0/Debian 12 Kernel Version 5.15.167
Docker: 28.0.1 , build 068a01e
Docker compose: v2.33.1

Pastebin/Screenshots

Additional context

References
@arnesetzer arnesetzer added the bug Something isn't working label Mar 10, 2025
@arnesetzer arnesetzer linked a pull request Mar 10, 2025 that will close this issue
Open
@missinglink
Copy link
Member

missinglink commented Mar 10, 2025 •
edited
Loading

I wonder how many other places this is hard-coded 🤔

What's the use-case here? I guess you would like to name it something else, maybe to have versioned indices in the same ES machine?

@arnesetzer
Copy link
Author

arnesetzer commented Mar 10, 2025 •
edited by missinglink
Loading

I wonder how many other places this is hard-coded 🤔

I didn't found any other case in this repo. But I wouldn't bet on it.

What's the use-case here? I guess you would like to name it something else, maybe to have versioned indices in the same ES machine?

More or less. Basically stumbled over it while experimenting with different indices for updating the data.

@arnesetzer arnesetzer linked a pull request Mar 10, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@missinglink @arnesetzer