Skip to content

chore: Remove helm from examples validation#964

Merged
acroca merged 2 commits intodapr:mainfrom
acroca:remove-helm-dependency
Mar 25, 2026
Merged

chore: Remove helm from examples validation#964
acroca merged 2 commits intodapr:mainfrom
acroca:remove-helm-dependency

Conversation

@acroca
Copy link
Copy Markdown
Member

@acroca acroca commented Mar 25, 2026

This workflow doesn't use helm. I guess is some left over from previous versions where dapr was installed using helm.

Also fixes the CI by filtering cloudevents < 2.0.0, which just got released and it breaks the grpc extension.

Signed-off-by: Albert Callarisa <albert@diagrid.io>
@acroca acroca requested review from a team as code owners March 25, 2026 10:00
Signed-off-by: Albert Callarisa <albert@diagrid.io>
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.06%. Comparing base (bffb749) to head (0a27f68).
⚠️ Report is 94 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #964      +/-   ##
==========================================
+ Coverage   86.63%   89.06%   +2.43%     
==========================================
  Files          84      104      +20     
  Lines        4473     7410    +2937     
==========================================
+ Hits         3875     6600    +2725     
- Misses        598      810     +212     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@acroca acroca added this pull request to the merge queue Mar 25, 2026
Merged via the queue into dapr:main with commit 5937e28 Mar 25, 2026
22 of 23 checks passed
@dapr-bot
Copy link
Copy Markdown
Collaborator

The backport to release-1.17 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-1.17 release-1.17
# Navigate to the new working tree
cd .worktrees/backport-release-1.17
# Create a new branch
git switch --create backport-964-to-release-1.17
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 5937e2828f3f4c8c128bae31ae9c38d4610ea459
# Push it to GitHub
git push --set-upstream origin backport-964-to-release-1.17
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-1.17

Then, create a pull request where the base branch is release-1.17 and the compare/head branch is backport-964-to-release-1.17.

@dapr-bot
Copy link
Copy Markdown
Collaborator

The backport to release-1.16 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-release-1.16 release-1.16
# Navigate to the new working tree
cd .worktrees/backport-release-1.16
# Create a new branch
git switch --create backport-964-to-release-1.16
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 5937e2828f3f4c8c128bae31ae9c38d4610ea459
# Push it to GitHub
git push --set-upstream origin backport-964-to-release-1.16
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-release-1.16

Then, create a pull request where the base branch is release-1.16 and the compare/head branch is backport-964-to-release-1.16.

@acroca acroca deleted the remove-helm-dependency branch March 25, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants