Workflow middlewares don't raise error on workflow execution failure
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia con worker.rb e metrics_workflow_middleware.rb per capire come viene registrato il middleware del workflow e come vengono gestiti attualmente i fallimenti intorno a yield. Segui il percorso di esecuzione del workflow che invoca il middleware e determina come sia possibile osservare lì un’esecuzione fallita. Il lavoro è completato quando il middleware può emettere workflow_failed per le esecuzioni fallite, mantenendo workflow_completed per quelle riuscite.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I'd like to emit the metric workflow_failed when the workflow execution fails (replicating metrics emitted by other SDKs). To achieve that, I registered a workflow middleware in my worker. In my middleware, I rescue errors and emit this metric.
The problem is that the workflow middleware doesn't raise an error when the workflow execution fails. AFAIK, there is no way of knowing if the workflow execution failed in the context of a workflow middleware. Conversely, in the activity middleware, errors are raised so it's possible to emit the metric activity_exectuion_failed. How can I emit the workflow_failed metrics when the workflow execution fails?
Setup:
worker.rb:
class Worker < Temporal::Worker
def self.start
worker = Temporal::Worker.new
# configure...
worker.add_workflow_middleware(MetricsWorkflowMiddleware)
worker.start
end
end
metrics_workflow_middleware.rb:
class MetricsWorkflowMiddleware
def call(metadata)
yield
Statsd.increment('workflow_completed')
rescue StandardError
Statsd.increment('workflow_failed')
raise
end
end
- Lingua principale
- Ruby
- Stelle
- 288
- Fork
- 113
- Merge medio
- 10g 15h
- PR unite (30g)
- 2
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di coinbase/temporal-ruby
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
coinbase/temporal-ruby#361 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
coinbase/temporal-ruby#341 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
coinbase/temporal-ruby#328 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
coinbase/temporal-ruby#326 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
coinbase/temporal-ruby#324 · 3 commenti · 2 reazioni ·
Tutte le issue di coinbase/temporal-ruby
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
riscv/riscv-unified-db#2626 ·
-
Component: GLib
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
ds-drift
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
we-promise/sure#3693 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
simp/pupmod-simp-simp#395 ·