Skip to content

GEODE-10440: add Micrometer Observation support - #8095

Open
makarandhinge wants to merge 3 commits into
apache:developfrom
makarandhinge:develop
Open

makarandhinge wants to merge 3 commits into
apache:developfrom
makarandhinge:develop

Conversation

@makarandhinge

Copy link
Copy Markdown

For all changes, please confirm:

  • Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
  • Has your PR been rebased against the latest commit within the target branch (typically develop)?
  • Is your initial contribution a single, squashed commit?
  • Does gradlew build run cleanly?
  • Have you written or updated unit tests to verify your changes?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Description

Adds Micrometer Observation support to Apache Geode.

The changes introduce observation support for cache and server operations using Micrometer's ObservationRegistry.

Changes

  • Added Geode observation services and session management.
  • Added ObservationRegistry support to the cache lifecycle.
  • Added Micrometer observations for region get and put operations.
  • Added observation support around server command processing.
  • Added error recording for failed observations.
  • Added the Micrometer Observation dependency.
  • Added integration tests covering observation creation and error handling.

Testing

  • Existing geode-core tests pass.
  • Added integration tests for Micrometer Observation support.
  • Verified observations are created for region get and put operations.
  • Verified failed operations record errors correctly.

@JinwooHwang

Copy link
Copy Markdown
Contributor

Hi @makarandhinge , could you please address the failures in the integration test? Thank you.

@makarandhinge

Copy link
Copy Markdown
Author

Hi @makarandhinge , could you please address the failures in the integration test? Thank you.

Hi @JinwooHwang,
I had address the issue, please run the workflow

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.

2 participants