Skip to content

Merge release-1.15 (v1.15.1) to master#8543

Merged
yaron2 merged 359 commits into
masterfrom
release-1.15
Mar 4, 2025
Merged

Merge release-1.15 (v1.15.1) to master#8543
yaron2 merged 359 commits into
masterfrom
release-1.15

Conversation

@JoshVanL

Copy link
Copy Markdown
Contributor

Merge release-1.15 (v1.15.1) into master after latest release.

elena-kolevska and others added 30 commits January 29, 2025 13:23
Signed-off-by: Elena Kolevska <[email protected]>
Signed-off-by: Elena Kolevska <[email protected]>
Signed-off-by: Elena Kolevska <[email protected]>
Signed-off-by: Luis Rascao <[email protected]>
security: allow override of trust anchors
Update state store actor reminder engine to lock evaluation during
drain events, and give a 2 second grace period concurrently to all
currently active reminders to complete before completing dissemination.

Signed-off-by: joshvanl <[email protected]>
* Workflow: default max parallel items to 1000

Up the number of work items which can be run in parallel from 100 to
1000

Signed-off-by: joshvanl <[email protected]>

* Fix unit test for new default max parallel work items value

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
Co-authored-by: Yaron Schneider <[email protected]>
Remove memory leak whereby workflow actor objects were not be pruned
from the actor table slice.

After an activity fires, delete the object from the actor table.

When a workflow reaches a terminal state or is purged, remove it from
the actor table.

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: Elena Kolevska <[email protected]>
Actors: State store actor reminder drain wait group
Updates scheduler for ETCD to run with mTLS between hosts. Updates
sentry to make dapr-scheduler identity certificates to be signed with
`dapr-scheduler.${ns}.svc` DNS. Updates ETCD embeded config to use mTLS
on this DNS when mTLS is enabled.

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: joshvanl <[email protected]>
yaron2 and others added 22 commits February 25, 2025 14:51
Signed-off-by: yaron2 <[email protected]>
Signed-off-by: yaron2 <[email protected]>
Fix workflow engine starting when actor runtime is disabled
fix workflow name empty on first event
* golang.org/x/crypto: update to v0.35.0

Signed-off-by: joshvanl <[email protected]>

* mod tidy all

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: Fabian Martinez <[email protected]>
Co-authored-by: Josh van Leeuwen <[email protected]>
unmarshal to string

Signed-off-by: Elena Kolevska <[email protected]>
Co-authored-by: Josh van Leeuwen <[email protected]>
Co-authored-by: Dapr Bot <[email protected]>
Signed-off-by: Cassandra Coyle <[email protected]>
* release: add notes

Signed-off-by: Mike Nguyen <[email protected]>

* release: update rust sdk notes and add dapr workflow method deprecation notice

Signed-off-by: Mike Nguyen <[email protected]>

* release: update componentscontrib and go-sdk notes

Signed-off-by: Mike Nguyen <[email protected]>

* release: add rust highlight

Signed-off-by: Mike Nguyen <[email protected]>

* release: bump notes

Signed-off-by: mikeee <[email protected]>

* release: bump

Signed-off-by: Mike Nguyen <[email protected]>

* release: bump relnotes

Signed-off-by: Mike Nguyen <[email protected]>

* release: bump notes

Signed-off-by: Mike Nguyen <[email protected]>

* release: bump notes

Signed-off-by: Mike Nguyen <[email protected]>

* release: bump docs

Signed-off-by: Mike Nguyen <[email protected]>

* release: docs bump

Signed-off-by: Mike Nguyen <[email protected]>

* release: update docs

Signed-off-by: Mike Nguyen <[email protected]>

* release: fix links

Signed-off-by: Mike Nguyen <[email protected]>

---------

Signed-off-by: Mike Nguyen <[email protected]>
Signed-off-by: mikeee <[email protected]>
* add missing grpc call options when invoking actors

Signed-off-by: yaron2 <[email protected]>

* adds release notes + changes options struct

Signed-off-by: yaron2 <[email protected]>

* remove redundant struct

Signed-off-by: yaron2 <[email protected]>

* Adds integration test for max-body-size for actor invocation

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: yaron2 <[email protected]>
Signed-off-by: joshvanl <[email protected]>
Co-authored-by: joshvanl <[email protected]>
* [v1.15] Actors: Don't sent Reentrancy ID header when disabled

Fix sending Actor Reentrancy ID HTTP header when disabled

Problem
Calling `this.StateManager.TryGetStateAsync()` in the DotNet Actor SDK would return stale data during some invocation scenarios.

Impact
The latest Actor state data was not being correctly returned during some Actor invocation scenarios using the DotNet Actor SDK.

Root cause
When Reentrancy was disabled, the Actor Reentrancy ID HTTP header was still being sent to Actor HTTP servers.
The DotNet SDK uses the existence of this HTTP header in logic to determine what state should be returned to the Actor.

Solution
Don't send the Actor Reentrancy ID HTTP header (`"Dapr-Reentrancy-Id"`) when Reentrancy is disabled.

Signed-off-by: joshvanl <[email protected]>

* Fix actor call placement lock due to reentrancy ID HTTP header.

Remove reentancy ID HTTP header check from placement lock during actor
calls. Since we always added the reentrancy ID header in v1.15.0,
`isReentrancy` bool was always true, so `!isReentrancy` was always
false- a no-op to the OR statement. This was added before other changes
were made to the actor locking system, but since it was a no-op, it was
never caught and removed. Now the reentrancy ID header is (correctly)
optionally added to keep backwards compatibility, and the previous actor
engine state this check was catering for no longer exists, this check
actually breaks the runtime.

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
@JoshVanL
JoshVanL requested review from a team as code owners February 28, 2025 18:48
…#8546)

* [v1.15] Actors: Always attempt to connect to placement if configured.

Fixes a regression whereby daprd would no longer connect to placement if
an Actor state store is not configured. This meant an app which did not
have an actor state store configured (& host Actors), could not invoke
Actors on other apps.

Update daprd to always start Placement and the Actor runtime if
Placement addresses are configured.

Adds integration test for this scenario.

Adds metadatda integration tests.

Signed-off-by: joshvanl <[email protected]>

* Move actor store registration on state store existing

Signed-off-by: joshvanl <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
This update includes bug fixes:

- [Fix Scheduler memory leak](#fix-scheduler-memory-leak)

Fix Scheduler memory leak

Problem

Scheduler would continually grow in memory during usage.

Impact

Scheduler would consume the entirety of memory on the host machine, or to it's cgroup limit, then OOM crash.

Root cause

A Go routine was doing being closed on connection sends.

Solution

Correctly close the Go routine on completion of the connection send.

Signed-off-by: joshvanl <[email protected]>
Dapr 1.15.3

This update includes bug fixes:

- [Fix Timers Deactivating after timer invocation fails](#fix-timers-deactivating-after-timer-invocation-fails)

Fix Timers Deactivating after timer invocation fails

Problem

An app returning a non-2xx status code from a timer invocation would cause a periodic timer to no longer trigger.

Impact

An Actor app which restarted/crashed, or was otherwise busy, would cause a timer to no longer trigger.
This breaks backwards compatibility where a periodic Actor timer would continue to trigger at the defined period, even if the actor was busy or had an error.

Root cause

The Actor timer handle logic deactivates the timer if _any_ timer invocation failed.
Regardless of whether the timer had further ticks defined in it's period schedule.

Solution

As did before v1.15.0, treat any successful or failed timer invocation as the same, and tick the Actor timer forward allowing for future invocations.

Signed-off-by: joshvanl <[email protected]>
@yaron2
yaron2 merged commit 2810bf4 into master Mar 4, 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.