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.

Sign up for GitHub

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

Jump to bottom

[BACKPORT/23.0.x] Multiple backports #5439

Merged
merged 7 commits into from
Nov 13, 2023
Merged

[BACKPORT/23.0.x] Multiple backports #5439

merged 7 commits into from
Nov 13, 2023

Conversation

kostko
Copy link
Member

@kostko kostko commented Nov 13, 2023

ptrus and others added 7 commits November 13, 2023 11:48
@ptrus @kostko
go/oais-node/grpc: remove unneeded datadir check
ffeca6d
@peternose @kostko
go/runtime/host/multi: Add runtime ID to the logger
972711e
@peternose @kostko
go/consensus/roothash: Filter executor commitments by runtime ID
13194b1 
Compute executor committee workers no longer have to verify the signatures
of observed commitments simply to identify them as invalid.
@kostko
go/runtime/host/sandbox: Release lock before calling into runtime
228db89 
Similar to how this is handled in the multi runtime host, we need to
release the lock before calling into the runtime as otherwise this could
lead to a deadlock in certain situations.
@kostko
go/worker/compute: Bound batch execution time
612096c
@kostko
runtime: Add compile-time feature for debug logging
0d9765c
@kostko
runtime: Remove incorrectly implemented abort requests
afcf6c9
@kostko kostko added the c:backport Category: backport to stable branch label Nov 13, 2023
@kostko kostko marked this pull request as ready for review November 13, 2023 10:51
@kostko kostko enabled auto-merge November 13, 2023 11:12
@kostko kostko merged commit 622d0bd into stable/23.0.x Nov 13, 2023
@kostko kostko deleted the kostko/stable/23.0.x/backport-multi-2 branch November 13, 2023 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peternose peternose peternose approved these changes

@peterjgilbert peterjgilbert Awaiting requested review from peterjgilbert peterjgilbert is a code owner

@pro-wh pro-wh Awaiting requested review from pro-wh pro-wh is a code owner

@ptrus ptrus Awaiting requested review from ptrus ptrus is a code owner

Assignees
No one assigned
Labels
c:backport Category: backport to stable branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kostko @peternose @ptrus