Skip to content
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

Update OpenTelemetry Documentation with addition of WSO2 API Manager as application integration #5081

Open
wants to merge 8 commits 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.

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

Adding WSO2 opentelemetry integration
Adding contents to WSO2 application integration document
@sanjeewa-malalgoda sanjeewa-malalgoda requested a review from a team August 22, 2024 06:24
Copy link

linux-foundation-easycla bot commented Aug 22, 2024

CLA Not Signed

@theletterf
Copy link
Member

Hi @sanjeewa-malalgoda ! Could you please sign the CLA?

@sanjeewa-malalgoda
Copy link
Author

Hi @sanjeewa-malalgoda ! Could you please sign the CLA?

Hi @theletterf, I have signed CLA. Thank you.

@tiffany76
Copy link
Contributor

I have signed CLA. Thank you.

Hi @sanjeewa-malalgoda, thanks for signing! It looks like one of your commits was made under a different login not covered by the CLA. You might want to squash that commit.

@svrnm
Copy link
Member

svrnm commented Sep 24, 2024

@sanjeewa-malalgoda @isudana please take a look

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

Successfully merging this pull request may close these issues.

5 participants