[Relay] E2E Relay Gaps #177

bryanchriswhite · 2023-11-10T14:23:52Z

Summary

Spike to close the E2E relay & claim/proof loop.

Human Summary

Issue

[☂️ DRAFT] Umbrella Ticket for automated E2E Relay #126

Type of change

Select one or more:

Testing

Run all unit tests: make go_develop_and_test
Verify Localnet manually: See the instructions [here](TODO: add link to instructions)

Sanity Checklist

I have tested my changes using the available tooling
I have performed a self-review of my own code
I have commented my code, updated documentation and left TODOs throughout the codebase

* pokt/main: [Off-chain] feat: observable utils (#171) [Off-chain] refactor: `MapFn`s receive context arg (#170)

RelayerOption parameter

- Fixed helpers for localnet regenesis - Added an application & supplier to the genesis file - Initializing appMap & supplierMap in E2E tests - Add support for the app's codec (for unmarshaling responses) in E2E tests - Adding a placeholder for `e2e/tests/relay.feature` --- Co-authored-by: harry <[email protected]>

* refactor: `MapFn`s receive context arg * feat: add `MapExpand` observable operator * refactor: `RelayerSessionsManager` to be more reactive * chore: add godoc comment * chore: review feedback improvements * trigger CI

Co-authored-by: Daniel Olshansky <[email protected]>

* pokt/main: [Relayer] refactor: simplify `RelayerSessionsManager` (#169) [Test] First step for automated E2E Relay test (#167)

* pokt/main: [Proxy] chore: Use depinject for relayerProxy (#173) [Sessions] chore: Use depinject for sessions mgr construction (#175)

…ger`.

pkg/client/block/client.go

pkg/relayer/cmd/cmd.go

h5law

Have addressed the comments I left and did a little shared code cleanup, LGTM

pkg/client/tx/client.go

h5law

LGTM Lets get this merged ASAP

red-0ne

Love the shape it's taking! LGTM

…relayer_cli * pokt/main: [LocalNet] Run Relayer and AppGateServer (#179) [Relay] E2E Relay Gaps (#177)

…-update * merge/e2e_test/relay_x_relayer_cli: [LocalNet] Run Relayer and AppGateServer (#179) [Relay] E2E Relay Gaps (#177) More tiny comment updates Added a couple more comments Update some comments and TODOs Update the names and references to queryNode/sequencerNode/fullNode etc Update pkg/relayer/cmd/cmd.go [Test] Updating `relay.feature` to run curl command to enable E2E Relay Test (#178) Updated comments for post 177+179 work for okdas Update OpenAPI spec Update .gitignore chore: update comment chore: move shared dependency setup logic to shared pkg chore: cleanup flags and dependencies for appgateserver cmd [Supplier] chore: improve supplier not found error message (#183) [CI] Integrate E2E tests with GitHub CI (#152)

bryanchriswhite and others added 30 commits November 9, 2023 14:41

refactor: MapFns receive context arg

29b9bdf

chore: add ForEach map shorthand operator

b6c9c71

chore: add /pkg/observable/filter

f257b46

chore: add /pkg/observable/logging

194cee5

chore: add /pkg/relayer/protocol

cca55c2

chore: add Miner interface

c666542

feat: add Miner implementation

ea8d848

test: Miner implementation

38329bb

chore: fix comment

d2f9cb4

chore: add godoc comments

d502b75

feat: Add Relayer struct

48cef80

Merge remote-tracking branch 'pokt/main' into issues/13/feat/miner

f85a381

* pokt/main: [Off-chain] feat: observable utils (#171) [Off-chain] refactor: `MapFn`s receive context arg (#170)

chore: Rename to RelayMiner

cba145e

chore: Rename relay miner file

c81c274

chore: Remove unused

0e38e7d

RelayerOption parameter

[Relayer] refactor: simplify RelayerSessionsManager (#169)

0e72490

* refactor: `MapFn`s receive context arg * feat: add `MapExpand` observable operator * refactor: `RelayerSessionsManager` to be more reactive * chore: add godoc comment * chore: review feedback improvements * trigger CI

chore: review feedback improvements

107f6dd

Co-authored-by: Daniel Olshansky <[email protected]>

chore: review feedback improvements

ccad087

Merge remote-tracking branch 'pokt/main' into issues/13/feat/miner

8737024

* pokt/main: [Relayer] refactor: simplify `RelayerSessionsManager` (#169) [Test] First step for automated E2E Relay test (#167)

chore: update start mining comment

e041472

fix: Update Miner interface

8cf0784

fix: import cycle & goimports

85a49b7

chore: review feedback improvements

0788e1d

chore: cleanup TODO_THIS_COMMIT comments

e6a558b

Merge branch 'main' into issues/13/feat/miner

d04ea2a

chore: improve var & func names for clarity and consistency

9c75b2c

Merge remote-tracking branch 'pokt/main' into issues/13/feat/miner

286e1b2

* pokt/main: [Proxy] chore: Use depinject for relayerProxy (#173) [Sessions] chore: Use depinject for sessions mgr construction (#175)

refactor: move claim/proof lifecycle concerns to `relayerSessionsMana…

a0ffe9d

…ger`.

chore: review feedback improvements

2019637

bryanchriswhite added 3 commits November 15, 2023 13:20

chore: add long command description

8d7e577

fix: supplier client test

f945513

Merge branch 'main' into relayer/cli

0a53626

bryanchriswhite requested review from Olshansk and h5law November 15, 2023 12:44

h5law reviewed Nov 15, 2023

View reviewed changes

pkg/client/block/client.go Show resolved Hide resolved

h5law reviewed Nov 15, 2023

View reviewed changes

pkg/relayer/cmd/cmd.go Outdated Show resolved Hide resolved

h5law added 3 commits November 15, 2023 15:40

chore: cleanup flags and dependencies for appgateserver cmd

4ce831b

chore: move shared dependency setup logic to shared pkg

d83c1e7

chore: update comment

e58c848

h5law approved these changes Nov 15, 2023

View reviewed changes

Olshansk added 2 commits November 15, 2023 12:05

Update .gitignore

75343bb

Update OpenAPI spec

1cc7085

Olshansk reviewed Nov 15, 2023

View reviewed changes

pkg/client/tx/client.go Show resolved Hide resolved

Olshansk added 7 commits November 15, 2023 12:45

Updated comments for post 177+179 work for okdas

a04ebfc

Update pkg/relayer/cmd/cmd.go

59f863c

Update the names and references to queryNode/sequencerNode/fullNode etc

753c27a

Update some comments and TODOs

966b874

Added a couple more comments

c726dfc

Merge with main

b9c0d62

More tiny comment updates

c36db92

Olshansk approved these changes Nov 15, 2023

View reviewed changes

h5law approved these changes Nov 15, 2023

View reviewed changes

red-0ne approved these changes Nov 16, 2023

View reviewed changes

Olshansk merged commit a07fd64 into main Nov 16, 2023
8 checks passed

bryanchriswhite added a commit that referenced this pull request Nov 16, 2023

Merge remote-tracking branch 'pokt/main' into merge/e2e_test/relay_x_…

09de036

…relayer_cli * pokt/main: [LocalNet] Run Relayer and AppGateServer (#179) [Relay] E2E Relay Gaps (#177)

bryanchriswhite mentioned this pull request Nov 16, 2023

[Relayminer] chore: cleanup after 177 #190

Merged

10 tasks

bryanchriswhite deleted the relayer/cli branch November 20, 2023 07:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Relay] E2E Relay Gaps #177

[Relay] E2E Relay Gaps #177

bryanchriswhite commented Nov 10, 2023 •

edited

Loading

h5law left a comment

h5law left a comment

red-0ne left a comment

[Relay] E2E Relay Gaps #177

[Relay] E2E Relay Gaps #177

Conversation

bryanchriswhite commented Nov 10, 2023 • edited Loading

Summary

Human Summary

Issue

Type of change

Testing

Sanity Checklist

h5law left a comment

Choose a reason for hiding this comment

h5law left a comment

Choose a reason for hiding this comment

red-0ne left a comment

Choose a reason for hiding this comment

bryanchriswhite commented Nov 10, 2023 •

edited

Loading