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

Hotfix attempt for Canary : v2.8.0-rc0 upgrade fails on Polygon. #11828

Merged
merged 6 commits into from
Jan 19, 2024

Conversation

DylanTinianov
Copy link
Contributor

Hotfix attempt for Canary : v2.8.0-rc0 upgrade fails on Polygon.

Removed start lock on tracker and added some additional logging.

Copy link
Contributor

I see that you haven't updated any README files. Would it make sense to do so?

common/txmgr/tracker.go Outdated Show resolved Hide resolved
common/txmgr/tracker.go Outdated Show resolved Hide resolved
amit-momin
amit-momin previously approved these changes Jan 19, 2024
common/txmgr/txmgr.go Outdated Show resolved Hide resolved
Co-authored-by: Jordan Krage <[email protected]>
common/txmgr/txmgr.go Outdated Show resolved Hide resolved
@cl-sonarqube-production
Copy link

@snehaagni snehaagni added this pull request to the merge queue Jan 19, 2024
Merged via the queue into develop with commit 6133df8 Jan 19, 2024
82 checks passed
@snehaagni snehaagni deleted the hotfix-BCF-2896 branch January 19, 2024 23:43
DylanTinianov added a commit that referenced this pull request Jan 22, 2024
)

* Remove start lock + add logging

* Add loggs

* Update tracker.go

* Update tracker.go

* Update common/txmgr/txmgr.go

Co-authored-by: Jordan Krage <[email protected]>

* Update logging

---------

Co-authored-by: Jordan Krage <[email protected]>
DylanTinianov added a commit that referenced this pull request Jan 22, 2024
)

* Remove start lock + add logging

* Add loggs

* Update tracker.go

* Update tracker.go

* Update common/txmgr/txmgr.go

Co-authored-by: Jordan Krage <[email protected]>

* Update logging

---------

Co-authored-by: Jordan Krage <[email protected]>
(cherry picked from commit 6133df8)
DylanTinianov added a commit that referenced this pull request Jan 22, 2024
)

* Remove start lock + add logging

* Add loggs

* Update tracker.go

* Update tracker.go

* Update common/txmgr/txmgr.go

Co-authored-by: Jordan Krage <[email protected]>

* Update logging

---------

Co-authored-by: Jordan Krage <[email protected]>
(cherry picked from commit 6133df8)
snehaagni pushed a commit that referenced this pull request Jan 22, 2024
) (#11839)

* Remove start lock + add logging

* Add loggs

* Update tracker.go

* Update tracker.go

* Update common/txmgr/txmgr.go

Co-authored-by: Jordan Krage <[email protected]>

* Update logging

---------

Co-authored-by: Jordan Krage <[email protected]>
(cherry picked from commit 6133df8)
github-merge-queue bot pushed a commit that referenced this pull request Jan 25, 2024
* Revert "make streams lookup modular (#11368)" (#11482)

This reverts commit fa0f16a.

* core/services/relay/evm: start RequestRoundTracker; report full health (#11643)

* core/services/relay/evm: start RequestRoundTracker; report full health

* Tests round requests and implicit changes separately

* Add test to CI

* Fixes other OCR2 checks

---------

Co-authored-by: Adam Hamrick <[email protected]>
(cherry picked from commit 7236361)

* Document metrics renaming

* Hotfix attempt for Canary : v2.8.0-rc0 upgrade fails on Polygon. (#11828) (#11839)

* Remove start lock + add logging

* Add loggs

* Update tracker.go

* Update tracker.go

* Update common/txmgr/txmgr.go

Co-authored-by: Jordan Krage <[email protected]>

* Update logging

---------

Co-authored-by: Jordan Krage <[email protected]>
(cherry picked from commit 6133df8)

* Temporarily disable tracker (#11857)

* Disable tracker

* Remove all tracker calls

(cherry picked from commit 5057899)

* Update CHANGELOG.md

* Update docs/CHANGELOG.md

Co-authored-by: Jordan Krage <[email protected]>

* Finalize date on changelog for 2.8.0 (#11881)

---------

Co-authored-by: Lei <[email protected]>
Co-authored-by: Jordan Krage <[email protected]>
Co-authored-by: Dmytro Haidashenko <[email protected]>
Co-authored-by: Dylan Tinianov <[email protected]>
Co-authored-by: Dylan Tinianov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants