Skip to content

fix(server): close worker on shutdown and parse in_memory#178

Open
gnanirahulnutakki wants to merge 1 commit into
devfrom
gnanirahulnutakki/ARD-447-server-shutdown-config
Open

fix(server): close worker on shutdown and parse in_memory#178
gnanirahulnutakki wants to merge 1 commit into
devfrom
gnanirahulnutakki/ARD-447-server-shutdown-config

Conversation

@gnanirahulnutakki

Copy link
Copy Markdown
Member

Summary

  • close the server worker queue during graceful shutdown and join the worker thread
  • stop channel forwarders from holding permanent worker sender clones
  • accept explicit ARDUR_MEMORY=in_memory and add a unit test for the literal

Issues

  • ARD-447
  • ARD-449

Verification

  • cargo check -p ardur-server — passed
  • cargo test -p ardur-server — passed
  • cargo clippy --workspace --all-targets -- -D warnings — passed
  • cargo test -p ardur-e2e-tests && cargo build --workspace --bins — passed

Signed-off-by: Ardur team@ardur.ai

ARD-447: close the worker queue during graceful shutdown and join the worker thread instead of only logging.
ARD-449: accept explicit ARDUR_MEMORY=in_memory and cover the literal with a unit test.

Signed-off-by: Ardur <team@ardur.ai>
@linear-code

linear-code Bot commented Jun 23, 2026

Copy link
Copy Markdown

ARD-447

@gnanirahulnutakki

Copy link
Copy Markdown
Member Author

Local verification and required GitHub checks are green. Merge is currently blocked by base branch policy; auto-merge is unavailable for this repo, and I am not bypassing branch protection with admin privileges from this session.

Verification:

  • cargo check -p ardur-server — passed
  • cargo test -p ardur-server — passed
  • cargo clippy --workspace --all-targets -- -D warnings — passed
  • cargo test -p ardur-e2e-tests && cargo build --workspace --bins — passed
  • GitHub checks: DCO, macos-latest / stable, ubuntu-latest / stable — passed

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.

1 participant