changes to run services in openshift#478
Draft
paulmurdoch19 wants to merge 195 commits intouc-cdis:masterfrom
Draft
changes to run services in openshift#478paulmurdoch19 wants to merge 195 commits intouc-cdis:masterfrom
paulmurdoch19 wants to merge 195 commits intouc-cdis:masterfrom
Conversation
Eliminates a redundant securityContext definition in the fence-deployment.yaml template, ensuring only a single securityContext is set for the pod spec.
make some changes to allow for persistent postgres storage, running charts with db's using tilt, GEAR-527, some updates to default values
Replaced custom HPA logic in amanuensis, gearbox, and gearbox-middleware Helm charts with a shared 'common.hpa' template for consistency and maintainability. Updated autoscaling configuration in values.yaml files: fixed typo 'minAvialable' to 'minAvailable', reduced maxReplicas from 100 to 10, and switched CPU/memory autoscaling from utilization percentage to average value.
Sync cdis pcdc
Introduced environment variables for database connection (DB_DATABASE, DB_HOST, DB_USER, DB_PASSWORD, DBREADY) sourced from the gearbox-dbcreds secret in values.yaml. This enables the application to securely access database credentials via Kubernetes secrets.
Sync: Upstream Master into pcdc_dev (2025-10-27)
Add DB credentials to environment variables
paulmurdoch19
commented
Dec 19, 2025
|
|
||
| # run arborist | ||
| /go/src/github.com/uc-cdis/arborist/bin/arborist | ||
| /go/src/github.com/uc-cdis/arborist/bin/arborist --port 8080 |
Author
There was a problem hiding this comment.
run the arborist server on port 8080 instead of 80
paulmurdoch19
commented
Dec 19, 2025
| user {{ .Values.nginx.user }}; | ||
| worker_processes 4; | ||
| pid /var/run/nginx.pid; | ||
| pid {{ .Values.nginx.pidFile }}; |
Author
There was a problem hiding this comment.
I made these variables in case the folder needs to be changed
paulmurdoch19
commented
Dec 19, 2025
| @@ -0,0 +1,27 @@ | |||
| {{- if .Values.openshiftRoute.enabled }} | |||
| apiVersion: route.openshift.io/v1 | |||
Author
There was a problem hiding this comment.
This is an OpenShift resource that I believe is preferred over ingress
paulmurdoch19
commented
Dec 19, 2025
| # - chart-example.local | ||
|
|
||
| # -- (map) Configuration for OpenShift Route. | ||
| openshiftRoute: |
Author
There was a problem hiding this comment.
add values for the route resource
paulmurdoch19
commented
Dec 19, 2025
| # https://distinctplace.com/2017/04/19/nginx-resolver-explained/ | ||
| # | ||
| resolver kube-dns.kube-system.svc.cluster.local ipv6=off; | ||
| resolver {{ .Values.nginx.resolver }} ipv6=off; |
Author
There was a problem hiding this comment.
I made this a variable since the value is different for openshift
paulmurdoch19
commented
Dec 19, 2025
| source "${GEN3_HOME}/gen3/lib/utils.sh" | ||
| gen3_load "gen3/gen3setup" | ||
| #source "${GEN3_HOME}/gen3/lib/utils.sh" | ||
| #gen3_load "gen3/gen3setup" |
Author
There was a problem hiding this comment.
cant run this in openshift
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.