feathr-ai / helm-charts

Apache License 2.0
1 stars 0 forks source link

Configmap with Helm #6

Closed ahlag closed 1 year ago

ahlag commented 1 year ago

Description

Resolves https://github.com/feathr-ai/helm-charts/issues/1

How was this PR tested?

Added configmap for environment variables of Feathr

Does this PR introduce any user-facing changes?

ahlag commented 1 year ago

@blrchen This is the command I suggested. Let me know if you need any modifications. https://github.com/feathr-ai/helm-charts/blob/8955f83fa6f6fca9d1f9d612ef910c76e558397c/charts/README.md

Environment Variables based approach

$ kubectl exec -it full-coral-feathr-online-55787cd85b-r8lhz -n default -- /bin/sh

/conf # echo $FEATHRCI_HOST
Test
ahlag commented 1 year ago

@blrchen This is the updated test document. https://github.com/feathr-ai/helm-charts/blob/8e2829c7031e81727fc6c293bb57c5b199c490ce/charts/README.md