-
Notifications
You must be signed in to change notification settings - Fork 614
ci: add workflow to test default targetAllocator image #1721
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
base: main
Are you sure you want to change the base?
ci: add workflow to test default targetAllocator image #1721
Conversation
Hi there 👋 I'm trying to contribute to this project as an individual, but I encountered a message from EasyCLA stating that the Individual CLA option is not enabled for this repository. Could you please advise on how I should proceed? I’m not contributing on behalf of any organization, and I want to ensure my contribution complies with the project's requirements. Thank you very much for your help! Best regards, |
@NavaMaklev Individual CLA signing is definitely allowed in Otel. Probably the best next step to get unstuck is to ask in the CNCF's slack channel for easycla. |
I fix it |
@NavaMaklev I'd like to test that this change does the thing we want. Can you push a commit that changes the operator chart's README, bump the chart's version, and re-generates the examples? This will cause the action you've changed to run. Once the tests pass we'll revert the chart changes. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Summary
Add CI validation for target allocator image in opentelemetry-operator Helm chart.
Motivation
Issue #929 describes a failure in version 0.87.0 where the target allocator image wasn’t deployed.
This change integrates a CI check that:
manager.targetAllocator
via Helm,This prevents regressions in future releases.
Changes
.github/workflows/operator-test.yaml
ct install
with--set manager.targetAllocator.enabled=true
and waits for target allocator pod readiness.Testing
ct install
locally — chart installs successfully with target allocator pod.Related Issue
Closes #929