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

Install Yourkit Java Profiler agent in the geoserver base image #414

Merged
merged 1 commit into from
Jan 20, 2024

Conversation

groldan
Copy link
Member

@groldan groldan commented Jan 20, 2024

Allow profiling the applications directly from the containers

@groldan groldan added the docker Issues related to docker images or docker composition label Jan 20, 2024
Allow profiling the applications directly from the containers
@groldan groldan merged commit feabe78 into geoserver:main Jan 20, 2024
2 checks passed
@groldan groldan deleted the yjp branch January 20, 2024 04:26
# Then run with docker compose --env-file .env --env-file .env.yjp -f ... up -d
# See https://www.yourkit.com/docs/connection-broker/2023.9/help/

YJP_OPTS=-agentpath:/usr/local/YourKit-JavaProfiler-2023.9/bin/linux-x86-64/libyjpagent.so=broker_url=https://broker.yourkit.com/<broker-id>/,broker_token=<broker-token>
Copy link
Collaborator

Choose a reason for hiding this comment

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

may need to revisit in the context of multiplatform docker images (#406)

Copy link
Collaborator

@pmauduit pmauduit left a comment

Choose a reason for hiding this comment

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

just a remark on native libraries usage and platform dependency

@groldan
Copy link
Member Author

groldan commented Jan 25, 2024

noted. thanks. This is only to aid development though, so when we have multiplatform images and a developer running the profiler on arm we can revisit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Issues related to docker images or docker composition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants