-
Notifications
You must be signed in to change notification settings - Fork 21
Updates to K8s manifests and services for K8s compatibility #117
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
Merged
Conversation
This file contains hidden or 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
|
I followed the steps outlined in the branch's k8s README (starting from installing the Datadog Operator) and when I went to actually spin up the Storedog services, I got a bunch of errors: root@control-plane:~/lab/storedog# for file in k8s-manifests/storedog-app/**/*.yaml; do envsubst < "$file" | kubectl apply -n storedog -f -; done
configmap/feature-flags-config created
configmap/postgres-config created
configmap/postgres-scripts created
configmap/storedog-config created
service/ads created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 27: mapping values are not allowed in this context
service/backend created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 31: mapping values are not allowed in this context
service/discounts created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 31: mapping values are not allowed in this context
service/frontend created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 34: mapping values are not allowed in this context
service/service-proxy created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 38: mapping values are not allowed in this context
error: error parsing STDIN: error converting YAML to JSON: yaml: line 31: mapping values are not allowed in this context
ingress.networking.k8s.io/storedog-ingress created
secret/storedog-secrets created
service/postgres created
error: error parsing STDIN: error converting YAML to JSON: yaml: line 35: mapping values are not allowed in this context
service/redis created
statefulset.apps/redis created |
arosenkranz
approved these changes
Sep 25, 2025
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.
Description
This update improves the K8s manifest files and makes changes to the services for better compatibility when running in K8s.
k8s datadog config:
DD_ENVDD_CLOUD_PROVIDER_METADATAsettingStoredog files:
README files:
Docker Compose:
DD_CLOUD_PROVIDER_METADATAsettingServices files:
How to test
In Datadog
Infrastructure: Confirm that Instruqt tags are not listed on hosts
Logs: Confirm no logs have empty
Envby looking at facet menuAPM: Look at traces to confirm traces are received, including
service-proxyAPM: Software Catalog, confirm services listed, including
service-proxyDocker
docker compose -f docker-compose.dev.yml up -dKubernetes
K8s testing steps are outline in the
k8s-manifests/README.md. It's important to note that unlike Docker compose, K8s won't build container images. Images must be pre-built and hosted in a registry. For development, you can run a local registry. Then build and push images. The images must be built and pushed on theworkernode as that is where the services will run and look forlocalhost.The development K8s Sandbox is currently set to test.
Use the Storedog tab to the right of the
workertabs to load the site. Again, the pods are running on the worker node.