feat: docker-compose with opensearch #768
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
yevhen-cap
commented
Aug 9, 2024
yevhen-cap
commented
- core
- camunda platform
- core - camunda platform
|
|
igpetrov
reviewed
Aug 9, 2024
- core - camunda platform
igpetrov
force-pushed
the
platform-opensearch
branch
from
5a7c2c2
to
f510c62
Compare
igpetrov
force-pushed
the
platform-opensearch
branch
3 times, most recently
from
84d3734
to
50a8d7c
Compare
igpetrov
force-pushed
the
platform-opensearch
branch
from
50a8d7c
to
8ebc229
Compare
|
Hi @yevhen-cap It seems like OpenSearch cluster never turns green and always stays yellow. Can you please take a look? |
igpetrov
reviewed
Aug 15, 2024
igpetrov
reviewed
Aug 15, 2024
Likely related to this opensearch-project/security-analytics#1352 |
…form into platform-opensearch
…form into platform-opensearch
yevhen-cap
force-pushed
the
platform-opensearch
branch
2 times, most recently
from
f17b516
to
8c980ff
Compare
- health check workaround
yevhen-cap
force-pushed
the
platform-opensearch
branch
from
8c980ff
to
ed78e98
Compare
aabouzaid
requested changes
Aug 22, 2024
yevhen-cap
force-pushed
the
platform-opensearch
branch
from
c5e87b0
to
ed78e98
Compare
* initial unified compose file * removed ssl * remove healthcheck * removed test data * fix: core changes - added .env configs - removed sh file - unified docker-compose.yml and docker-compose-core.yml * fix: core changes - added recommended config - removed ssl - added .env configs - removed sh file - unified docker-compose.yml and docker-compose-core.yml --------- Co-authored-by: Yevhen Podluzhnyi <kpiwelder@gmail.com>
igpetrov
approved these changes
Sep 9, 2024
There was a problem hiding this comment.
LGTM! Few optional comments.
Let's wait review/approval from @aabouzaid before we merge. Thank you!
- fixed comments - added params for opensearch
yevhen-cap
force-pushed
the
platform-opensearch
branch
2 times, most recently
from
0b72c8a
to
92aeb29
Compare
yevhen-cap
force-pushed
the
platform-opensearch
branch
5 times, most recently
from
cfdb641
to
10d0dda
Compare
yevhen-cap
force-pushed
the
platform-opensearch
branch
from
10d0dda
to
5b48adf
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.