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

fix: Added missing values for HistoryError to be sync with nwaku #1952

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

NagyZoltanPeter
Copy link
Contributor

Problem

Find test failure in nwaku ci:

2024-04-09T09:49:52.0513572Z   8) Waku Store, general
2024-04-09T09:49:52.0513871Z        Query generator for 2000 messages:
2024-04-09T09:49:52.0514217Z      Error: Invalid enum value
2024-04-09T09:49:52.0514929Z       at findValue (file:///home/runner/work/nwaku/nwaku/node_modules/protons-runtime/src/codecs/enum.ts:9:13)
2024-04-09T09:49:52.0516150Z       at Object.enumDecode [as decode] (file:///home/runner/work/nwaku/nwaku/node_modules/protons-runtime/src/codecs/enum.ts:24:12)
2024-04-09T09:49:52.0517376Z       at Object.decode (file:///home/runner/work/nwaku/nwaku/packages/proto/src/generated/store.ts:451:64)
2024-04-09T09:49:52.0519087Z       at Object.decode (file:///home/runner/work/nwaku/nwaku/packages/proto/src/generated/store.ts:533:54)
2024-04-09T09:49:52.0521545Z       at decodeMessage (file:///home/runner/work/nwaku/nwaku/node_modules/protons-runtime/src/decode.ts:8:16)
2024-04-09T09:49:52.0522987Z       at Object.HistoryRpc.decode (file:///home/runner/work/nwaku/nwaku/packages/proto/src/generated/store.ts:557:12)
2024-04-09T09:49:52.0524260Z       at HistoryRpc.decode (file:///home/runner/work/nwaku/nwaku/packages/core/src/lib/store/history_rpc.ts:73:34)
2024-04-09T09:49:52.0525300Z       at StoreCore.queryPerPage (file:///home/runner/work/nwaku/nwaku/packages/core/src/lib/store/index.ts:110:37)
2024-04-09T09:49:52.0526135Z       at processTicksAndRejections (node:internal/process/task_queues:95:5)
2024-04-09T09:49:52.0526996Z       at async StoreSDK.queryGenerator (file:///home/runner/work/nwaku/nwaku/packages/sdk/src/protocols/store.ts:83:22)
2024-04-09T09:49:52.0527571Z 

Found missing enum values from HistoryError in protobuf definition.
Please help me elaborate this PR for me ;-)

@NagyZoltanPeter NagyZoltanPeter self-assigned this Apr 9, 2024
@NagyZoltanPeter NagyZoltanPeter requested a review from a team as a code owner April 9, 2024 11:06
Copy link

github-actions bot commented Apr 9, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
Waku node 180.93 KB (+0.05% 🔺) 3.7 s (+0.05% 🔺) 1.7 s (-10.76% 🔽) 5.3 s
Waku Simple Light Node 180.85 KB (+0.07% 🔺) 3.7 s (+0.07% 🔺) 1.4 s (-11.9% 🔽) 5 s
ECIES encryption 23.12 KB (+0.06% 🔺) 463 ms (+0.06% 🔺) 507 ms (+27.26% 🔺) 970 ms
Symmetric encryption 22.57 KB (-0.07% 🔽) 452 ms (-0.07% 🔽) 612 ms (+33.04% 🔺) 1.1 s
DNS discovery 72.44 KB (+0.06% 🔺) 1.5 s (+0.06% 🔺) 731 ms (+14.97% 🔺) 2.2 s
Peer Exchange discovery 73.96 KB (-0.01% 🔽) 1.5 s (-0.01% 🔽) 1.2 s (+77.46% 🔺) 2.7 s
Local Peer Cache Discovery 67.71 KB (-0.01% 🔽) 1.4 s (-0.01% 🔽) 813 ms (-6.68% 🔽) 2.2 s
Privacy preserving protocols 38.78 KB (+0.07% 🔺) 776 ms (+0.07% 🔺) 610 ms (-21.81% 🔽) 1.4 s
Waku Filter 20.22 KB (+0.27% 🔺) 405 ms (+0.27% 🔺) 258 ms (-46.31% 🔽) 663 ms
Waku LightPush 110.13 KB (+0.01% 🔺) 2.3 s (+0.01% 🔺) 1.4 s (+30.51% 🔺) 3.6 s
History retrieval protocols 110.74 KB (+0.11% 🔺) 2.3 s (+0.11% 🔺) 997 ms (-52.61% 🔽) 3.3 s
Deterministic Message Hashing 4.83 KB (0%) 97 ms (0%) 33 ms (-28.62% 🔽) 130 ms

@NagyZoltanPeter NagyZoltanPeter force-pushed the fix/missing-history-error-values branch from 0871684 to aa3ce1f Compare April 9, 2024 13:11
@NagyZoltanPeter
Copy link
Contributor Author

@weboko: Can you help me? Is it "normal" that some of the CI checks fails? I see some peer discovery errors.
This change is needed for my PR on nwaku as some interface has changed on protocol level.

@adklempner
Copy link
Member

@weboko: Can you help me? Is it "normal" that some of the CI checks fails? I see some peer discovery errors. This change is needed for my PR on nwaku as some interface has changed on protocol level.

as long as all the required CI checks pass then it's considered good

@adklempner adklempner merged commit 4607d9e into master Apr 10, 2024
8 of 10 checks passed
@adklempner adklempner deleted the fix/missing-history-error-values branch April 10, 2024 15:55
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.

2 participants