-
Notifications
You must be signed in to change notification settings - Fork 987
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
upgrade nixpkgs to 23-11 #19369
upgrade nixpkgs to 23-11 #19369
Conversation
Jenkins BuildsClick to see older builds (80)
|
d3c9d9c
to
03afddf
Compare
03afddf
to
51adf21
Compare
98% of end-end tests have passed
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (47)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
292b9d9
to
f824608
Compare
e63f5b3
to
6dc63b6
Compare
92f5e7c
to
939e291
Compare
939e291
to
bb17681
Compare
5a6fe05
to
0195e7e
Compare
0e60c29
to
a7c33ba
Compare
Hey @siddarthkay! In this PR we experience issues with connection to store node. We are not reproducing this in develop, so assume this could be PR regression. See description below. In case this error is related to waku team please let us know. ISSUE 1 Communities are not fetched, "store node is not available" in logsSteps:
Actual result: telegram-cloud-document-2-5203941607451148720.mp4log from Status log
log from geth log
|
Thanks for testing @pavloburykh : I'll take a look. |
843780f
to
2714eb6
Compare
Hi @siddarthkay ! Also found extra scenario caused by connection to store node. May be useful to reproducing problem. ISSUE 2: User created by using seed phrase has no restored data
Expected result: user will be recovered with data |
@siddarthkay I've checked new builds but unfortunately all issues are still exist on both platforms: iOS/Android
|
@pavloburykh @mariia-skrypnyk : |
aa41e85
to
cfa1bd3
Compare
85% of end-end tests have passed
Failed tests (6)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (41)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
|
100% of end-end tests have passed
Passed tests (6)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
@siddarthkay I am not reproducing ISSUE 1 anymore. PR is ready for merge.
@siddarthkay by last commit you mean these changes #19369 (comment) ? If yes - then it is weird, because issue actually was reproducible for me and you after this commit when we checked on Friday. Or you committed some other changes which fixed the bug? |
Thanks for testing @pavloburykh ! |
Great, thank you @siddarthkay! |
fixes #18311 This PR upgrades `nixpkgs` to latest release version of 23-11 ref -> https://github.com/NixOS/nixpkgs/releases/tag/23.11 - `Gradle` from `8.0.1` -> `8.4` - `Git` from `2.40.1` -> `2.42.0` - `Curl` from `8.0.1` -> `8.4.0` - `OpenSSL` from `3.0.8` -> `3.0.13` - `NodeJS` from `18.16.0` -> `18.19.1` - `Python` from `3.10.11` -> `3.10.13` - `Clojure` from `1.11.1.1273` -> `1.11.1.1413` - and some more stuff - `xcbeautify` , this was added in 23-11 ->NixOS/nixpkgs#289446 - `idb-companion`, this was also added in 23-11 -> NixOS/nixpkgs#296440
cfa1bd3
to
4bd62eb
Compare
fixes #18311
Summary
This PR upgrades
nixpkgs
to latest release version of 23-11ref -> https://github.com/NixOS/nixpkgs/releases/tag/23.11
Following are the noteable upgrades :
Gradle
from8.0.1
->8.4
Git
from2.40.1
->2.42.0
Curl
from8.0.1
->8.4.0
OpenSSL
from3.0.8
->3.0.13
NodeJS
from18.16.0
->18.19.1
Python
from3.10.11
->3.10.13
Clojure
from1.11.1.1273
->1.11.1.1413
Following derivations were removed :
xcbeautify
, this was added in 23-11 -> [23.11] backport xcbeautify: init at 1.4.0 NixOS/nixpkgs#289446idb-companion
, this was also added in 23-11 -> [23.11] backport idb-companion: init at 1.1.8 NixOS/nixpkgs#296440Testing notes
Test Everything
Platforms
status: ready