-
Notifications
You must be signed in to change notification settings - Fork 73
Closed
devfile/registry-support
#221Labels
area/registryDevfile registry for stacks and infrastructureDevfile registry for stacks and infrastructure
Description
Which area/kind this issue is related to?
/area registry
Issue Description
To create consistancy with the registry operator within the helm chart, the following changes need to be done:
- Outdated probe time values need to be updated
Index Server - startupProbe
Timeouts on the helm chart should be updated to the timeouts on the registry operator:
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 20Registry Viewer - livenessProbe
Timeouts on the helm chart should be updated to the timeouts on the registry operator:
initialDelaySeconds: 15
periodSeconds: 10
timeoutSeconds: 20Registry Viewer - readinessProbe
Timeouts on the helm chart should be updated to the timeouts on the registry operator:
initialDelaySeconds: 15
periodSeconds: 10
timeoutSeconds: 20- Pod level security context found in the registry operator should be defined in the helm chart as well
- The registry viewer
.envshould be removed as the environment variables are already being set under the specification: https://github.com/devfile/registry-support/blob/f66264f22310c3c93bf0c77db5f4cd4b560aba5e/deploy/chart/devfile-registry/templates/deployment.yaml#L138
Parent Epic: #1274
Acceptance Criteria
- Set probe time values under helm chart to match same probes under registry operator
- Define pod level security context under helm chart to match registry operator
- Remove
.envmount from the helm chart - Check deployment integrity of changes
Metadata
Metadata
Assignees
Labels
area/registryDevfile registry for stacks and infrastructureDevfile registry for stacks and infrastructure
Type
Projects
Status
Done ✅