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

feat: Add eth merge pause flag (to main) #376

Merged
merged 11 commits into from
Oct 1, 2022
Merged

Conversation

facundomedica
Copy link

@facundomedica facundomedica commented Oct 1, 2022

Add --eth-merge-pause="true" to pause some messages from being sent during the pause period of the bridge.

Things paused:

SendToCosmos
SendToEth
DeployedERC20
Batch requests
Confirms are all enabled. If we see any batch confirms, it means we'll need to add an extra upgrade step.

mergify bot and others added 8 commits September 26, 2022 16:34
* chore: modify eth block delay

* chore: modify eth block delay

* fix test

* fix test

* changelog

* changelog

(cherry picked from commit 86ac6c4)

Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>
* update test scripts

* reduce noise messages of pf

* add more providers to default

* add workflow for v1.0.x

* fix branch name

(cherry picked from commit 13fb428)

Co-authored-by: Rafael Tenfen <rafaeltenfen.rt@gmail.com>
Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>
* chore: prepare v1.1.0 release

* chore: prepare v1.1.0 release
@mergify
Copy link

mergify bot commented Oct 1, 2022

⚠️ The sha of the head commit of this PR conflicts with #375. Mergify cannot evaluate rules on this PR. ⚠️

@mergify
Copy link

mergify bot commented Oct 1, 2022

⚠️ The sha of the head commit of this PR conflicts with #375. Mergify cannot evaluate rules on this PR. ⚠️

@codecov-commenter
Copy link

Codecov Report

Merging #376 (3e9e39c) into main (86ac6c4) will decrease coverage by 0.12%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #376      +/-   ##
==========================================
- Coverage   57.81%   57.68%   -0.13%     
==========================================
  Files          17       17              
  Lines        2041     2049       +8     
==========================================
+ Hits         1180     1182       +2     
- Misses        758      763       +5     
- Partials      103      104       +1     
Impacted Files Coverage Δ
orchestrator/main_loops.go 7.44% <0.00%> (-0.02%) ⬇️
orchestrator/eth_event_watcher.go 76.71% <16.66%> (-1.70%) ⬇️
orchestrator/orchestrator.go 88.46% <100.00%> (+0.46%) ⬆️

@facundomedica facundomedica changed the title Merge back changes from v1 into main feat: Add eth merge pause flag (to main) Oct 1, 2022
@facundomedica facundomedica merged commit 8b19cf8 into main Oct 1, 2022
@facundomedica facundomedica deleted the facu/bp-merge-pause branch October 1, 2022 18:12
mergify bot pushed a commit that referenced this pull request Oct 1, 2022
* chore: modify eth block delay (#367) (#368)

* chore: modify eth block delay

* chore: modify eth block delay

* fix test

* fix test

* changelog

* changelog

(cherry picked from commit 86ac6c4)

Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>

* feat: reduce oracle noise msgs (#365) (#369)

* update test scripts

* reduce noise messages of pf

* add more providers to default

* add workflow for v1.0.x

* fix branch name

(cherry picked from commit 13fb428)

Co-authored-by: Rafael Tenfen <rafaeltenfen.rt@gmail.com>
Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>

* chore: prepare v1.1.0 release (#371)

* chore: prepare v1.1.0 release

* chore: prepare v1.1.0 release

* feat: Add eth merge pause flag

* fix

* fix add flag to cmd

* lint

* fix mergify

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Rafael Tenfen <rafaeltenfen.rt@gmail.com>
(cherry picked from commit 8b19cf8)

# Conflicts:
#	CHANGELOG.md
facundomedica added a commit that referenced this pull request Oct 1, 2022
* feat: Add eth merge pause flag (to main) (#376)

* chore: modify eth block delay (#367) (#368)

* chore: modify eth block delay

* chore: modify eth block delay

* fix test

* fix test

* changelog

* changelog

(cherry picked from commit 86ac6c4)

Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>

* feat: reduce oracle noise msgs (#365) (#369)

* update test scripts

* reduce noise messages of pf

* add more providers to default

* add workflow for v1.0.x

* fix branch name

(cherry picked from commit 13fb428)

Co-authored-by: Rafael Tenfen <rafaeltenfen.rt@gmail.com>
Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>

* chore: prepare v1.1.0 release (#371)

* chore: prepare v1.1.0 release

* chore: prepare v1.1.0 release

* feat: Add eth merge pause flag

* fix

* fix add flag to cmd

* lint

* fix mergify

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Rafael Tenfen <rafaeltenfen.rt@gmail.com>
(cherry picked from commit 8b19cf8)

# Conflicts:
#	CHANGELOG.md

* fix cl

Co-authored-by: Facundo Medica <14063057+facundomedica@users.noreply.github.com>
Co-authored-by: Facundo Medica <facundomedica@gmail.com>
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.

2 participants