Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use gitlab image, add docs for gitlab container regisgry pull secrets, removed home dir, and bumped memory required by new image #15

Merged
merged 2 commits into from
Mar 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions charts/langkit/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,16 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning]
(https://semver.org/spec/v2.0.0.html).

## [0.13.0] - 2023-02-29

### Changed

- Updated default image to `1.0.8` within GitLab container registry.

### Added

- Documentation for creating a secret for access to the private image registry

## [0.12.0] - 2023-02-29

### Changed
Expand Down
2 changes: 1 addition & 1 deletion charts/langkit/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@ apiVersion: v2
name: langkit
description: A Helm chart for LangKit container deployment
type: application
version: 0.12.0
version: 0.13.0
appVersion: "1.0.8"
26 changes: 18 additions & 8 deletions charts/langkit/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
NOTE: Change the `--namespace` value if you will be deploying into a namespace other
than `langkit`.


### Credentials
* Create a [WhyLabs API Key](https://docs.whylabs.ai/docs/whylabs-capabilities/#access-token-management)
which must be stored in a `whylabs-api-key` Kubernetes secret, described below.
Expand All @@ -31,6 +30,17 @@ kubectl create secret generic langkit-api-secret \
--from-literal=CONTAINER_PASSWORD=<langkit-api-secret>
```

* Create a secret with a WhyLabs provided GitLab token to pull the LangKit image

```
kubectl create secret docker-registry whylabs-gitlab-registry-secret \
--docker-server=registry.gitlab.com \
--docker-username="project_55361491_bot_4a4030bd91182991f959fe2a3cdd2762" \
--docker-password="<token>" \
--docker-email="project_55361491_bot_4a4030bd91182991f959fe2a3cdd2762@noreply.gitlab.com" \
--namespace=langkit
```

### LangKit Configuration

No LangKit configurations are required out of the box. However, for further customizations,
Expand Down Expand Up @@ -94,13 +104,13 @@ helm-docs --dry-run
|-----|------|---------|-------------|
| containers.env | list | `[]` | Environment variables for the containers |
| containers.securityContext | object | `{"readOnlyRootFilesystem":true,"runAsUser":1000}` | Container security context |
| containers.volumeMounts | list | `[{"mountPath":"/tmp","name":"temp-dir"},{"mountPath":"/home","name":"home"}]` | Volume mounts for containers |
| containers.volumeMounts | list | `[{"mountPath":"/tmp","name":"temp-dir"}]` | Volume mounts for containers |
| fullnameOverride | string | `""` | |
| image.containerPort | int | `8000` | |
| image.pullPolicy | string | `"Always"` | |
| image.repository | string | `"whylabs/whylogs"` | |
| image.tag | string | `"py-llm-1.0.8"` | |
| imagePullSecrets | list | `[]` | |
| image.repository | string | `"registry.gitlab.com/whylabs/langkit-container"` | |
| image.tag | string | `"1.0.8"` | |
| imagePullSecrets[0].name | string | `"whylabs-gitlab-registry-secret"` | |
| ingress.annotations | object | `{}` | |
| ingress.className | string | `""` | |
| ingress.enabled | bool | `false` | |
Expand All @@ -123,9 +133,9 @@ helm-docs --dry-run
| readinessProbe.periodSeconds | int | `30` | |
| replicaCount | int | `2` | |
| resources.limits.cpu | string | `"4"` | |
| resources.limits.memory | string | `"3Gi"` | |
| resources.limits.memory | string | `"4Gi"` | |
| resources.requests.cpu | string | `"4"` | |
| resources.requests.memory | string | `"3Gi"` | |
| resources.requests.memory | string | `"4Gi"` | |
| secrets.langkitApiSecret.name | string | `"langkit-api-secret"` | Name of the secret that stores the WhyLabs LangKit API Secret |
| secrets.whylabsApiKey.name | string | `"whylabs-api-key"` | Name of the secret that stores the WhyLabs API Key |
| service.port | int | `80` | |
Expand All @@ -135,7 +145,7 @@ helm-docs --dry-run
| serviceAccount.create | bool | `true` | |
| serviceAccount.name | string | `""` | |
| tolerations | list | `[]` | |
| volumes | list | `[{"emptyDir":{},"name":"temp-dir"},{"emptyDir":{},"name":"home"}]` | Volumes to create |
| volumes | list | `[{"emptyDir":{},"name":"temp-dir"}]` | Volumes to create |

----------------------------------------------
Autogenerated from chart metadata using [helm-docs v1.12.0](https://github.com/norwoodj/helm-docs/releases/v1.12.0)
15 changes: 6 additions & 9 deletions charts/langkit/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,13 @@ ingress:
# - chart-example.local

image:
repository: whylabs/whylogs
repository: registry.gitlab.com/whylabs/langkit-container
pullPolicy: Always
tag: py-llm-1.0.8
tag: "1.0.8"
containerPort: 8000

imagePullSecrets: []
imagePullSecrets:
- name: whylabs-gitlab-registry-secret

pod:
annotations: {}
Expand All @@ -60,8 +61,6 @@ containers:
volumeMounts:
- name: temp-dir
mountPath: /tmp
- name: home
mountPath: /home

service:
type: ClusterIP
Expand All @@ -70,10 +69,10 @@ service:
resources:
requests:
cpu: "4"
memory: 3Gi
memory: 4Gi
limits:
cpu: "4"
memory: 3Gi
memory: 4Gi

secrets:
whylabsApiKey:
Expand Down Expand Up @@ -124,5 +123,3 @@ affinity: {}
volumes:
- name: temp-dir
emptyDir: {}
- name: home
emptyDir: {}
Loading