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

[BUG] system_indices missing some default plugin indices in the helm chart opensearch values.yaml file #479

Open
Flyingliuhub opened this issue Oct 5, 2023 · 3 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@Flyingliuhub
Copy link
Member

Describe the bug
There are some default plugin's system_indices missing in the OpenSearch/values.yaml, please see the security plugin's system_indices list

To Reproduce
Compare helm chart opensearch's value.yaml and security plugin's install_demo_configuration.sh

Expected behavior
A clear and concise description of what you expected to happen.

Chart Name
OpenSearch

Screenshots
N/A

Host/Environment (please complete the following information):

  • Helm Version: 2.10

Additional context
Add any other context about the problem here.

@Flyingliuhub Flyingliuhub added bug Something isn't working untriaged Issues that have not yet been triaged labels Oct 5, 2023
@ylwu-amzn
Copy link

This is something we should prioritize. Some system index stores important data. Missing system index in the helm-chats configuration will cause security issue. All teams maintain their system index in security plugin's install_demo_configuration.sh , I'd suggest use some way to sync from security plugin, or at least check if they are consistent with some cron job.

@stephen-crawford
Copy link

@ylwu-amzn's suggestion to use the Security plugin's framework is a good path forward. You can always grab the different versions from the published versions on the security repos and then map 1:1 to the different helm-chart versions. To pull this in you can checkout the file from the security repos during your workflow process and then insert it using github actions. Alternatively you can look at some bash based solutions to this.

@prudhvigodithi prudhvigodithi added good first issue Good for newcomers and removed untriaged Issues that have not yet been triaged labels Oct 10, 2023
@prudhvigodithi
Copy link
Member

[Untriage]
Adding @tranngocsongtruc @SkollRyu @premkirank can you please take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Status: 📦 Backlog
Development

No branches or pull requests

4 participants