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

[bitnami/opensearch] Fix plugins installation #29586

Merged
merged 6 commits into from
Sep 24, 2024

Conversation

juan131
Copy link
Contributor

@juan131 juan131 commented Sep 24, 2024

Description of the change

This PR fixes the conditions under init containers are rendered in OpenSearch pods and it fixes custom plugins installation by ensuring we respect original container permissions on plugins and config directories via the "copy-default-plugins" init container.

Benefits

User can install custom plugins without issues.

Possible drawbacks

None

Applicable issues

Additional information

N/A

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • 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)

Signed-off-by: juan131 <jariza@vmware.com>
@github-actions github-actions bot added opensearch verify Execute verification workflow for these changes bitnami labels Sep 24, 2024
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Copy link
Contributor

@andresbono andresbono left a comment

Choose a reason for hiding this comment

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

Are these changes needed for bitnami/elasticsearch as well?

@juan131
Copy link
Contributor Author

juan131 commented Sep 24, 2024

Are these changes needed for bitnami/elasticsearch as well?

They'll be implemented in a separate PR

@andresbono
Copy link
Contributor

Please, check the CI result, something is wrong with the config directory

juan131 and others added 4 commits September 24, 2024 13:33
Signed-off-by: juan131 <jariza@vmware.com>
Signed-off-by: juan131 <jariza@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
@juan131 juan131 merged commit 464c683 into bitnami:main Sep 24, 2024
9 checks passed
@juan131 juan131 deleted the bugfix/opensearch-plugins branch September 24, 2024 12:25
mike12806 pushed a commit to mike12806/charts that referenced this pull request Oct 30, 2024
Signed-off-by: Michael Faherty <80599490+mike12806@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami opensearch solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bitnami/opensearch] Can't install plugins
3 participants