Skip to content

Commit

Permalink
mobu: idfdev sentry
Browse files Browse the repository at this point in the history
  • Loading branch information
fajpunk committed Jan 15, 2025
1 parent cb7f79f commit 2d538cc
Show file tree
Hide file tree
Showing 6 changed files with 28 additions and 2 deletions.
4 changes: 3 additions & 1 deletion applications/mobu/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Continuous integration testing
|-----|------|---------|-------------|
| affinity | object | `{}` | Affinity rules for the mobu frontend pod |
| config.autostart | list | `[]` | Autostart specification. Must be a list of mobu flock specifications. Each flock listed will be automatically started when mobu is started. |
| config.githubCiApp | string | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration |
| config.githubCiApp | string | disabled. | Configuration for the GitHub CI app integration. See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration |
| config.githubRefreshApp | string | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_refresh_app.html#add-phalanx-configuration |
| config.logLevel | string | `"INFO"` | Log level. Set to 'DEBUG' to include the output from all flocks in the main mobu log. |
| config.metrics.application | string | `"mobu"` | Name under which to log metrics. Generally there is no reason to change this. |
Expand All @@ -24,6 +24,8 @@ Continuous integration testing
| config.metrics.schemaManager.suffix | string | `""` | Suffix to add to all registered subjects. This is sometimes useful for experimentation during development. |
| config.pathPrefix | string | `"/mobu"` | Prefix for mobu's API routes. |
| config.profile | string | `"production"` | One of 'production' or 'development'. 'production' configures structured JSON logging, and 'development' configures unstructured human readable logging. |
| config.sentryEnvironment | string | `nil` | The environment to report to Sentry |
| config.sentryTracesSampleConfig | float | `0` | Sentry tracing config: a float to specify a percentage, or "errors" to send all transactions with errors. |
| config.slackAlerts | bool | `true` | Whether to send alerts and status to Slack. |
| fullnameOverride | string | `""` | Override the full name for resources (includes the release name) |
| global.baseUrl | string | Set by Argo CD | Base URL for the environment |
Expand Down
7 changes: 7 additions & 0 deletions applications/mobu/secrets.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -68,3 +68,10 @@ github-refresh-app-webhook-secret:
This value is also configured in the GitHub UI for the matching GitHub
application. If we need to change this value, the Phalanx secret must be
updated, and the value must be changed in the GitHub UI.
sentry-dsn:
description: >-
The Sentry DSN for the mobu Sentry project. This is a URL that includes an
authentication token that tells the Sentry SDK where to send events. It can
be found here:
https://rubin-observatory.sentry.io/settings/projects/mobu/keys/
7 changes: 7 additions & 0 deletions applications/mobu/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,13 @@ spec:
secretKeyRef:
name: {{ template "mobu.fullname" . }}-gafaelfawr-token
key: "token"
- name: "MOBU_SENTRY_ENVIRONMENT"
value: {{ .Values.global.environmentName }}
- name: "MOBU_SENTRY_DSN"
valueFrom:
secretKeyRef:
name: {{ template "mobu.fullname" . }}-secret
key: "sentry-dsn"
{{- if .Values.config.githubRefreshApp }}
- name: "MOBU_GITHUB_REFRESH_APP_WEBHOOK_SECRET"
valueFrom:
Expand Down
2 changes: 2 additions & 0 deletions applications/mobu/values-idfdev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ config:
profile: development
metrics:
enabled: true
sentryEnvironment: idfdev
sentryTracesSampleConfig: errors
githubRefreshApp:
acceptedGithubOrgs:
- lsst-sqre
Expand Down
8 changes: 7 additions & 1 deletion applications/mobu/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,17 @@ config:
githubRefreshApp: null

# -- Configuration for the GitHub CI app integration.
# -- Configuration for the GitHub refresh app integration.
# See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration
# @default -- disabled.
githubCiApp: null

# -- The environment to report to Sentry
sentryEnvironment: null

# -- Sentry tracing config: a float to specify a percentage, or "errors" to
# send all transactions with errors.
sentryTracesSampleConfig: 0.0

# -- Log level. Set to 'DEBUG' to include the output from all flocks in the
# main mobu log.
logLevel: INFO
Expand Down
2 changes: 2 additions & 0 deletions environments/templates/applications/infrastructure/mobu.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ spec:
value: "https://{{ .Values.fqdn }}"
- name: "global.vaultSecretsPath"
value: {{ .Values.vaultPathPrefix | quote }}
- name: "global.environmentName"
value: {{ .Values.name | quote }}
valueFiles:
- "values.yaml"
- "values-{{ .Values.name }}.yaml"
Expand Down

0 comments on commit 2d538cc

Please sign in to comment.