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

Add gae auto-configure/auto-instrumentation sample #243

Closed
wants to merge 6 commits into from

Conversation

psx95
Copy link
Contributor

@psx95 psx95 commented May 9, 2023

Adds a stand-alone example that runs OTel Autoinstrumentation, auto-configured to export telemetry to Google Cloud from Google App Engine.

@psx95 psx95 force-pushed the add-gae-example branch from 1c5eff5 to f6afa44 Compare May 9, 2023 02:44
@psx95 psx95 requested review from jsuereth and punya May 9, 2023 02:52
@psx95 psx95 added the enhancement New feature or request label May 9, 2023
@psx95 psx95 marked this pull request as ready for review May 9, 2023 02:52
@psx95 psx95 requested a review from a team as a code owner May 9, 2023 02:52
@@ -0,0 +1,63 @@
Google AppEngine Standard with OTel Auto-instrumentation
============================
**This sample app is a Maven project and builds separately from the rest of the rest of the examples.**
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is it built using Maven? Gradle appears to be supported for AppEngine: https://cloud.google.com/appengine/docs/standard/java-gen2/using-gradle.

examples/gae-autoinstrument-example/nbactions.xml Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

Please have the build download these binaries rather than checking them into Github.

============================
**This sample app is a Maven project and builds separately from the rest of the rest of the examples.**

This example Java app runs on GAE standard environment and is instrumented using OpenTelemetry's [auto-instrumentation](https://github.com/open-telemetry/opentelemetry-java-instrumentation) agent.
Copy link
Contributor

Choose a reason for hiding this comment

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

How much of this sample is really specific to AppEngine? Why couldn't one of our other samples be deployed to AppEngine?

@psx95 psx95 force-pushed the add-gae-example branch from dad38c1 to d275756 Compare May 9, 2023 15:12
@psx95 psx95 closed this Apr 29, 2024
@psx95 psx95 deleted the add-gae-example branch April 29, 2024 20:32
@psx95 psx95 restored the add-gae-example branch April 29, 2024 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants