Skip to content

Conversation

@batconjurer
Copy link
Collaborator

Resolves Issue #1726
Allow eth-oracle to be activated / deactivated via config updates sent from ledger. This allows governance to shutdown the ledger w/o restarts and not resulting in the ledger crashing.

This is covered with a test, but it is flaky due to async channel race conditions.

@batconjurer batconjurer requested a review from sug0 July 26, 2023 10:21
@batconjurer batconjurer requested a review from sug0 July 26, 2023 14:52
sug0
sug0 previously approved these changes Jul 26, 2023
Copy link
Collaborator

@sug0 sug0 left a comment

Choose a reason for hiding this comment

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

lgtm. at some point I want to refactor the active key has_key checks as part of #1720 (adding this here just to cross-ref the PR in the issue)

@batconjurer batconjurer requested a review from tzemanovic July 27, 2023 08:27
tzemanovic
tzemanovic previously approved these changes Jul 28, 2023
@tzemanovic
Copy link
Collaborator

Pls add a change log

@batconjurer batconjurer dismissed stale reviews from tzemanovic and sug0 via ecb89e2 August 17, 2023 13:50
@tzemanovic
Copy link
Collaborator

looks like we missed this in the last release

@Fraccaman
Copy link
Collaborator

ah totally forgot about this :( should we plan it for 0.25.0 or later?

@sug0
Copy link
Collaborator

sug0 commented Nov 4, 2023

I think this is not a priority. can add this to a later release

@tzemanovic tzemanovic mentioned this pull request Nov 22, 2023
tzemanovic added a commit that referenced this pull request Nov 27, 2023
* origin/bat/feature/disable-oracle-gov:
  [chore]: Add changelog
  [fix]: Fixed block height typo in test
  [feat]: Allow eth-oracle to be activated / deactivated via config updates sent from ledger
@tzemanovic tzemanovic merged commit b19e9f8 into main Dec 7, 2023
@tzemanovic tzemanovic deleted the bat/feature/disable-oracle-gov branch December 7, 2023 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants