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

Adds Daprd --dapr-block-shutdown-duration reference #3893

Merged
merged 4 commits into from
Dec 8, 2023

Conversation

JoshVanL
Copy link
Contributor

@JoshVanL JoshVanL commented Dec 4, 2023

Part of dapr/dapr#4313

Signed-off-by: joshvanl <me@joshvanl.dev>
@JoshVanL JoshVanL requested review from a team as code owners December 4, 2023 23:40
JoshVanL added a commit to JoshVanL/dapr that referenced this pull request Dec 4, 2023
Closes dapr#4313
Docs: dapr/docs#3893

PR adds the `--block-shutdown-seconds` CLI flag and corresponding
`dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
Daprd to block the graceful shutdown procedure until _either_, the
block shutdown seconds has elapsed _or_ the application has become
unhealthy, as according to the normal app health status.

By default, this option is unset, and therefore there is no effect to
the current behaviour of graceful shutdown. When set, Daprd will
block the interrupt signal cascading into runtime until the above
requirements have been met.

The framework process `Cleanup` order has been reversed to mimic
`t.Cleanup` and allow the `logline` process to function correctly.

Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
@JoshVanL JoshVanL changed the title Adds Daprd --block-shutdown-seconds reference Adds Daprd --dapr-block-shutdown-duration reference Dec 5, 2023
yaron2 pushed a commit to dapr/dapr that referenced this pull request Dec 6, 2023
* Adds Daprd option `--block-shutdown-seconds`

Closes #4313
Docs: dapr/docs#3893

PR adds the `--block-shutdown-seconds` CLI flag and corresponding
`dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
Daprd to block the graceful shutdown procedure until _either_, the
block shutdown seconds has elapsed _or_ the application has become
unhealthy, as according to the normal app health status.

By default, this option is unset, and therefore there is no effect to
the current behaviour of graceful shutdown. When set, Daprd will
block the interrupt signal cascading into runtime until the above
requirements have been met.

The framework process `Cleanup` order has been reversed to mimic
`t.Cleanup` and allow the `logline` process to function correctly.

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert if check on killing process exec proc cleanup

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert error ignore of processes already killed in unix

Signed-off-by: joshvanl <me@joshvanl.dev>

* Skip shutdown/graceful/block/healthy on windows.

* Skip shutdown/block/unhealthy test on windows.

* Linting

Signed-off-by: joshvanl <me@joshvanl.dev>

* Updates `dapr-block-shutdown-seconds` to `dapr-block-shutdown-duration`

Signed-off-by: joshvanl <me@joshvanl.dev>

---------

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Loong Dai <long.dai@intel.com>
Copy link
Collaborator

@hhunter-ms hhunter-ms left a comment

Choose a reason for hiding this comment

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

quick review, otherwise lgtm!

Co-authored-by: Hannah Hunter <94493363+hhunter-ms@users.noreply.github.com>
Signed-off-by: Josh van Leeuwen <me@joshvanl.dev>
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 8, 2023

Thanks @hhunter-ms !

@hhunter-ms hhunter-ms merged commit 477451e into dapr:v1.13 Dec 8, 2023
4 checks passed
DeepanshuA pushed a commit to DeepanshuA/dapr that referenced this pull request Dec 11, 2023
* Adds Daprd option `--block-shutdown-seconds`

Closes dapr#4313
Docs: dapr/docs#3893

PR adds the `--block-shutdown-seconds` CLI flag and corresponding
`dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
Daprd to block the graceful shutdown procedure until _either_, the
block shutdown seconds has elapsed _or_ the application has become
unhealthy, as according to the normal app health status.

By default, this option is unset, and therefore there is no effect to
the current behaviour of graceful shutdown. When set, Daprd will
block the interrupt signal cascading into runtime until the above
requirements have been met.

The framework process `Cleanup` order has been reversed to mimic
`t.Cleanup` and allow the `logline` process to function correctly.

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert if check on killing process exec proc cleanup

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert error ignore of processes already killed in unix

Signed-off-by: joshvanl <me@joshvanl.dev>

* Skip shutdown/graceful/block/healthy on windows.

* Skip shutdown/block/unhealthy test on windows.

* Linting

Signed-off-by: joshvanl <me@joshvanl.dev>

* Updates `dapr-block-shutdown-seconds` to `dapr-block-shutdown-duration`

Signed-off-by: joshvanl <me@joshvanl.dev>

---------

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Loong Dai <long.dai@intel.com>
JoshVanL added a commit to JoshVanL/dapr that referenced this pull request Dec 14, 2023
* Adds Daprd option `--block-shutdown-seconds`

Closes dapr#4313
Docs: dapr/docs#3893

PR adds the `--block-shutdown-seconds` CLI flag and corresponding
`dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
Daprd to block the graceful shutdown procedure until _either_, the
block shutdown seconds has elapsed _or_ the application has become
unhealthy, as according to the normal app health status.

By default, this option is unset, and therefore there is no effect to
the current behaviour of graceful shutdown. When set, Daprd will
block the interrupt signal cascading into runtime until the above
requirements have been met.

The framework process `Cleanup` order has been reversed to mimic
`t.Cleanup` and allow the `logline` process to function correctly.

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert if check on killing process exec proc cleanup

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert error ignore of processes already killed in unix

Signed-off-by: joshvanl <me@joshvanl.dev>

* Skip shutdown/graceful/block/healthy on windows.

* Skip shutdown/block/unhealthy test on windows.

* Linting

Signed-off-by: joshvanl <me@joshvanl.dev>

* Updates `dapr-block-shutdown-seconds` to `dapr-block-shutdown-duration`

Signed-off-by: joshvanl <me@joshvanl.dev>

---------

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Loong Dai <long.dai@intel.com>
cicoyle pushed a commit to cicoyle/dapr that referenced this pull request May 24, 2024
* Adds Daprd option `--block-shutdown-seconds`

Closes dapr#4313
Docs: dapr/docs#3893

PR adds the `--block-shutdown-seconds` CLI flag and corresponding
`dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
Daprd to block the graceful shutdown procedure until _either_, the
block shutdown seconds has elapsed _or_ the application has become
unhealthy, as according to the normal app health status.

By default, this option is unset, and therefore there is no effect to
the current behaviour of graceful shutdown. When set, Daprd will
block the interrupt signal cascading into runtime until the above
requirements have been met.

The framework process `Cleanup` order has been reversed to mimic
`t.Cleanup` and allow the `logline` process to function correctly.

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert if check on killing process exec proc cleanup

Signed-off-by: joshvanl <me@joshvanl.dev>

* Revert error ignore of processes already killed in unix

Signed-off-by: joshvanl <me@joshvanl.dev>

* Skip shutdown/graceful/block/healthy on windows.

* Skip shutdown/block/unhealthy test on windows.

* Linting

Signed-off-by: joshvanl <me@joshvanl.dev>

* Updates `dapr-block-shutdown-seconds` to `dapr-block-shutdown-duration`

Signed-off-by: joshvanl <me@joshvanl.dev>

---------

Signed-off-by: joshvanl <me@joshvanl.dev>
Co-authored-by: Loong Dai <long.dai@intel.com>
Signed-off-by: Cassandra Coyle <cassie@diagrid.io>
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.

None yet

2 participants