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

OPSEXP-2950 Prepare to update to Elasticsearch 8 by switching to the bitnami/elasticsearch chart #1267

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

pmacius
Copy link
Contributor

@pmacius pmacius commented Dec 9, 2024

@pmacius pmacius requested review from alxgomz and gionn December 18, 2024 10:13
@pmacius pmacius requested review from alxgomz December 20, 2024 13:12
helm/alfresco-content-services/values.yaml Outdated Show resolved Hide resolved
helm/alfresco-content-services/values.yaml Outdated Show resolved Hide resolved
helm/alfresco-content-services/values.yaml Show resolved Hide resolved
docker-compose/pre-release-compose.yaml Outdated Show resolved Hide resolved
@gionn gionn removed their request for review January 2, 2025 11:06
@pmacius pmacius changed the title OPSEXP-2950 Bump elasticsearch to 8.16.1 for pre-release OPSEXP-2950 Prepare to update to Elasticsearch 8 by switching to the bitnami/elasticsearch chart Jan 7, 2025
@pmacius pmacius requested review from gionn and alxgomz and removed request for alxgomz January 7, 2025 18:27
test/enterprise-integration-test-values.yaml Outdated Show resolved Hide resolved
helm/alfresco-content-services/values.yaml Outdated Show resolved Hide resolved
helm/alfresco-content-services/values.yaml Outdated Show resolved Hide resolved
helm/alfresco-content-services/values.yaml Show resolved Hide resolved
helm/alfresco-content-services/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@alxgomz alxgomz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK for me with Kibana disabled by default and a small doc on how to enable and secure it leveraging mechanisms provided by the bitnami chart

@pmacius pmacius requested review from alxgomz and gionn January 9, 2025 11:17
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should make it clear this is in case one uses the embedded elasticsearch which is not recommended for prod

@pmacius pmacius requested a review from alxgomz January 9, 2025 14:44
--from-literal=AUDIT_ELASTICSEARCH_USERNAME=$ELASTIC_USERNAME \
--from-literal=AUDIT_ELASTICSEARCH_PASSWORD=$ELASTIC_PASSWORD \
--from-literal=SEARCH_USERNAME=$ELASTIC_USERNAME \
--from-literal=SEARCH_PASSWORD=$ELASTIC_PASSWORD
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see --from-env-file which should prevent user leaking credentials in its history

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants