mirror of
https://github.com/hanzoai/tasks.git
synced 2026-11-03 03:56:04 +00:00
## What changed?
Add chasm Archetype tag to dlq metrics.
## Why?
Need a way to filter CHASM tasks by Archetype, through use of indexed
labels in Loki. Instead of filtering queries like
`{k8s_component="history"} |~ "ChasmPure" |~ "Scheduler" |~ "error|fail"
| json`, we can filter using
`{k8s_component="history",archetype="Scheduler"} ` for better
performance.
## How did you test it?
- [X] built
- [X] run locally and tested manually
- [X] covered by existing tests
- [X] added new unit test(s)
- [ ] added new functional test(s)
156 lines
4.6 KiB
Go
156 lines
4.6 KiB
Go
package queues
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"fmt"
|
|
"sync"
|
|
|
|
"go.temporal.io/server/chasm"
|
|
"go.temporal.io/server/common/log"
|
|
"go.temporal.io/server/common/log/tag"
|
|
"go.temporal.io/server/common/metrics"
|
|
"go.temporal.io/server/common/namespace"
|
|
"go.temporal.io/server/common/persistence"
|
|
"go.temporal.io/server/service/history/tasks"
|
|
)
|
|
|
|
type (
|
|
// DLQWriter can be used to write tasks to the DLQ.
|
|
DLQWriter struct {
|
|
dlqWriter QueueWriter
|
|
metricsHandler metrics.Handler
|
|
logger log.SnTaggedLogger
|
|
namespaceRegistry namespace.Registry
|
|
chasmRegistry *chasm.Registry
|
|
enqueueMutex sync.Map // map[persistence.QueueKey]*sync.Mutex for per-queue locking
|
|
}
|
|
// QueueWriter is a subset of persistence.HistoryTaskQueueManager.
|
|
QueueWriter interface {
|
|
CreateQueue(
|
|
ctx context.Context,
|
|
request *persistence.CreateQueueRequest,
|
|
) (*persistence.CreateQueueResponse, error)
|
|
EnqueueTask(
|
|
ctx context.Context,
|
|
request *persistence.EnqueueTaskRequest,
|
|
) (*persistence.EnqueueTaskResponse, error)
|
|
}
|
|
)
|
|
|
|
var (
|
|
ErrSendTaskToDLQ = errors.New("failed to send task to DLQ")
|
|
ErrCreateDLQ = errors.New("failed to create DLQ")
|
|
)
|
|
|
|
// NewDLQWriter returns a DLQ which will write to the given QueueWriter.
|
|
func NewDLQWriter(
|
|
w QueueWriter,
|
|
h metrics.Handler,
|
|
l log.SnTaggedLogger,
|
|
r namespace.Registry,
|
|
cr *chasm.Registry,
|
|
) *DLQWriter {
|
|
return &DLQWriter{
|
|
dlqWriter: w,
|
|
metricsHandler: h,
|
|
logger: l,
|
|
namespaceRegistry: r,
|
|
chasmRegistry: cr,
|
|
}
|
|
}
|
|
|
|
// WriteTaskToDLQ writes a task to the DLQ, creating the underlying queue if it doesn't already exist.
|
|
func (q *DLQWriter) WriteTaskToDLQ(
|
|
ctx context.Context,
|
|
sourceCluster, targetCluster string,
|
|
sourceShardID int,
|
|
task tasks.Task,
|
|
isNamespaceActive bool,
|
|
) error {
|
|
queueKey := persistence.QueueKey{
|
|
QueueType: persistence.QueueTypeHistoryDLQ,
|
|
Category: task.GetCategory(),
|
|
SourceCluster: sourceCluster,
|
|
TargetCluster: targetCluster,
|
|
}
|
|
_, err := q.dlqWriter.CreateQueue(ctx, &persistence.CreateQueueRequest{
|
|
QueueKey: queueKey,
|
|
})
|
|
if err != nil {
|
|
if !errors.Is(err, persistence.ErrQueueAlreadyExists) {
|
|
return fmt.Errorf("%w: %v", ErrCreateDLQ, err)
|
|
}
|
|
}
|
|
|
|
resp, err := func() (*persistence.EnqueueTaskResponse, error) {
|
|
// Acquire a process-level lock for this specific DLQ to prevent concurrent writes
|
|
// from multiple shards causing CAS conflicts in the persistence layer.
|
|
mu := q.getQueueMutex(queueKey)
|
|
mu.Lock()
|
|
defer mu.Unlock()
|
|
|
|
return q.dlqWriter.EnqueueTask(ctx, &persistence.EnqueueTaskRequest{
|
|
QueueType: queueKey.QueueType,
|
|
SourceCluster: queueKey.SourceCluster,
|
|
TargetCluster: queueKey.TargetCluster,
|
|
Task: task,
|
|
SourceShardID: sourceShardID,
|
|
})
|
|
}()
|
|
if err != nil {
|
|
return fmt.Errorf("%w: %v", ErrSendTaskToDLQ, err)
|
|
}
|
|
|
|
nsMetricTag := metrics.NamespaceUnknownTag()
|
|
var nsLogTag tag.Tag
|
|
ns, err := q.namespaceRegistry.GetNamespaceByID(namespace.ID(task.GetNamespaceID()))
|
|
if err != nil {
|
|
q.logger.Warn("Failed to get namespace name while trying to write a task to DLQ",
|
|
tag.WorkflowNamespace(task.GetNamespaceID()),
|
|
tag.Error(err),
|
|
)
|
|
nsLogTag = tag.WorkflowNamespaceID(task.GetNamespaceID())
|
|
} else {
|
|
nsMetricTag = metrics.NamespaceTag(ns.Name().String())
|
|
nsLogTag = tag.WorkflowNamespace(ns.Name().String())
|
|
}
|
|
|
|
// "passive" means the namespace is in standby mode and only replicates data
|
|
namespaceState := metrics.PassiveNamespaceStateTagValue
|
|
if isNamespaceActive {
|
|
namespaceState = metrics.ActiveNamespaceStateTagValue
|
|
}
|
|
taskType := GetTaskTypeTagValue(task, isNamespaceActive, q.chasmRegistry)
|
|
metrics.DLQWrites.With(q.metricsHandler).Record(
|
|
1,
|
|
metrics.TaskCategoryTag(task.GetCategory().Name()),
|
|
metrics.NamespaceStateTag(namespaceState),
|
|
nsMetricTag,
|
|
metrics.TaskTypeTag(taskType),
|
|
metrics.OperationTag(taskType),
|
|
getArchetypeTag(task, q.chasmRegistry),
|
|
)
|
|
q.logger.Warn("Task enqueued to DLQ",
|
|
tag.DLQMessageID(resp.Metadata.ID),
|
|
tag.SourceCluster(sourceCluster),
|
|
tag.TargetCluster(targetCluster),
|
|
tag.TaskType(task.GetType()),
|
|
tag.String("task-category", task.GetCategory().Name()),
|
|
nsLogTag,
|
|
)
|
|
return nil
|
|
}
|
|
|
|
// getQueueMutex returns a per-queue mutex, creating it if it doesn't exist.
|
|
// This provides process-level locking to serialize concurrent writes to the same queue.
|
|
func (q *DLQWriter) getQueueMutex(queueKey persistence.QueueKey) *sync.Mutex {
|
|
if mu, ok := q.enqueueMutex.Load(queueKey); ok {
|
|
return mu.(*sync.Mutex) //nolint:revive
|
|
}
|
|
|
|
newMutex := &sync.Mutex{}
|
|
actual, _ := q.enqueueMutex.LoadOrStore(queueKey, newMutex)
|
|
return actual.(*sync.Mutex) //nolint:revive
|
|
}
|