-
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
ci: upgrade go to 1.22 #21940
base: develop
Are you sure you want to change the base?
ci: upgrade go to 1.22 #21940
Conversation
Jenkins BuildsClick to see older builds (12)
|
88% of end-end tests have passed
Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMerged:
Passed tests (7)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestWalletOneDevice:
Class TestWalletMultipleDevice:
Class TestCommunityOneDeviceMerged:
|
91% of end-end tests have passed
Failed tests (3)Click to expandClass TestFallbackMultipleDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Passed tests (51)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletMultipleDevice:
Class TestWalletOneDevice:
Class TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestFallbackMultipleDevice:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
244e83c
to
f6477ef
Compare
@siddarthkay what can be affected area / possible regression? |
@churik: from past experience, the previous go upgrade had impacted store node communication. I've updated the PR description from template on possible impacted areas. |
f6477ef
to
7d38570
Compare
83% of end-end tests have passed
Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Passed tests (5)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestFallbackMultipleDevice:
|
7d38570
to
feeac1a
Compare
@siddarthkay Hi! Thanks for your PR! However, it was observed that switching from Wi-Fi to a mobile network caused message delays of 1-2 minutes in 1-1 chats, group chats, and communities. Additionally, it was noticed a few times that the icon for certain contacts disappeared (showing a cross), but it later reappeared. Logs reciever: Please let me know if any additional information is needed. |
@Horupa-Olena with a cross icons I have seen it in nightly too; as for delay when switching from wi-Fi to cellular, it shoudl be checked, may be @chaitanyaprem have an idea might this PR affect his fix? |
Nothing in this PR should affect this. But i am wondering if could be due to store node issue. @Horupa-Olena can i get |
@chaitanyaprem I couldn't obtain this log because the file was too large and didn't attach when reporting the issue via email. I tried throughout the day to reproduce this issue (to attempt getting the geth.log), but messages started arriving quickly without delays. |
if it is not reproducible, then it could be due to temporary glitch with store nodes or some kind of a corner scenario which doesn't get hit always. |
@Horupa-Olena in this case we can safely move PR to "Merge", thank you! |
feeac1a
to
0f21342
Compare
Summary
go
to1.22
gomobile
latest derivation at master ofnixpkgs
: NixOS/nixpkgs@f5abef9go
is also upgraded to1.22
-> ci_: upgrade go to 1.22 status-go#6258Testing notes
Please test thoroughly.
Areas that may be impacted
Platforms
status: ready
related: status-im/status-go#6254