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

[LX-1442] Adjust Helm deployment templates and value files #87

Merged
merged 46 commits into from
Nov 30, 2023

Conversation

cachemeoutside
Copy link
Contributor

The following changes have been made:

  • oral-history-env now accepts the literal value of web.env. This means you don't have to copy/paste values in both the values.yaml file and the configmap localtions
  • Added env section in deployment template for dependent environment variable substitution. This was added to support setting SOLR_URL. See here: https://kubernetes.io/docs/tasks/inject-data-application/define-interdependent-environment-variables/
  • Turned off Solr Cloud and Zookeeper from Bitnami chart
  • Added JVM flags to set heap space to 80% of assigned resources on Solr deployment
  • Added blacklight-core name to Solr bitnami chart
  • Added extra environment variables(pulled from our existing Docker Swarm configs). These were not documented in the README.
  • Added liveness and readiness probes back into deployment
  • Adjust worker deployment to use CMD bin/delayed_job run
  • Set resource limits/requests for Solr and Web deployment
  • Other minor pointers and syntax issues.

@cachemeoutside cachemeoutside self-assigned this Nov 30, 2023
@cachemeoutside cachemeoutside merged commit c841bb4 into main Nov 30, 2023
1 check passed
@cachemeoutside cachemeoutside deleted the fix-helm-chart branch November 30, 2023 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant