Skip to content

Update dapr v1.15#1242

Merged
cicoyle merged 3 commits into
dapr:masterfrom
cicoyle:update-dapr-v1.15
Mar 14, 2025
Merged

Update dapr v1.15#1242
cicoyle merged 3 commits into
dapr:masterfrom
cicoyle:update-dapr-v1.15

Conversation

@cicoyle

@cicoyle cicoyle commented Mar 7, 2025

Copy link
Copy Markdown
Contributor

want to ensure the release doesnt encounter any issues. Going to bump versions bc I was seeing the actor timer issue that runtime v1.15.2 fixed. Saw v1.15.3 is out, so going to use latest.

cicoyle added 2 commits March 7, 2025 14:39
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
@cicoyle cicoyle requested review from a team as code owners March 7, 2025 20:42

@artur-ciocanu artur-ciocanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, let's see if all the tests are 🟢

@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle I have run the "Auto Validate Examples" multiple times, but it keeps failing, I am not sure what has changed in the 1.15 but apparently the output is no longer the same since the validation fails.

@cicoyle

cicoyle commented Mar 14, 2025

Copy link
Copy Markdown
Contributor Author

I'll re run it a few times and see whats happening and potentially flag it as a regression if its consistent

@artur-ciocanu artur-ciocanu mentioned this pull request Mar 14, 2025
3 tasks
@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle this PR seems to be 🟢 as well, I think it is safe to merge it. Thank you.

@cicoyle

cicoyle commented Mar 14, 2025

Copy link
Copy Markdown
Contributor Author

I re-ran it like twice, so I think we are good here as well. Will merge

@cicoyle cicoyle merged commit a03f220 into dapr:master Mar 14, 2025
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.

3 participants