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.

Sign up for GitHub

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

Jump to bottom

solr helm chart PR #6

Merged
merged 9 commits into from
Aug 12, 2021
Merged

solr helm chart PR #6

merged 9 commits into from
Aug 12, 2021

Conversation

naveen-armedia
Copy link
Contributor

@naveen-armedia naveen-armedia commented Aug 8, 2021 •
edited
Loading

  • solr is defined as the statefulsets with only one replication.
  • cofiguration files are added as part of configMap objects.
  • templatized storage requirements so that it could be deployed to on-prem, eks, minikube clusters.
  • Tested this solr chart with ebs and efs as persistent storage.
  • sample deploy-values.yaml that can be used to deploy is as follows. These values leverage the EFS as persistent storage for the application.
nameOverride: "ark-solr"
deployment: "eks"
persistence:
  enabled: true
  storageClass: "efs"
  fsHandlerId: "fs-55909184"
  accessModes: ["ReadWriteMany"]
namespace: "arkcase"

@naveen-armedia naveen-armedia changed the title solr helm chart added solr helm chart PR Aug 9, 2021
fabricetriboix
fabricetriboix suggested changes Aug 9, 2021
View reviewed changes
charts/ark-solr/Readme.md Outdated Show resolved Hide resolved
charts/ark-solr/templates/_helpers.tpl Outdated Show resolved Hide resolved
charts/ark-solr/templates/_tplValues.tpl Show resolved Hide resolved
charts/ark-solr/Chart.yaml Outdated Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Outdated Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Show resolved Hide resolved
charts/ark-solr/values.yaml Outdated Show resolved Hide resolved
charts/ark-solr/values.yaml Outdated Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Outdated Show resolved Hide resolved
fabricetriboix
fabricetriboix suggested changes Aug 9, 2021
View reviewed changes
charts/ark-solr/templates/statefulset.yaml Outdated Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Show resolved Hide resolved
charts/ark-solr/templates/statefulset.yaml Outdated Show resolved Hide resolved
charts/ark-solr/values.yaml Outdated Show resolved Hide resolved
fabricetriboix
fabricetriboix previously requested changes Aug 10, 2021
View reviewed changes
charts/ark-solr/Readme.md Outdated Show resolved Hide resolved
@naveen-armedia naveen-armedia dismissed fabricetriboix’s stale review August 12, 2021 15:54

Agreed while on review call to resolve the comments.

@naveen-armedia naveen-armedia merged commit 1584c2b into develop Aug 12, 2021
@naveen-armedia naveen-armedia deleted the solr_helm_pr branch August 12, 2021 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JonHolman JonHolman JonHolman approved these changes

@ymuwakki ymuwakki ymuwakki approved these changes

@jdshinnick jdshinnick Awaiting requested review from jdshinnick

@david-oc-miller david-oc-miller Awaiting requested review from david-oc-miller

@drivera-armedia drivera-armedia Awaiting requested review from drivera-armedia

@fabricetriboix fabricetriboix Awaiting requested review from fabricetriboix

Assignees

@fabricetriboix fabricetriboix

@ymuwakki ymuwakki

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@naveen-armedia @JonHolman @fabricetriboix @ymuwakki