8000 Start a new workflow trace when triggered from UI by raymonddenhaan · Pull Request #6606 · elsa-workflows/elsa-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Start a new workflow trace when triggered from UI #6606

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.

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

Merged
merged 2 commits into from
Apr 23, 2025

Conversation

raymonddenhaan
Copy link
Contributor
@raymonddenhaan raymonddenhaan commented Apr 22, 2025

Adds an option to use a dummy activity as a hack to prevent other middleware of reinstating the parent activity


This change is Reviewable

Comment on lines 12 to 21
/// <summary>
/// When active this will create a new root Activity if the current Activity has a remote parent.
/// </summary>
public bool UseNewRootActivityForRemoteParent { get; set; } = true;

/// <summary>
/// Determines if instead of a empty parent, a dummy parent Activity should be used to create a new Root Activity.
/// </summary>
/// <remarks>This is needed when middleware is active that uses the previous Parent Activity despite creating an Activity based on an empty Parent.</remarks>
public bool UseDummyParentActivityAsRootSpan { get; set; } = false;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we include a comment that these settings exist for the purpose of mitigating a limitation with the datadog autoinstrumentation?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't mention Datadog directly, however I tried to use the message in the remarks tag to get this point across. Is there anything you are missing in this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unless I misunderstood, this effect is directly tied to what Datadog's auto instrumentation is doing, so for me it would be helpful to have that context.

@raymonddenhaan raymonddenhaan merged commit b9d4eb8 into rc/3.4.0 Apr 23, 2025
2 checks passed
@raymonddenhaan raymonddenhaan deleted the fix/otel_spans branch April 23, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0