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.

Sign up for GitHub

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

DM-48165: Invalid Prompt Processing config for LATISS-prod #4000

Merged
merged 3 commits into from
Dec 16, 2024
Merged

DM-48165: Invalid Prompt Processing config for LATISS-prod #4000

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e7d98bf
Fix corrupted config in Prompt Processing LATISS.
kfindeisen Dec 16, 2024
8691618
Fix cargo-culted comment in Prompt Processing.
kfindeisen Dec 16, 2024
9c19b3e
Modernize alert broker configs for Prompt Processing.
kfindeisen Dec 16, 2024
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: 1 addition & 1 deletion applications/prompt-proto-service-hsc-gpu/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | None, must be set | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-hsc-gpu/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
# @default -- None, must be set
topic: ""
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-hsc/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | None, must be set | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-hsc/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
# @default -- None, must be set
topic: ""
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-latiss/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | None, must be set | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
6 changes: 4 additions & 2 deletions applications/prompt-proto-service-latiss/values-usdfprod-prompt-processing.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,10 @@ prompt-proto-service:
- {survey: cwfs, pipelines: []}
- {survey: cwfs-focus-sweep, pipelines: []}
- {survey: spec-survey, pipelines: []}
- {survey: BLOCK-271, pipelines: []} # photon transfer curve calibrations
- {survey: BLOCK-295, pipelines: []} # the daily calibration sequence as of May 27, 2024
# photon transfer curve calibrations
- {survey: BLOCK-271, pipelines: []}
# the daily calibration sequence as of May 27, 2024
- {survey: BLOCK-295, pipelines: []}
- {survey: "", pipelines: []}
preprocessing: |-
- survey: BLOCK-306
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-latiss/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
# @default -- None, must be set
topic: ""
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcam/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | None, must be set | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcam/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
# @default -- None, must be set
topic: ""
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcomcam/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | `""` | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
4 changes: 1 addition & 3 deletions applications/prompt-proto-service-lsstcomcam/values-usdfprod-prompt-processing.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,7 @@ prompt-proto-service:
imageNotifications:
kafkaClusterAddress: prompt-processing-2-kafka-bootstrap.kafka:9092
topic: rubin-summit-notification
# Scheduler adds an extra 60-80-second delay for first visit in a sequence,
# and files can take up to 20 seconds to arrive. Scheduler delay associated
# with CWFS engineering data, should not apply to other cameras.
# Copied over from LATISS, works well empirically
imageTimeout: 110

apdb:
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcomcam/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
topic: ""

Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcomcamsim/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Prompt Proto Service is an event driven service for processing camera images. Th
|-----|------|---------|-------------|
| prompt-proto-service.additionalVolumeMounts | list | `[]` | Kubernetes YAML configs for extra container volume(s). Any volumes required by other config options are automatically handled by the Helm chart. |
| prompt-proto-service.affinity | object | `{}` | Affinity rules for the prompt processing pods |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev-broker-0.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.server | string | `"usdf-alert-stream-dev.lsst.cloud:9094"` | Server address for the alert stream |
| prompt-proto-service.alerts.topic | string | None, must be set | Topic name where alerts will be sent |
| prompt-proto-service.alerts.username | string | `"kafka-admin"` | Username for sending alerts to the alert stream |
| prompt-proto-service.apdb.config | string | None, must be set | URL to a serialized APDB configuration, or the "label:" prefix followed by the indexed name of such a config. |
Expand Down
2 changes: 1 addition & 1 deletion applications/prompt-proto-service-lsstcomcamsim/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ prompt-proto-service:
# -- Username for sending alerts to the alert stream
username: "kafka-admin"
# -- Server address for the alert stream
server: "usdf-alert-stream-dev-broker-0.lsst.cloud:9094"
server: "usdf-alert-stream-dev.lsst.cloud:9094"
# -- Topic name where alerts will be sent
# @default -- None, must be set
topic: ""
Expand Down
Loading