search

senzing-garage / charts

:no_entry: [DEPRECATED]
Apache License 2.0
0 stars 4 forks source link

Support serviceAccount and serviceAccountName #89

Closed docktermj closed 2 years ago

docktermj commented 4 years ago

Is your feature request related to a problem? Please describe. Currently there is no support in some charts for serviceAccount nor serviceAccountName.

docktermj commented 4 years ago

According to https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.14/#podspec-v1-core the use of ServiceAccount is deprecated in favor of ServiceAccountName.

Also see https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/

docktermj commented 4 years ago 
export CHART_NAMES=( \
  "coleifer-sqlite-web" \
  "db2-client" \
  "ibm-db2-driver-installer" \
  "resolver" \
  "senzing-api-server" \
  "senzing-base" \
  "senzing-configurator" \
  "senzing-debug" \
  "senzing-entity-search-web-app" \
  "senzing-init-container" \
  "senzing-mock-data-generator" \
  "senzing-redoer" \
  "senzing-stream-loader" \
  "senzing-yum" \
)
docktermj commented 2 years ago

Fixed via https://github.com/Senzing/charts/issues/164 and

Pull requests

  1. https://github.com/Senzing/charts/pull/90
  2. https://github.com/Senzing/charts/pull/93
  3. https://github.com/Senzing/charts/pull/94