Skip to content

Observability: Clean up Oshi#931

Open
arklian wants to merge 1 commit into
05-24-observability_remove_opensearchfrom
05-26-observability_clean_up_oshi
Open

Observability: Clean up Oshi#931
arklian wants to merge 1 commit into
05-24-observability_remove_opensearchfrom
05-26-observability_clean_up_oshi

Conversation

@arklian
Copy link
Copy Markdown
Contributor

@arklian arklian commented May 26, 2026

Description of changes

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

Copy link
Copy Markdown
Contributor Author

arklian commented May 26, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@arklian arklian marked this pull request as ready for review May 26, 2026 17:40
@arklian arklian requested a review from a team as a code owner May 26, 2026 17:40
@arklian arklian force-pushed the 05-24-observability_remove_opensearch branch from c65adf9 to 99a6bb5 Compare May 26, 2026 18:38
@arklian arklian force-pushed the 05-26-observability_clean_up_oshi branch from 19035e4 to 5d307ed Compare May 26, 2026 18:38
@github-actions
Copy link
Copy Markdown
Contributor

PR Description Required

Please fill out the Description of changes section of the PR.

@arklian arklian force-pushed the 05-26-observability_clean_up_oshi branch from 5d307ed to 4eca365 Compare May 26, 2026 18:43
@arklian arklian force-pushed the 05-24-observability_remove_opensearch branch from 99a6bb5 to 1ba9fbe Compare May 26, 2026 18:43
@github-actions
Copy link
Copy Markdown
Contributor

PR Description Required

Please fill out the Description of changes section of the PR.

No longer need Oshi to send system metrics to Prometheus, since we can
directly monitor system resource usage of kubernetes nodes.

Keeping the commit sha, since it's still included in all of the metrics
generated by Spring Actuator.
@arklian arklian force-pushed the 05-24-observability_remove_opensearch branch from 1ba9fbe to f126107 Compare May 26, 2026 18:57
@arklian arklian force-pushed the 05-26-observability_clean_up_oshi branch from 4eca365 to 78d7f54 Compare May 26, 2026 18:57
@github-actions
Copy link
Copy Markdown
Contributor

PR Description Required

Please fill out the Description of changes section of the PR.

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.

1 participant