Skip to content

fix/target-allocator: check CRDs availability in the cluster#4118

Merged
swiatekm merged 17 commits intoopen-telemetry:mainfrom
cazorla19:check-crd-availability
Jul 10, 2025
Merged

fix/target-allocator: check CRDs availability in the cluster#4118
swiatekm merged 17 commits intoopen-telemetry:mainfrom
cazorla19:check-crd-availability

Conversation

@cazorla19
Copy link
Contributor

Description:

target-allocator: check CRDs availability in the cluster before querying them in order to prevent an unhandled error

Link to tracking Issue(s):

Testing:

tested on local k8s cluster to ensure the error is gone

@cazorla19 cazorla19 requested a review from a team as a code owner June 18, 2025 14:19
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Contributor

@iblancasa iblancasa left a comment

Choose a reason for hiding this comment

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

Please add a changelog, fix the lint and add a test. Thanks!

@cazorla19 cazorla19 requested a review from swiatekm July 9, 2025 11:30
@cazorla19 cazorla19 requested a review from swiatekm July 10, 2025 10:05
@github-actions
Copy link
Contributor

github-actions bot commented Jul 10, 2025

E2E Test Results

 33 files  ±0  221 suites  ±0   3h 39m 56s ⏱️ - 2m 30s
 84 tests ±0   84 ✅ ±0  0 💤 ±0  0 ❌ ±0 
221 runs  ±0  221 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 33749c2. ± Comparison against base commit dfb5d81.

♻️ This comment has been updated with latest results.

@cazorla19
Copy link
Contributor Author

@swiatekm @jaronoff97 I think one of you needs to merge it :)

@swiatekm swiatekm merged commit ef60b57 into open-telemetry:main Jul 10, 2025
83 of 87 checks passed
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.

target-allocator doesn't check if CRDs exist before querying them

4 participants