Skip to content

docs(DATAGO-128303): observability | documentation#1467

Merged
solace-mdupls merged 2 commits intomainfrom
en/DATAGO-128308
Apr 30, 2026
Merged

docs(DATAGO-128303): observability | documentation#1467
solace-mdupls merged 2 commits intomainfrom
en/DATAGO-128308

Conversation

@enavitan
Copy link
Copy Markdown
Contributor

What is the purpose of this change?

OTEL documentation

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

✅ FOSSA Guard: Licensing (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (1.20.5) • 0 new, 9 total (9 in base)

Scan Report | View Details in FOSSA

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

✅ FOSSA Guard: Vulnerability (SolaceLabs_solace-agent-mesh) • PASSED

Compared against main (1.20.5) • 0 new, 8 total (8 in base)

Scan Report | View Details in FOSSA


db.duration:
buckets: [0.001, 0.01, 0.1, 0.5, 1.0]
exclude_labels: [*] # This metric is effectively disabled
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Do we have special yaml handling for this. I think this would throw an error with a pure yaml parser.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

metric_prefix: sam
```

The `observability.path` field specifies the endpoint path for metrics (default: `/metrics`). The `observability.metric_prefix` field duplicates the top-level `metric_prefix` setting for convenience; both fields have the same effect.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What does top-level mean here? I didn't see any other metric_prefix attribute in the doc.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

great catch! no idea how it sneak in.


- **Agent Mesh Enterprise**: Application metrics with OpenTelemetry is an enterprise-only feature. For information about installing Agent Mesh Enterprise, see [Enterprise Installation](./installation.md).

- **Agent Mesh Version**: Ensure you are running a version of Agent Mesh Enterprise that includes OpenTelemetry metrics support (version 1.20.0 or later).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

1.20.0 sounds like it is relating to OTEL versions correct, and not the SAM Enterprise version?

One note to adjust here - it reads like you're stating that SAM Enterprise needs to be v1.20.0 or above.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good point, i should probably remove version altogether, WDYT?

@sonarqube-solacecloud
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Copy link
Copy Markdown
Collaborator

@gregmeldrum gregmeldrum left a comment

Choose a reason for hiding this comment

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

LG

@solace-mdupls
Copy link
Copy Markdown
Collaborator

@enavitan remember to merge only after we have released SAM Enterprise this week.

@solace-mdupls solace-mdupls changed the title DATAGO-128303 - observability| documentation docs(DATAGO-128303): observability | documentation Apr 30, 2026
@solace-mdupls solace-mdupls merged commit 31897dd into main Apr 30, 2026
28 of 31 checks passed
@solace-mdupls solace-mdupls deleted the en/DATAGO-128308 branch April 30, 2026 20:58
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.

3 participants