Skip to content

Conversation

@Ivansete-status
Copy link
Contributor

Simple PR to validate how tests behave when using nwaku

@Ivansete-status Ivansete-status requested a review from a team as a code owner November 6, 2025 16:42
@status-im-auto
Copy link
Member

status-im-auto commented Nov 6, 2025

Jenkins Builds

Click to see older builds (6)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 64d3cd6 #1 2025-11-06 16:46:13 ~3 min macos/status-go 📦zip
✔️ 64d3cd6 #1 2025-11-06 16:47:28 ~4 min windows/status-go 📦zip
✔️ 64d3cd6 #1 2025-11-06 16:47:40 ~5 min linux/nwaku 📦zip
✔️ 64d3cd6 #1 2025-11-06 16:52:58 ~10 min linux/nwaku 📦zip
✔️ 64d3cd6 #1 2025-11-06 16:54:56 ~12 min tests-rpc 📄log
✔️ 64d3cd6 #1 2025-11-06 17:10:24 ~27 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ 0de1e49 #2 2025-11-06 20:55:45 ~2 min tests 📄log
✔️ 0de1e49 #2 2025-11-06 20:58:33 ~5 min linux/nwaku 📦zip
0de1e49 #2 2025-11-06 20:58:35 ~5 min macos/status-go 📄log
0de1e49 #2 2025-11-06 21:02:41 ~9 min windows/status-go 📄log
✔️ 0de1e49 #2 2025-11-06 21:04:13 ~11 min linux/nwaku 📦zip
✔️ 0de1e49 #2 2025-11-06 21:09:27 ~16 min tests-rpc 📄log
✔️ 769f554 #3 2025-11-07 11:18:06 ~3 min macos/status-go 📦zip
✔️ 769f554 #3 2025-11-07 11:19:08 ~4 min windows/status-go 📦zip
✔️ 769f554 #3 2025-11-07 11:32:48 ~18 min linux/nwaku 📦zip
✔️ 769f554 #3 2025-11-07 11:33:36 ~19 min linux/nwaku 📦zip
✖️ 769f554 #3 2025-11-07 11:50:22 ~35 min tests-rpc 📄log
✔️ 769f554 #3 2025-11-07 11:52:45 ~38 min tests 📄log

@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.92%. Comparing base (3f4a368) to head (769f554).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7074      +/-   ##
===========================================
- Coverage    59.84%   54.92%   -4.93%     
===========================================
  Files          803      810       +7     
  Lines       114133   115458    +1325     
===========================================
- Hits         68305    63410    -4895     
- Misses       38911    45344    +6433     
+ Partials      6917     6704     -213     
Flag Coverage Δ
functional 1.30% <ø> (-33.65%) ⬇️
unit 55.40% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 213 files with indirect coverage changes

@Ivansete-status Ivansete-status marked this pull request as draft November 6, 2025 20:30
@igor-sirotin
Copy link
Collaborator

All functional tests passed, which is weird in the assumption of light client not working 🤔

Looking at the logs,

2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1430	received new envelope	{"messageType": "relay", "envelopeHash": "0xf169ca7e3be57f195533f1f55019cb52cbc03794da9c63aba71c5ff920d8a162", "pubsubTopic": "/waku/2/rs/16/32", "contentTopic": "/waku/1/0x95369798/rfc26", "timestamp": "1762463212600599074"}
2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1476	cached w envelope	{"envelopeHash": "0xf169ca7e3be57f195533f1f55019cb52cbc03794da9c63aba71c5ff920d8a162"}
2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1486	posting event	{"envelopeHash": "0xf169ca7e3be57f195533f1f55019cb52cbc03794da9c63aba71c5ff920d8a162"}
2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1430	received new envelope	{"messageType": "relay", "envelopeHash": "0x90ccb79d864582f0261380c9585f71e4c84961a57c4a74d71dbb719ca44ccfbb", "pubsubTopic": "/waku/2/rs/16/32", "contentTopic": "/waku/1/0xc31a2ea0/rfc26", "timestamp": "1762463213429999128"}
2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1476	cached w envelope	{"envelopeHash": "0x90ccb79d864582f0261380c9585f71e4c84961a57c4a74d71dbb719ca44ccfbb"}
2025-11-06T21:06:55.356Z	DEBUG	GethStatusBackend.StatusNode.protocol	waku/gowaku.go:1486	posting event	{"envelopeHash": "0x90ccb79d864582f0261380c9585f71e4c84961a57c4a74d71dbb719ca44ccfbb"}

Seems that it's still gowaku working here. Not sure why though.

@igor-sirotin
Copy link
Collaborator

Well... the docker build failed

[2025-11-06T20:56:00.988Z] #15 117.1 �[92mBuilding:�[39m Nim compiler
[2025-11-06T20:58:57.509Z] #15 292.8 �[92mBuilding:�[39m install/usr/lib/libbacktrace.a
[2025-11-06T20:59:01.802Z] #15 297.0 �[92mBuilding:�[39m install/usr/lib/libbacktracenim.a
[2025-11-06T20:59:01.802Z] #15 297.0 �[92mBuilding:�[39m librln_v0.9.0.a
[2025-11-06T20:59:01.802Z] #15 297.1 ./scripts/build_rln.sh: line 18: rustc: command not found
[2025-11-06T20:59:01.802Z] #15 297.3 Failed to download -rln.tar.gz
[2025-11-06T20:59:01.802Z] #15 297.3 ./scripts/build_rln.sh: line 37: cargo: command not found
[2025-11-06T20:59:01.802Z] #15 297.3 make[1]: *** [Makefile:189: librln_v0.9.0.a] Error 127
[2025-11-06T20:59:01.802Z] #15 297.3 make[1]: Leaving directory '/go/src/github.com/status-im/nwaku'
[2025-11-06T20:59:01.802Z] #15 297.3 make: *** [Makefile:196: /go/src/github.com/status-im/status-go//../nwaku/build/libwaku.so] Error 2
[2025-11-06T20:59:01.802Z] #15 ERROR: process "/bin/sh -c make $build_target BUILD_TAGS=\"$build_tags\" BUILD_FLAGS=\"$build_flags\"       GO_GENERATE_FAST_RECACHE=$([ \"$enable_go_cache\" = \"true\" ] && echo false || echo true)       GO_GENERATE_FAST_DIR=\"/root/.cache/go-generate-fast\"" did not complete successfully: exit code: 2
[2025-11-06T20:59:01.802Z] ------
[2025-11-06T20:59:01.803Z]  > [builder 9/9] RUN --mount=type=cache,target="/root/.cache/go-build",id=statusgo-build-local     --mount=type=cache,target="/root/.cache/go-generate-fast",id=statusgo-build-local     make cmd BUILD_TAGS="'gowaku_no_rln'" BUILD_FLAGS="-cover"       GO_GENERATE_FAST_RECACHE=$([ "false" = "true" ] && echo false || echo true)       GO_GENERATE_FAST_DIR="/root/.cache/go-generate-fast":
[2025-11-06T20:59:01.803Z] 117.1 �[92mBuilding:�[39m Nim compiler
[2025-11-06T20:59:01.803Z] 292.8 �[92mBuilding:�[39m install/usr/lib/libbacktrace.a
[2025-11-06T20:59:01.803Z] 297.0 �[92mBuilding:�[39m install/usr/lib/libbacktracenim.a
[2025-11-06T20:59:01.803Z] 297.0 �[92mBuilding:�[39m librln_v0.9.0.a
[2025-11-06T20:59:01.803Z] 297.1 ./scripts/build_rln.sh: line 18: rustc: command not found
[2025-11-06T20:59:01.803Z] 297.3 Failed to download -rln.tar.gz
[2025-11-06T20:59:01.803Z] 297.3 ./scripts/build_rln.sh: line 37: cargo: command not found
[2025-11-06T20:59:01.803Z] 297.3 make[1]: *** [Makefile:189: librln_v0.9.0.a] Error 127
[2025-11-06T20:59:01.803Z] 297.3 make[1]: Leaving directory '/go/src/github.com/status-im/nwaku'
[2025-11-06T20:59:01.803Z] 297.3 make: *** [Makefile:196: /go/src/github.com/status-im/status-go//../nwaku/build/libwaku.so] Error 2
[2025-11-06T20:59:01.803Z] ------
[2025-11-06T20:59:01.803Z] Dockerfile:30
[2025-11-06T20:59:01.803Z] --------------------
[2025-11-06T20:59:01.803Z]   29 |         fi
[2025-11-06T20:59:01.803Z]   30 | >>> RUN --mount=type=cache,target="/root/.cache/go-build",id=statusgo-build-$cache_id \
[2025-11-06T20:59:01.803Z]   31 | >>>     --mount=type=cache,target="/root/.cache/go-generate-fast",id=statusgo-build-$cache_id \
[2025-11-06T20:59:01.803Z]   32 | >>>     make $build_target BUILD_TAGS="$build_tags" BUILD_FLAGS="$build_flags" \
[2025-11-06T20:59:01.803Z]   33 | >>>       GO_GENERATE_FAST_RECACHE=$([ "$enable_go_cache" = "true" ] && echo false || echo true) \
[2025-11-06T20:59:01.803Z]   34 | >>>       GO_GENERATE_FAST_DIR="/root/.cache/go-generate-fast"
[2025-11-06T20:59:01.803Z]   35 |     
[2025-11-06T20:59:01.803Z] --------------------
[2025-11-06T20:59:01.803Z] ERROR: failed to build: failed to solve: process "/bin/sh -c make $build_target BUILD_TAGS=\"$build_tags\" BUILD_FLAGS=\"$build_flags\"       GO_GENERATE_FAST_RECACHE=$([ \"$enable_go_cache\" = \"true\" ] && echo false || echo true)       GO_GENERATE_FAST_DIR=\"/root/.cache/go-generate-fast\"" did not complete successfully: exit code: 2

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Thank you for opening this pull request!

We require pull request titles and commits to follow the Conventional Commits specification and it looks like your PR needs to be adjusted.

Details:

According to the conventional-commits specification, some of the commit messages are not valid.

@igor-sirotin
Copy link
Collaborator

igor-sirotin commented Nov 7, 2025

@Ivansete-status I've pushed a (temporal) change of Dockerfile to this branch. Let's see how it works.
But we need to have tests to run on each PR really.

@igor-sirotin
Copy link
Collaborator

In the first version, docker build failed (because of rust missing). But this was ignored by run_functional_tests.py script. And then an older image was used to actually run the tests.

Fixed it here:

@igor-sirotin
Copy link
Collaborator

Looks quite good, closing

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.

3 participants