Skip to content
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

Remove deprecated RenameActorReminder methods #6945

Merged
merged 5 commits into from Sep 20, 2023

Conversation

ItalyPaleAle
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle commented Sep 19, 2023

Fixes the issue #6711

Methods were deprecated in 1.9 and slated to be removed from 1.13.

Fixes dapr#6711

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
@ItalyPaleAle ItalyPaleAle added breaking-change This is a breaking change autoupdate DaprBot will keep the Pull Request up to date with master branch labels Sep 19, 2023
@dapr-bot

This comment was marked as resolved.

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.10% 🎉

Comparison is base (8bc41e7) 64.84% compared to head (60c0f97) 64.95%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6945      +/-   ##
==========================================
+ Coverage   64.84%   64.95%   +0.10%     
==========================================
  Files         228      228              
  Lines       20823    20731      -92     
==========================================
- Hits        13503    13466      -37     
+ Misses       6189     6143      -46     
+ Partials     1131     1122       -9     
Files Changed Coverage Δ
pkg/actors/actors.go 52.97% <ø> (+0.30%) ⬆️
pkg/actors/actors_mock.go 0.00% <ø> (ø)
pkg/actors/internal/requests.go 64.91% <ø> (ø)
pkg/actors/reminders/reminders.go 62.41% <ø> (-0.65%) ⬇️
pkg/actors/requests_responses.go 79.16% <ø> (ø)
pkg/grpc/api.go 74.15% <ø> (+1.23%) ⬆️
pkg/grpc/endpoints.go 100.00% <ø> (ø)
pkg/http/api.go 78.01% <ø> (-0.62%) ⬇️
pkg/placement/membership.go 53.77% <ø> (+3.77%) ⬆️

... and 8 files with indirect coverage changes

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

Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
@ItalyPaleAle
Copy link
Contributor Author

/ok-to-test

@dapr-bot
Copy link
Collaborator

dapr-bot commented Sep 19, 2023

Dapr E2E test

🔗 Link to Action run

Commit ref: d1e2508

✅ Build succeeded for linux/amd64

  • Image tag: dapre2e905b490371l
  • Test image tag: dapre2e905b490371l

✅ Build succeeded for windows/amd64

  • Image tag: dapre2e905b490371w
  • Test image tag: dapre2e905b490371w

✅ Infrastructure deployed

Cluster Resource group name Azure region
Linux Dapr-E2E-dapre2e905b490371l westus3
Windows Dapr-E2E-dapre2e905b490371w westus3
Linux/arm64 Dapr-E2E-dapre2e905b490371la eastus

✅ Tests succeeded on windows/amd64

  • Image tag: dapre2e905b490371w
  • Test image tag: dapre2e905b490371w

✅ Tests succeeded on linux/amd64

  • Image tag: dapre2e905b490371l
  • Test image tag: dapre2e905b490371l

@ItalyPaleAle ItalyPaleAle marked this pull request as ready for review September 19, 2023 21:33
@ItalyPaleAle ItalyPaleAle requested review from a team as code owners September 19, 2023 21:33
Copy link
Contributor

@mukundansundar mukundansundar left a comment

Choose a reason for hiding this comment

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

lgtm

@dapr/maintainers-dapr merging this based off of #6669 (comment)

Copy link
Member

@yaron2 yaron2 left a comment

Choose a reason for hiding this comment

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

LGTM

@mukundansundar
Copy link
Contributor

@ItalyPaleAle Please create a tracking issue if this breaking change requires a change in SDKs as well.

@mukundansundar
Copy link
Contributor

Letting #6711 remain open for any changes that are needed in SDKs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autoupdate DaprBot will keep the Pull Request up to date with master branch breaking-change This is a breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants