Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[elasticsearch] use bash for keystore init container #1464

Merged
merged 1 commit into from
Dec 13, 2021
Merged

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Dec 13, 2021

This commit is a follow up of 167278e and is updating the keystore
initContainer to also use bash instead of sh. This is required for
Elasticsearch > 7.16.0 because the Docker image is now based on Ubuntu
instead of CentOS 8, and sh on Ubuntu isn't compatible with the
if [[... -eq ... ]] statements.

This commit is a follow up of 167278e and is updating the keystore
initContainer to also use bash instead of sh. This is required for
Elasticsearch > 7.16.0 because the Docker image is now based on Ubuntu
instead of CentOS 8, and sh on Ubuntu isn't compatible with the
`if [[... -eq ... ]]` statements.
@jmlrt jmlrt requested review from framsouza and a team December 13, 2021 09:49
@jmlrt jmlrt added v7.16.1 and removed v7.16.0 labels Dec 13, 2021
@jmlrt jmlrt merged commit 4e31e0c into elastic:main Dec 13, 2021
@jmlrt jmlrt added need-backport bug Something isn't working labels Dec 13, 2021
jmlrt added a commit to jmlrt/helm-charts that referenced this pull request Dec 13, 2021
This commit is a follow up of 167278e and is updating the keystore
initContainer to also use bash instead of sh. This is required for
Elasticsearch > 7.16.0 because the Docker image is now based on Ubuntu
instead of CentOS 8, and sh on Ubuntu isn't compatible with the
`if [[... -eq ... ]]` statements.
jmlrt added a commit that referenced this pull request Dec 13, 2021
This commit is a follow up of 167278e and is updating the keystore
initContainer to also use bash instead of sh. This is required for
Elasticsearch > 7.16.0 because the Docker image is now based on Ubuntu
instead of CentOS 8, and sh on Ubuntu isn't compatible with the
`if [[... -eq ... ]]` statements.
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant