Skip to content

Adding WSO2 opentelemetry integration #6832

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sanjeewa-malalgoda
Copy link

Description

This pull request updates the documentation URL for the OpenTelemetry integration with WSO2 API Manager. The previous URL led to a general page, while the new URL directs users specifically to the page detailing how to monitor the WSO2 API Manager using OpenTelemetry. This change ensures that users have direct access to relevant and detailed information about implementing OpenTelemetry for enhanced observability.

Impact

  • Provides users with direct and precise documentation, improving ease of access and user experience.
  • Ensures that the information is up-to-date and relevant to current integration capabilities between WSO2 API Manager and OpenTelemetry.

I have signed CLA.

Thank you for considering this update. Please let me know if there are any further adjustments needed.

Copy link
Member

@svrnm svrnm left a comment

Choose a reason for hiding this comment

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

thanks, here's a first pass of reviews

Comment on lines +14 to +16
urls:
website: https://wso2.com/api-management/
docs: https://apk.docs.wso2.com/en/latest/administration/distributed-tracing/#opentelemetry-protocol-otlp
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
urls:
website: https://wso2.com/api-management/
docs: https://apk.docs.wso2.com/en/latest/administration/distributed-tracing/#opentelemetry-protocol-otlp
urls:
website: https://wso2.com/api-management/
docs: https://apk.docs.wso2.com/en/latest/administration/distributed-tracing/#opentelemetry-protocol-otlp
repo:

Since this is open source, please add the repo to the related project

Comment on lines +14 to +16
urls:
website: https://wso2.com/api-management/
docs: https://apim.docs.wso2.com/en/latest/observe/api-manager/traces/monitoring-with-opentelemetry/
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
urls:
website: https://wso2.com/api-management/
docs: https://apim.docs.wso2.com/en/latest/observe/api-manager/traces/monitoring-with-opentelemetry/
urls:
website: https://wso2.com/api-management/
docs: https://apim.docs.wso2.com/en/latest/observe/api-manager/traces/monitoring-with-opentelemetry/
repo:

Since this is open source, please add the repo to the related project

Comment on lines +14 to +16
urls:
website: https://wso2.com/integration/
docs: https://mi.docs.wso2.com/en/latest/observe-and-manage/classic-observability-traces/monitoring-with-opentelemetry-mi/
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
urls:
website: https://wso2.com/integration/
docs: https://mi.docs.wso2.com/en/latest/observe-and-manage/classic-observability-traces/monitoring-with-opentelemetry-mi/
urls:
website: https://wso2.com/integration/
docs: https://mi.docs.wso2.com/en/latest/observe-and-manage/classic-observability-traces/monitoring-with-opentelemetry-mi/
repo:

Since this is open source, please add the repo to the related project

@@ -0,0 +1,18 @@
title: WSO2 Micro Integrator
registryType: application integration
language: Java
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
language: Java
language: java

@@ -0,0 +1,18 @@
title: WSO2 API Manager
registryType: application integration
language: Java
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
language: Java
language: java

@@ -0,0 +1,18 @@
title: WSO2 Kubernetes Gateway
registryType: application integration
language: Java
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
language: Java
language: java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants