-
Notifications
You must be signed in to change notification settings - Fork 3
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
Awesome! #1
Comments
Overall, it seems like this should be re-done as a fork of ddev-elasticsearch, then it could be updated when ddev-elasticsearch changes. |
Hi @rfay I will also edit the readme soon. |
Maybe you can move this issue over to the new repo, thanks. |
Hi @rfay |
I recommend the tag "2", looks like their most current (will always be 2.x)? Does it work or not? Alternately, the tag "1" will always be latest 1.x. See https://hub.docker.com/r/opensearchproject/opensearch/tags Thankfully both of them have both amd64 and arm64 images. You'll be happier in the long term if you add something with a little more substance to the tests, https://github.com/sebastian-ehrling/ddev-opensearch/blob/main/tests/test.bats (I do see that ddev-elasticsearch doesn't have more than that, but maybe you can come up with a little something?) |
Regardless, when you're ready, let's get this promoted. I do think a longer-lived tag (1 or 2) will be good. |
I'd like to see this get promoted; people will be happy to have it. Right now, the tests are all failing, and have been failing as long as I can see. We don't want that happening! Also, the tests have been turned off by github; you'll want to look at the .github in the template to see how to fix that, perhaps compare against recent commits in ddev-elasticsearch. I'd sure love to see this get some love and get the tests working and enabled, thanks! I'm baffled why it doesn't show up in Edit: Turns out that the github search results are paged, and ddev never anticipated there would be so many add-ons. |
This is great - I've subscribed to everything.
Some notes:
I'm sure we'll talk more. Thanks for the initiative on this!
The text was updated successfully, but these errors were encountered: