Skip to content

[Feature] enhancement service graph with virtual node.#11

Merged
JaredTan95 merged 3 commits into
insight-mainfrom
feature_virtual_node
Dec 20, 2022
Merged

[Feature] enhancement service graph with virtual node.#11
JaredTan95 merged 3 commits into
insight-mainfrom
feature_virtual_node

Conversation

@JaredTan95

Copy link
Copy Markdown

Description:

Link to tracking Issue:

Testing:

Documentation:

@JaredTan95 JaredTan95 changed the title [Feature] enhancement service graph with virtual node. [WIP][Feature] enhancement service graph with virtual node. Dec 1, 2022
@github-actions

Copy link
Copy Markdown

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Dec 16, 2022
@JaredTan95

Copy link
Copy Markdown
Author

/label -Stale

@JaredTan95 JaredTan95 removed the Stale label Dec 16, 2022
@JaredTan95 JaredTan95 merged this pull request into insight-main Dec 20, 2022
@JaredTan95 JaredTan95 deleted the feature_virtual_node branch December 20, 2022 01:49
@JaredTan95 JaredTan95 changed the title [WIP][Feature] enhancement service graph with virtual node. [Feature] enhancement service graph with virtual node. Dec 20, 2022
JaredTan95 added a commit that referenced this pull request Dec 22, 2022
* Fix servcegraph (#6)

* fix

* fix

* update

* enhancement service graph with virtual node.

Signed-off-by: Jared Tan <jian.tan@daocloud.io>

Signed-off-by: Jared Tan <jian.tan@daocloud.io>
Co-authored-by: Murphy Chen <minquan.chen@daocloud.io>
JaredTan95 pushed a commit that referenced this pull request Apr 11, 2025
…ry#38146)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

This PR adds support for configuring metric temporality (delta or
cumulative) in the telemetrygen tool. This allows users to generate
metrics with different temporality types, which is particularly useful
for testing different metric collection scenarios.

##### Changes

- Added `temporalityType` flag for metrics pipeline that accepts values:
`delta` or `cumulative`


<!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#38073 

<!--Describe what testing was performed and which tests were added.-->
#### Testing

Testing was performed by setting up as simple collector with `otlp
`receiver and `debug` exporter

##### Sum Datapoint

```
Resource SchemaURL: https://opentelemetry.io/schemas/1.13.0
Resource attributes:
     -> service.name: Str(telemetrygen)
ScopeMetrics #0
ScopeMetrics SchemaURL: 
InstrumentationScope  
Metric #0
Descriptor:
     -> Name: gen
     -> Description: 
     -> Unit: 
     -> DataType: Histogram
     -> AggregationTemporality: Delta
```

##### Histogram Datapoint

```
Count: 0
Sum: 3940.000000
ExplicitBounds #0: 0.000000
ExplicitBounds #1: 5.000000
ExplicitBounds #2: 10.000000
ExplicitBounds #3: 25.000000
ExplicitBounds #4: 50.000000
ExplicitBounds #5: 75.000000
ExplicitBounds #6: 100.000000
ExplicitBounds #7: 250.000000
ExplicitBounds #8: 500.000000
ExplicitBounds #9: 750.000000
ExplicitBounds #10: 1000.000000
ExplicitBounds #11: 2500.000000
ExplicitBounds #12: 5000.000000
ExplicitBounds #13: 7500.000000
ExplicitBounds #14: 10000.000000
Buckets #0, Count: 0
Buckets #1, Count: 0
Buckets #2, Count: 1
Buckets #3, Count: 0
Buckets #4, Count: 0
Buckets #5, Count: 0
Buckets #6, Count: 3
Buckets #7, Count: 4
Buckets #8, Count: 1
Buckets #9, Count: 1
Buckets #10, Count: 0
Buckets #11, Count: 0
Buckets #12, Count: 0
Buckets #13, Count: 0
Buckets #14, Count: 0
```


- [x] Need to add tests for the config

---------

Signed-off-by: Nikos Angelopoulos <nikolaos.angelopoulos@grafana.com>
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