Update K8s example #3493
Merged
Update K8s example #3493
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tillprochaska
force-pushed
the
fix/k8s-example
branch
5 times, most recently
from
7116e8a to
bd13669
Compare
…re the ingress resource
Otherwise a Helm chart for ES 8 will be used, which by default installs ES 8 and has some breaking changes with regards to the structure of values.
tillprochaska
force-pushed
the
fix/k8s-example
branch
from
bd13669 to
57da03c
Compare
This adjusts the example so that it can be easily run on a single-node K8s cluster provided by Docker Compose. I have also tweaked the default values to reduce the resources to a minimum and tried to make the example as easy to install as possible.
tillprochaska
force-pushed
the
fix/k8s-example
branch
from
57da03c to
ee064a6
Compare
stchris
reviewed
Nov 13, 2023
stchris
reviewed
Nov 13, 2023
| dependencies: | ||
| - name: aleph | ||
| repository: file://../../charts/aleph | ||
| version: 3.15.4 |
There was a problem hiding this comment.
I'm not sure if I did something wrong, but I get aleph 3.15.1 :/
There was a problem hiding this comment.
Hm, that’s weird… I just reset my cluster, removed local images, and set up everything again from scratch. But looking at the about page and at the version specified inside of the container I see 3.15.4. We could look into it together tomorrow if you’d like :)
There was a problem hiding this comment.
Only other reason for this I could imagine is that this branch used to be based off of 3.15.1, and I later rebased it and force pushed to be up to date with develop. But that was a few days before you looked at it, I think…
tillprochaska
force-pushed
the
fix/k8s-example
branch
from
f5972ec to
76a4111
Compare
simonwoerpel
pushed a commit
to investigativedata/aleph
that referenced
this pull request
Apr 22, 2024
* Update example values to work with current Postgres chart version * Add ingress class annotation to ensure the Nginx ingress doesn’t ignore the ingress resource * Use ES Helm chart for ES 7 Otherwise a Helm chart for ES 8 will be used, which by default installs ES 8 and has some breaking changes with regards to the structure of values. * Mark secret key ref for `SENTRY_DSN` as optional * Update K8s example This adjusts the example so that it can be easily run on a single-node K8s cluster provided by Docker Compose. I have also tweaked the default values to reduce the resources to a minimum and tried to make the example as easy to install as possible. * Emphasize that selecting the correct kubectl context may be necessary * Update default ingest-file version in Helm chart * Add note on Docker Desktop memory resource limits * Add note about `kubernetes.docker.internal` host * Add upgrade instructions * Simplify secrets setup * Document how to create a user, known issues * Fix typo * Replace deprecated `kubernetes.io/ingress.class` annotation As suggested by @stchris * Fix default ingest-file version in Helm chart (again…)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The previous example for running Aleph in a local Kubernetes cluster was based on kind. This simplifies the example and default configuration, making it easier to run a minimal Aleph instance in a local Kubernetes cluster provided by Docker Desktop.
Additionally, this PR includes two changes to the Aleph Helm chart:
Sets the default
ingest-fileversion to3.19.3, the latest release (the default version in the chart was out of sync with the latest release).Marks the
SENTRY_DSNkey ref as optional. This way, starting containers will succeed, even if theSENTRY_DSNkey does not exist in thealeph-secretssecret. Previously, starting containers that reference theSENTRY_DSNkey would fail (requiring users to manually create an emptySENTRY_DSNkey in thealeph-secretssecret).