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

Feature: clear pending offchain #117

Merged
merged 3 commits into from
Jan 19, 2022
Merged

Feature: clear pending offchain #117

merged 3 commits into from
Jan 19, 2022

Conversation

aalu1418
Copy link
Collaborator

@aalu1418 aalu1418 commented Jan 18, 2022 •
edited
Loading

  • add additional validation to prevent reaching a broken state (validation on write_offchain_config)
  • add function to clear pending offchain config (original flow of calling commit_offchain_config to clear would potentially impact running feeds, and this provides a way to easily clear pending configs without impacting the important parts of state)
  • add gauntlet command (ocr2:reset_pending_offchain_config)

@aalu1418 aalu1418 marked this pull request as ready for review January 18, 2022 20:38
@aalu1418 aalu1418 requested a review from RodrigoAD January 18, 2022 20:38
@aalu1418 aalu1418 merged commit b56eac8 into develop Jan 19, 2022
@aalu1418 aalu1418 deleted the feature/clear-pending-offchain branch January 19, 2022 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@archseer archseer archseer approved these changes

@krebernisak krebernisak Awaiting requested review from krebernisak

@RodrigoAD RodrigoAD Awaiting requested review from RodrigoAD

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aalu1418 @archseer