Skip to content

Conversation

@savanipoojan78
Copy link
Contributor

@savanipoojan78 savanipoojan78 commented May 26, 2025

PR implements healths check for opentelemetry.io/OpenTelemetryCollector CRDs

opentelemetry.io/OpenTelemetryCollector:

  • health.lua
  • tests

@savanipoojan78 savanipoojan78 requested a review from a team as a code owner May 26, 2025 14:09
@bunnyshell
Copy link

bunnyshell bot commented May 26, 2025

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

@savanipoojan78 savanipoojan78 changed the title OpenTelemetryCollector CRD Healthcheck chore: OpenTelemetryCollector CRD Healthcheck May 26, 2025
@codecov
Copy link

codecov bot commented May 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@0205eaf). Learn more about missing BASE report.
⚠️ Report is 433 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master   #23156   +/-   ##
=========================================
  Coverage          ?   59.99%           
=========================================
  Files             ?      343           
  Lines             ?    57888           
  Branches          ?        0           
=========================================
  Hits              ?    34729           
  Misses            ?    20375           
  Partials          ?     2784           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@crenshaw-dev crenshaw-dev changed the title chore: OpenTelemetryCollector CRD Healthcheck feat(health): OpenTelemetryCollector CRD Healthcheck May 27, 2025
@crenshaw-dev crenshaw-dev added the component:health-check Issue related to built-in Health Check customizations label May 27, 2025
Copy link
Member

@crenshaw-dev crenshaw-dev left a comment

Choose a reason for hiding this comment

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

Thanks!

@crenshaw-dev crenshaw-dev merged commit 65464d8 into argoproj:master May 27, 2025
34 checks passed
tylerrosnett pushed a commit to StateFarmIns/argo-cd that referenced this pull request May 27, 2025
chansuke pushed a commit to chansuke/argo-cd that referenced this pull request Jun 4, 2025
dsuhinin pushed a commit to dsuhinin/argo-cd that referenced this pull request Jun 16, 2025
dsuhinin pushed a commit to dsuhinin/argo-cd that referenced this pull request Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:health-check Issue related to built-in Health Check customizations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants