Skip to content

Conversation

@GSVarsha
Copy link
Contributor

@GSVarsha GSVarsha commented Dec 15, 2025

What's there in this PR?

Added stack-trace support for all configuration sources in the following precedence:

 ENV vars > YAML config (INSTANA_CONFIG_PATH) > in-code config > Agent config

Runtime Stack Trace Collection Flow:

temp_image_1765877137141

Note

We support technology-specific overrides (for all EXIT spans!) for all the mapping based config sources (YAML, in-code and Agent)

@GSVarsha GSVarsha added this to the H1-2026 milestone Dec 15, 2025
@GSVarsha GSVarsha self-assigned this Dec 15, 2025
@GSVarsha GSVarsha added WIP work in progress do NOT merge Depends on another PR or WIP unittest feature labels Dec 15, 2025
@GSVarsha GSVarsha force-pushed the stack-trace-yaml-config branch from bad6b7b to 929a7df Compare December 16, 2025 06:58
@GSVarsha GSVarsha force-pushed the stack-trace-yaml-config branch from b82a852 to c5bfe08 Compare December 16, 2025 08:28
@GSVarsha GSVarsha changed the title Stack trace yaml config [Part 2] feat: Implement Span stack trace specification Dec 16, 2025
@sonarqubecloud
Copy link

@GSVarsha GSVarsha removed WIP work in progress do NOT merge Depends on another PR or WIP labels Dec 16, 2025
@GSVarsha GSVarsha marked this pull request as ready for review December 16, 2025 10:00
@GSVarsha GSVarsha requested a review from a team as a code owner December 16, 2025 10:00
@GSVarsha GSVarsha added enhancement FUP Fair Usage Policy and removed unittest labels Dec 16, 2025
Copy link
Member

@pvital pvital left a comment

Choose a reason for hiding this comment

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

Just three recommendations.

return [], []


def get_tracing_root_key(config_data: Dict[str, Any]) -> Union[str, None]:
Copy link
Member

Choose a reason for hiding this comment

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

It's a best practice to use Optional[X] instead of Union[X, None]:

Suggested change
def get_tracing_root_key(config_data: Dict[str, Any]) -> Union[str, None]:
def get_tracing_root_key(config_data: Dict[str, Any]) -> Optional[str]:

return [], []


def validate_stack_trace_level(level_value: Any, context: str = "") -> Union[str, None]:
Copy link
Member

Choose a reason for hiding this comment

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

Same as here.

return None


def validate_stack_trace_length(length_value: Any, context: str = "") -> Union[int, None]:
Copy link
Member

Choose a reason for hiding this comment

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

Same as here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants