Skip to content

[bitnami/elasticsearch] fix: šŸ› Mount emptyDir in /bitnami/elasticsear…#31607

Merged
fmulero merged 12 commits into
bitnami:mainfrom
leoarry:fix/elasticsearch-readonly-initscripts
Feb 27, 2025
Merged

[bitnami/elasticsearch] fix: šŸ› Mount emptyDir in /bitnami/elasticsear…#31607
fmulero merged 12 commits into
bitnami:mainfrom
leoarry:fix/elasticsearch-readonly-initscripts

Conversation

@leoarry
Copy link
Copy Markdown
Contributor

@leoarry leoarry commented Jan 27, 2025

Signed-off-by: leoarry leaorry@gmail.com

Description of the change

This PR adds an emptyDir in /bitnami/elasticsearch, which is necessary for allowing running init scripts.

Benefits

Possible drawbacks

Applicable issues

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

…ch to allow init scripts

Signed-off-by: leoarry <leaorry@gmail.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Jan 28, 2025
@github-actions github-actions Bot removed the triage Triage is needed label Jan 28, 2025
@github-actions github-actions Bot removed the request for review from carrodher January 28, 2025 09:45
@github-actions github-actions Bot requested a review from fmulero January 28, 2025 09:45
Bitnami Containers and others added 4 commits January 28, 2025 09:51
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Signed-off-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
fmulero
fmulero previously approved these changes Jan 31, 2025
Copy link
Copy Markdown
Contributor

@fmulero fmulero left a comment

Choose a reason for hiding this comment

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

Thanks a lo @leoarry for your contribution.

LGTM

@fmulero fmulero enabled auto-merge (squash) January 31, 2025 08:35
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
carrodher and others added 3 commits February 18, 2025 17:53
Signed-off-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Signed-off-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@leoarry
Copy link
Copy Markdown
Contributor Author

leoarry commented Feb 18, 2025

@fmulero @carrodher would be possible to merge this PR?

…eadonly-initscripts

Signed-off-by: Fran Mulero <francisco-jose.mulero@broadcom.com>
fmulero
fmulero previously approved these changes Feb 27, 2025
@fmulero
Copy link
Copy Markdown
Contributor

fmulero commented Feb 27, 2025

Sorry about the delay. I'll merge it as soon as possible

Just a quick note. Be carefull with the init scripts because it can be executed several times because the flag to avoid several execution is being stored in an empty volume.

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@fmulero fmulero merged commit 5d7d47f into bitnami:main Feb 27, 2025
viyullas pushed a commit to civitatis/bitnamicharts that referenced this pull request Mar 3, 2025
bitnami#31607)

* [bitnami/elasticsearch] fix: šŸ› Mount emptyDir in /bitnami/elasticsearch to allow init scripts

Signed-off-by: leoarry <leaorry@gmail.com>

---------

Signed-off-by: leoarry <leaorry@gmail.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Signed-off-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Co-authored-by: leoarry <leaorry@gmail.com>
Co-authored-by: Bitnami Containers <bitnami-bot@vmware.com>
Co-authored-by: Carlos Rodrƭguez HernƔndez <carlos.rodriguez-hernandez@broadcom.com>
Signed-off-by: Angel Abella <aabella@civitatis.com>
viyullas added a commit to civitatis/bitnamicharts that referenced this pull request Mar 3, 2025
Langleu referenced this pull request in camunda/c8-multi-region Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

elasticsearch solved verify Execute verification workflow for these changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bitnami/elasticsearch] touch Read-only file system error when initScripts defined

3 participants