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

Compute DO alarm offset properly #107

Closed
cjpatton opened this issue Aug 31, 2022 · 0 comments · Fixed by #108
Closed

Compute DO alarm offset properly #107

cjpatton opened this issue Aug 31, 2022 · 0 comments · Fixed by #108
Assignees

Comments

@cjpatton
Copy link
Contributor

Once this change lands, we'll need to update our code accordingly: cloudflare/workers-rs#216

Specifically, here we need to use an offset instead of the current time plus the offset: https://github.com/cloudflare/daphne/blob/main/daphne_worker/src/durable/helper_state_store.rs#L66

@cjpatton cjpatton changed the title Coimpute DO alarm offset properly Compute DO alarm offset properly Aug 31, 2022
@cjpatton cjpatton self-assigned this Aug 31, 2022
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 a pull request may close this issue.

1 participant