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

MERC-6304 ea telem support for view function #14426

Closed
wants to merge 40 commits into from

Conversation

akuzni2
Copy link
Contributor

@akuzni2 akuzni2 commented Sep 13, 2024

MERC-6304

  • Adds a method to search for descendent tasks given a task
  • Adds support for "tags" on tasks that can be handled by consuming users of Task interface
  • For Mercury Specifically - View function output is supported by adding a generic approach to tag fields for telemetry extraction

Requires

N/A

Resolves

N/A

@akuzni2 akuzni2 requested review from a team as code owners September 13, 2024 19:54
@akuzni2 akuzni2 requested review from cedric-cordenier and removed request for a team September 13, 2024 19:54
Tofel and others added 11 commits September 13, 2024 13:58
* pass *types.Transaction instead of common.Hash to chain's Confirm() function

* fix lints
* Add doc

* Update docs

* Fixes

* Nit fixes

* Update core/chains/evm/gas/docs/FEE_HISTORY_ESTIMATOR.md

Co-authored-by: amit-momin <[email protected]>

* Fixes

---------

Co-authored-by: amit-momin <[email protected]>
* DEVSVCS-499: adding VRF v2 test configs for CTF tests

* DEVSVCS-499: improving formatting

* Update vrfv2 workflows

* DEVSVCS-499: removing unnecessary workflows; adding on demand workflow for VRF v2 smoke tests

---------

Co-authored-by: lukaszcl <[email protected]>
* Change TelemetryIngress.UniConn default to false

* Add changeset
* Remove PriceMin check from attempt builder

* Remove TipCapMin

* Update changeset
* DEVSVCS-134: adding readme on how to run VRF CTF tests

* DEVSVCS-134: minor updates
* Port

* Use mcms lib

* Proposal signing working

* Almost working. Just need ramps to make timelock owner first

* Accept ownership proposal working

* Fee quoter accept ownership

* Fix existing tests

* Enable to new chain traffic working

* Mod tidy

* Fix build

* lint

* More lint

* Use Address() for MCM/Timelock

* Rename

* Self review cleanup

* Clean up job management and bootstrap handling

* Comments
…4230)

* Update the code owner of `feeds` service from the old FMS group to the new OPCORE one.

* Add missing key engineers.
* added unimplemented contract reader

* fix write targets test

* update common
@akuzni2 akuzni2 requested review from a team as code owners September 13, 2024 19:59
@akuzni2 akuzni2 force-pushed the feature/MERC-6304-ea-telem-view-function branch from b257d3e to 7db21f0 Compare September 13, 2024 23:00
@akuzni2 akuzni2 changed the title Feature/merc 6304 ea telem view function MERC-6304 ea telem support for view function Sep 16, 2024
samsondav
samsondav previously approved these changes Sep 16, 2024
@akuzni2
Copy link
Contributor Author

akuzni2 commented Sep 16, 2024

/refresh

@akuzni2 akuzni2 added the auto-update Keep pull request updated with develop branch label Sep 16, 2024
@akuzni2 akuzni2 requested review from ad0ll and Fletch153 September 16, 2024 22:41
@akuzni2 akuzni2 requested a review from samsondav September 17, 2024 13:33
@akuzni2 akuzni2 removed the auto-update Keep pull request updated with develop branch label Sep 17, 2024
samsondav
samsondav previously approved these changes Sep 17, 2024
samsondav
samsondav previously approved these changes Sep 17, 2024
Fletch153
Fletch153 previously approved these changes Sep 17, 2024
@akuzni2 akuzni2 dismissed stale reviews from Fletch153 and samsondav via b46af32 September 17, 2024 19:08
@akuzni2 akuzni2 closed this Sep 17, 2024
@akuzni2 akuzni2 deleted the feature/MERC-6304-ea-telem-view-function branch November 8, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-publish Build and Publish image to SDLC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants