Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

feat: not lock the keys store when running orchestrator/relayer #394

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Jun 14, 2023

Overview

Closes #359

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id added the store label Jun 14, 2023
@rach-id rach-id requested a review from evan-forbes as a code owner June 14, 2023 12:26
@rach-id rach-id self-assigned this Jun 14, 2023
Comment on lines +69 to +72
if err == fslock.ErrLocked {
return nil, ErrOpened
}
return nil, err
Copy link
Member

Choose a reason for hiding this comment

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

I know this was just copied over, but do we need this extra logic just to change the error type?

Copy link
Member Author

Choose a reason for hiding this comment

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

I didn't wanna change anything since i took it from the node, and it works as expected

@rach-id rach-id merged commit 03e24cc into main Jun 19, 2023
@rach-id rach-id deleted the not_lock_keys branch June 19, 2023 11:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not lock the P2P/EVM store when running the orchestrator/relayer
2 participants