Skip to content

Commit

Permalink
Do not emit execution id label by default in single binary (#5704)
Browse files Browse the repository at this point in the history
* Do not emit execution id label by default in single binary

Signed-off-by: Eduardo Apolinario <[email protected]>

* Update flytestdlib/contextutils/context.go

Co-authored-by: Thomas J. Fan <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>

---------

Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
Co-authored-by: Eduardo Apolinario <[email protected]>
Co-authored-by: Thomas J. Fan <[email protected]>
  • Loading branch information
3 people authored Aug 29, 2024
1 parent 5c147ef commit 27d9746
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion cmd/single/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -249,6 +249,6 @@ func init() {
RootCmd.AddCommand(startCmd)
// Set Keys
labeled.SetMetricKeys(contextutils.AppNameKey, contextutils.ProjectKey, contextutils.DomainKey,
contextutils.ExecIDKey, contextutils.WorkflowIDKey, contextutils.NodeIDKey, contextutils.TaskIDKey,
contextutils.WorkflowIDKey, contextutils.NodeIDKey, contextutils.TaskIDKey,
contextutils.TaskTypeKey, common.RuntimeTypeKey, common.RuntimeVersionKey, storage.FailureTypeLabel)
}
17 changes: 9 additions & 8 deletions flytestdlib/contextutils/context.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,15 @@ import (
type Key string

const (
AppNameKey Key = "app_name"
NamespaceKey Key = "ns"
TaskTypeKey Key = "tasktype"
ProjectKey Key = "project"
DomainKey Key = "domain"
WorkflowIDKey Key = "wf"
NodeIDKey Key = "node"
TaskIDKey Key = "task"
AppNameKey Key = "app_name"
NamespaceKey Key = "ns"
TaskTypeKey Key = "tasktype"
ProjectKey Key = "project"
DomainKey Key = "domain"
WorkflowIDKey Key = "wf"
NodeIDKey Key = "node"
TaskIDKey Key = "task"
// Adding the ExecIDKey label to a metric will cause higher cardinality. Use with caution.
ExecIDKey Key = "exec_id"
JobIDKey Key = "job_id"
PhaseKey Key = "phase"
Expand Down

0 comments on commit 27d9746

Please sign in to comment.