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

feat: Update to the latest Helm chart and add new services #42

Merged
merged 5 commits into from
Jan 16, 2023
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
2 changes: 2 additions & 0 deletions deploy/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ This environment can be used for demos, and should be considered "stable".

You can edit the following files in this directory:

- [`meta.yaml`](./meta.yaml) -- contains various meta configuration about the
deployment, e.g. the currently used version of the Helm chart.
- [`sentry-components.yaml`](./sentry-components.yaml) -- a list of components
that have Sentry instrumentation. If some service there is commented out, that
means that the vanilla version of the service (meaning, a prebuilt Docker image)
Expand Down
60 changes: 23 additions & 37 deletions deploy/global.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ observability:
enabled: true

prometheus:
# TODO(tonyo): run in the otel nodepool
rbac:
create: false
serverFiles:
Expand All @@ -25,11 +26,13 @@ prometheus:
- targets:
- "otel-demo-otelcol:8888"
grafana:
# TODO(tonyo): run in the otel nodepool
rbac:
create: false
pspEnabled: false

jaeger:
# TODO(tonyo): run in the otel nodepool
allInOne:
resources:
requests:
Expand All @@ -50,104 +53,87 @@ default:

# Demo components
components:
# Main services

accountingService:
resources:
requests:
cpu: 50m
adService:
resources:
requests:
cpu: 50m
schedulingRules:
# Here and below: this is a workaround for an existing issue in the Helm chart (to be fixed).
# The final values for nodeSelector and tolerations will still be takend from "default".
nodeSelector: {}
cartService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
checkoutService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
currencyService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
emailService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
featureflagService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
ffsPostgres:
frauddetectionService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
frontend:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
frontendProxy:
resources:
requests:
cpu: 50m
memory: 100Mi
limits:
memory: 100Mi
schedulingRules:
nodeSelector: {}
loadgenerator:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
paymentService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
productCatalogService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
quoteService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
recommendationService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
redis:
shippingService:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
shippingService:

# Supporting services and data storages

ffsPostgres:
resources:
requests:
cpu: 50m
kafka:
resources:
requests:
cpu: 50m
redis:
resources:
requests:
cpu: 50m
schedulingRules:
nodeSelector: {}
5 changes: 5 additions & 0 deletions deploy/meta.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
# This configuration file contains meta information for the Sentry-specific opentelemetry-demo deployments.

# Version of the Helm chart to use.
# Chart: https://github.com/open-telemetry/opentelemetry-helm-charts/tree/main/charts/opentelemetry-demo
helm_chart_version: "0.15.4"
4 changes: 4 additions & 0 deletions deploy/sentry-components.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@
#
---
components:
# accountingService: {}

adService: {}

# cartService: {}
Expand All @@ -23,6 +25,8 @@ components:

# featureflagService: {}

# frauddetectionService: {}

frontend: {}

paymentService: {}
Expand Down