Skip to content

Commit 1395f3f

Browse files
ashwinathashwinath
authored
Add Kubernetes Pod labels to Core and Serving. (#795)
* Add Kubernetes Pod labels to Core and Serving. * Missed a dot before Values.
1 parent 53cc46a commit 1395f3f

6 files changed

Lines changed: 21 additions & 7 deletions

File tree

‎infra/charts/feast/charts/feast-core/README.md‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Current chart version is `0.5.0-alpha.1`
1212

1313
| Key | Type | Default | Description |
1414
|-----|------|---------|-------------|
15-
| "application-generated.yaml".enabled | bool | `true` | Flag to include Helm generated configuration for Feast database URL, Kafka bootstrap servers and jobs metrics host. This is useful for deployment that uses default configuration for Kafka, Postgres and StatsD exporter. Please set `application-override.yaml` to override this configuration. |
15+
| "application-generated.yaml".enabled | bool | `true` | Flag to include Helm generated configuration for http port, Feast database URL, Kafka bootstrap servers and jobs metrics host. This is useful for deployment that uses default configuration for Kafka, Postgres and StatsD exporter. Please set `application-override.yaml` to override this configuration. |
1616
| "application-override.yaml" | object | `{"enabled":true}` | Configuration to override the default [application.yaml](https://github.com/feast-dev/feast/blob/master/core/src/main/resources/application.yml). Will be created as a ConfigMap. `application-override.yaml` has a higher precedence than `application-secret.yaml` |
1717
| "application-secret.yaml" | object | `{"enabled":true}` | Configuration to override the default [application.yaml](https://github.com/feast-dev/feast/blob/master/core/src/main/resources/application.yml). Will be created as a Secret. `application-override.yaml` has a higher precedence than `application-secret.yaml`. It is recommended to either set `application-override.yaml` or `application-secret.yaml` only to simplify config management. |
1818
| "application.yaml".enabled | bool | `true` | Flag to include the default [configuration](https://github.com/feast-dev/feast/blob/master/core/src/main/resources/application.yml). Please set `application-override.yaml` to override this configuration. |
@@ -42,7 +42,7 @@ Current chart version is `0.5.0-alpha.1`
4242
| ingress.http.https.secretNames | object | `{}` | Map of hostname to TLS secret name |
4343
| ingress.http.whitelist | string | `""` | Allowed client IP source ranges |
4444
| javaOpts | string | `nil` | [JVM options](https://docs.oracle.com/cd/E22289_01/html/821-1274/configuring-the-default-jvm-and-java-arguments.html). For better performance, it is advised to set the min and max heap: <br> `-Xms2048m -Xmx2048m` |
45-
| livenessProbe.enabled | bool | `true` | Flag to enabled the probe |
45+
| livenessProbe.enabled | bool | `false` | Flag to enabled the probe |
4646
| livenessProbe.failureThreshold | int | `5` | Min consecutive failures for the probe to be considered failed |
4747
| livenessProbe.initialDelaySeconds | int | `60` | Delay before the probe is initiated |
4848
| livenessProbe.periodSeconds | int | `10` | How often to perform the probe |
@@ -51,6 +51,7 @@ Current chart version is `0.5.0-alpha.1`
5151
| logLevel | string | `"WARN"` | Default log level, use either one of `DEBUG`, `INFO`, `WARN` or `ERROR` |
5252
| logType | string | `"Console"` | Log format, either `JSON` or `Console` |
5353
| nodeSelector | object | `{}` | Node labels for pod assignment |
54+
| podLabels | object | `{}` | Labels to be added to Feast Core pods |
5455
| postgresql.existingSecret | string | `""` | Existing secret to use for authenticating to Postgres |
5556
| prometheus.enabled | bool | `true` | Flag to enable scraping of Feast Core metrics |
5657
| readinessProbe.enabled | bool | `true` | Flag to enabled the probe |

‎infra/charts/feast/charts/feast-core/templates/deployment.yaml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ spec:
3030
app: {{ template "feast-core.name" . }}
3131
component: core
3232
release: {{ .Release.Name }}
33+
{{- if .Values.podLabels }}
34+
{{ toYaml .Values.podLabels | nindent 8 }}
35+
{{- end }}
3336
spec:
3437
{{- with .Values.nodeSelector }}
3538
nodeSelector:

‎infra/charts/feast/charts/feast-core/values.yaml‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,4 +150,7 @@ resources: {}
150150
nodeSelector: {}
151151

152152
# envOverrides -- Extra environment variables to set
153-
envOverrides: {}
153+
envOverrides: {}
154+
155+
# podLabels -- Labels to be added to Feast Core pods
156+
podLabels: {}

‎infra/charts/feast/charts/feast-serving/README.md‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,13 @@ Current chart version is `0.5.0-alpha.1`
1212

1313
| Key | Type | Default | Description |
1414
|-----|------|---------|-------------|
15-
| "application-generated.yaml".enabled | bool | `true` | Flag to include Helm generated configuration for Feast Core host, Redis store and job store. This is useful for deployment that uses default configuration for Redis. Please set `application-override.yaml` to override this configuration. |
15+
| "application-generated.yaml".enabled | bool | `true` | Flag to include Helm generated configuration for http port, Feast Core host, Redis store and job store. This is useful for deployment that uses default configuration for Redis. Please set `application-override.yaml` to override this configuration. |
1616
| "application-override.yaml" | object | `{"enabled":true}` | Configuration to override the default [application.yaml](https://github.com/feast-dev/feast/blob/master/serving/src/main/resources/application.yml). Will be created as a ConfigMap. `application-override.yaml` has a higher precedence than `application-secret.yaml` |
1717
| "application-secret.yaml" | object | `{"enabled":true}` | Configuration to override the default [application.yaml](https://github.com/feast-dev/feast/blob/master/serving/src/main/resources/application.yml). Will be created as a Secret. `application-override.yaml` has a higher precedence than `application-secret.yaml`. It is recommended to either set `application-override.yaml` or `application-secret.yaml` only to simplify config management. |
1818
| "application.yaml".enabled | bool | `true` | Flag to include the default [configuration](https://github.com/feast-dev/feast/blob/master/serving/src/main/resources/application.yml). Please set `application-override.yaml` to override this configuration. |
1919
| envOverrides | object | `{}` | Extra environment variables to set |
2020
| gcpProjectId | string | `""` | Project ID to use when using Google Cloud services such as BigQuery, Cloud Storage and Dataflow |
21-
| gcpServiceAccount.enabled | bool | `false` | Flag to use [service account](https://cloud.google.com/iam/docs/creating-managing-service-account-keys) JSON key |
21+
| gcpServiceAccount.enabled | bool | `false` | Flag to use [service account](https://cloud.google.com/iam/docs/creating-managing-service-account-keys) JSON key Cloud service account JSON key file. |
2222
| gcpServiceAccount.existingSecret.key | string | `"credentials.json"` | Key in the secret data (file name of the service account) |
2323
| gcpServiceAccount.existingSecret.name | string | `"feast-gcp-service-account"` | Name of the existing secret containing the service account |
2424
| image.pullPolicy | string | `"IfNotPresent"` | Image pull policy |
@@ -51,6 +51,7 @@ Current chart version is `0.5.0-alpha.1`
5151
| logLevel | string | `"WARN"` | Default log level, use either one of `DEBUG`, `INFO`, `WARN` or `ERROR` |
5252
| logType | string | `"Console"` | Log format, either `JSON` or `Console` |
5353
| nodeSelector | object | `{}` | Node labels for pod assignment |
54+
| podLabels | object | `{}` | Labels to be added to Feast Serving pods |
5455
| prometheus.enabled | bool | `true` | Flag to enable scraping of Feast Core metrics |
5556
| readinessProbe.enabled | bool | `true` | Flag to enabled the probe |
5657
| readinessProbe.failureThreshold | int | `5` | Min consecutive failures for the probe to be considered failed |
@@ -65,5 +66,5 @@ Current chart version is `0.5.0-alpha.1`
6566
| service.grpc.targetPort | int | `6566` | Container port serving GRPC requests |
6667
| service.http.nodePort | string | `nil` | Port number that each cluster node will listen to |
6768
| service.http.port | int | `80` | Service port for HTTP requests |
68-
| service.http.targetPort | int | `8080` | Container port serving HTTP requests |
69+
| service.http.targetPort | int | `8080` | Container port serving HTTP requests and Prometheus metrics |
6970
| service.type | string | `"ClusterIP"` | Kubernetes service type |

‎infra/charts/feast/charts/feast-serving/templates/deployment.yaml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ spec:
3030
app: {{ template "feast-serving.name" . }}
3131
component: serving
3232
release: {{ .Release.Name }}
33+
{{- if .Values.podLabels }}
34+
{{ toYaml .Values.podLabels | nindent 8 }}
35+
{{- end }}
3336
spec:
3437
{{- with .Values.nodeSelector }}
3538
nodeSelector:

‎infra/charts/feast/charts/feast-serving/values.yaml‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,4 +145,7 @@ resources: {}
145145
nodeSelector: {}
146146

147147
# envOverrides -- Extra environment variables to set
148-
envOverrides: {}
148+
envOverrides: {}
149+
150+
# podLabels -- Labels to be added to Feast Serving pods
151+
podLabels: {}

0 commit comments

Comments
 (0)