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

nix: point to nixpkgs unstable channel for gradle 8.8 #21267

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

siddarthkay
Copy link
Contributor

@siddarthkay siddarthkay commented Sep 14, 2024

Summary

This bump in nixpkgs is needed for react-native upgrade to v0.75
A summary of upgraded packages is below :

  • openjdk from 11 -> 17
  • gradle from 8.4 -> 8.8
  • git from 2.42.0 -> 2.44.1
  • NodeJS from 18.19.1 -> 20.12.2
  • Python from 3.10.13 -> 3.10.14
  • Clojure from 1.11.1.1413 -> 1.11.3.1463
  • Cocoapods from 1.13.0 -> 1.15.2

Platforms

  • Android
  • iOS

status: ready

@siddarthkay siddarthkay self-assigned this Sep 14, 2024
@status-im-auto
Copy link
Member

status-im-auto commented Sep 14, 2024

Jenkins Builds

Click to see older builds (45)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 5f24c85 #1 2024-09-14 12:47:42 ~21 min ios 📱ipa 📲
✔️ 0c6ec6d #2 2024-09-14 13:02:40 ~10 min tests 📄log
✔️ 0c6ec6d #2 2024-09-14 13:03:24 ~10 min ios 📱ipa 📲
✔️ 0c6ec6d #2 2024-09-14 13:06:06 ~13 min android-e2e 🤖apk 📲
✔️ 0c6ec6d #2 2024-09-14 13:07:03 ~14 min android 🤖apk 📲
✔️ 0241893 #3 2024-09-14 14:17:56 ~4 min tests 📄log
✔️ 0241893 #3 2024-09-14 14:20:33 ~7 min android-e2e 🤖apk 📲
✔️ 0241893 #3 2024-09-14 14:21:32 ~8 min android 🤖apk 📲
✔️ 0241893 #3 2024-09-14 14:24:21 ~10 min ios 📱ipa 📲
✔️ 58b55b4 #4 2024-09-14 14:52:00 ~5 min tests 📄log
✔️ 58b55b4 #4 2024-09-14 14:55:11 ~8 min android-e2e 🤖apk 📲
✔️ 58b55b4 #4 2024-09-14 14:55:37 ~8 min android 🤖apk 📲
✔️ 58b55b4 #4 2024-09-14 15:02:22 ~15 min ios 📱ipa 📲
✔️ 17180a0 #5 2024-09-14 15:40:35 ~5 min tests 📄log
✔️ 17180a0 #5 2024-09-14 15:43:23 ~8 min android-e2e 🤖apk 📲
✔️ 17180a0 #5 2024-09-14 15:44:18 ~9 min android 🤖apk 📲
✔️ 17180a0 #5 2024-09-14 15:51:33 ~16 min ios 📱ipa 📲
✔️ 9ed6679 #6 2024-09-18 05:11:57 ~4 min tests 📄log
✔️ 9ed6679 #6 2024-09-18 05:15:27 ~7 min android-e2e 🤖apk 📲
✔️ 9ed6679 #6 2024-09-18 05:16:45 ~9 min android 🤖apk 📲
✔️ 9ed6679 #6 2024-09-18 05:27:44 ~20 min ios 📱ipa 📲
✔️ f89e167 #7 2024-09-19 05:50:13 ~5 min tests 📄log
✔️ f89e167 #7 2024-09-19 05:53:01 ~7 min android-e2e 🤖apk 📲
✔️ f89e167 #7 2024-09-19 05:53:04 ~7 min android 🤖apk 📲
✔️ f89e167 #7 2024-09-19 06:00:46 ~15 min ios 📱ipa 📲
✔️ f880a86 #8 2024-09-19 06:53:39 ~4 min tests 📄log
✔️ f880a86 #8 2024-09-19 06:56:34 ~7 min android-e2e 🤖apk 📲
✔️ f880a86 #8 2024-09-19 06:58:18 ~8 min android 🤖apk 📲
✔️ 6dc9412 #9 2024-09-19 07:04:48 ~4 min tests 📄log
✔️ 6dc9412 #9 2024-09-19 07:06:39 ~6 min android-e2e 🤖apk 📲
✔️ 6dc9412 #9 2024-09-19 07:08:04 ~7 min android 🤖apk 📲
✔️ 6dc9412 #9 2024-09-19 07:13:23 ~13 min ios 📱ipa 📲
✔️ 22f6278 #12 2024-09-19 12:23:18 ~5 min tests 📄log
22f6278 #12 2024-09-19 12:25:32 ~7 min ios 📄log
✔️ 22f6278 #12 2024-09-19 12:26:13 ~8 min android 🤖apk 📲
✔️ 22f6278 #12 2024-09-19 12:47:22 ~29 min android-e2e 🤖apk 📲
22f6278 #14 2024-09-19 13:02:43 ~8 min ios 📄log
✔️ 7c9abd2 #13 2024-09-19 13:37:30 ~5 min tests 📄log
✔️ 7c9abd2 #13 2024-09-19 13:39:45 ~8 min android 🤖apk 📲
✔️ 7c9abd2 #13 2024-09-19 13:40:17 ~8 min android-e2e 🤖apk 📲
7c9abd2 #15 2024-09-19 13:40:42 ~9 min ios 📄log
✔️ e64d894 #14 2024-09-19 13:50:38 ~4 min tests 📄log
✔️ e64d894 #14 2024-09-19 13:52:38 ~6 min android 🤖apk 📲
✔️ e64d894 #14 2024-09-19 13:53:24 ~7 min android-e2e 🤖apk 📲
e64d894 #16 2024-09-19 13:58:28 ~12 min ios 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 3c5b671 #15 2024-09-19 15:30:47 ~4 min tests 📄log
✔️ 3c5b671 #15 2024-09-19 15:32:37 ~6 min android-e2e 🤖apk 📲
✔️ 3c5b671 #15 2024-09-19 15:34:15 ~7 min android 🤖apk 📲
✔️ b54979b #16 2024-09-19 15:49:33 ~4 min tests 📄log
✔️ b54979b #16 2024-09-19 15:51:56 ~6 min android-e2e 🤖apk 📲
✔️ b54979b #16 2024-09-19 15:52:46 ~7 min android 🤖apk 📲
✔️ b54979b #18 2024-09-19 15:57:40 ~12 min ios 📱ipa 📲

@siddarthkay siddarthkay force-pushed the use-nixpkgs-unstable-gradle-8-8 branch 2 times, most recently from 58b55b4 to 17180a0 Compare September 14, 2024 15:34
@siddarthkay
Copy link
Contributor Author

Hey hey @status-im/mobile @status-im/wallet-mobile-devs : Please check if the changes in this PR cause any issues with your local dev environments.
Thank you!

@seanstrom
Copy link
Member

I cleaned and built the ios app - everything seems okay so far ✅
I cleaned and built the andaroid app - everything seems good too ✅

for context I'm on macOS 13.6.7
and I'm using Xcode 15.2
and I'm using Android Studio 2023.2.1

Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, looks good.

nix/status-go/mobile/build.nix Outdated Show resolved Hide resolved
@jakubgs
Copy link
Member

jakubgs commented Sep 17, 2024

Also, if all the commits here are necessary to make the upgrade work I would squash them into one commit with all the context in the description. You can combine the titles of existing commits as a list for the commit description.

Copy link
Contributor

@ilmotta ilmotta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did a smoke test for NixOS, running tests, REPL usage, hot-reload, and checked the PR build, all good.

@siddarthkay
Copy link
Contributor Author

@status-im/mobile-qa : This PR also needs just a smoke test, we upgrade all our build tools here when we move to newer nixpkgs version.

@siddarthkay siddarthkay force-pushed the use-nixpkgs-unstable-gradle-8-8 branch 2 times, most recently from f880a86 to 6dc9412 Compare September 19, 2024 07:00
@siddarthkay siddarthkay force-pushed the use-nixpkgs-unstable-gradle-8-8 branch 2 times, most recently from c6fe507 to 22f6278 Compare September 19, 2024 12:17
@mariia-skrypnyk
Copy link

@siddarthkay on my way to test it!

@mariia-skrypnyk mariia-skrypnyk self-assigned this Sep 19, 2024
@status-im-auto
Copy link
Member

0% of end-end tests have passed

Total executed tests: 7
Failed tests: 7
Expected to fail tests: 0
Passed tests: 0
IDs of failed tests: 727231,727230,702745,702742,703133,702843,727229 

Failed tests (7)

Click to expand
  • Rerun failed tests

  • Class TestCommunityMultipleDeviceMerged:

    1. test_community_message_edit, id: 702843
    Test setup failed: critical/chats/test_public_chat_browsing.py:309: in prepare_devices
        self.device_1, self.device_2 = SignInView(self.drivers[0]), SignInView(self.drivers[1])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    Device sessions

    Class TestWalletOneDevice:

    1. test_wallet_add_remove_regular_account, id: 727231

    Test setup failed: critical/test_wallet.py:201: in prepare_devices
        self.sign_in_view = SignInView(self.drivers[0])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    Device sessions

    Class TestOneToOneChatMultipleSharedDevicesNewUi:

    1. test_1_1_chat_non_latin_messages_stack_update_profile_photo, id: 702745

    Test setup failed: critical/chats/test_1_1_public_chats.py:21: in prepare_devices
        self.device_1, self.device_2 = SignInView(self.drivers[0]), SignInView(self.drivers[1])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    Device sessions

    Class TestCommunityOneDeviceMerged:

    1. test_community_copy_and_paste_message_in_chat_input, id: 702742

    Test setup failed: critical/chats/test_public_chat_browsing.py:23: in prepare_devices
        self.sign_in = SignInView(self.drivers[0])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    Device sessions

    2. test_restore_multiaccount_with_waku_backup_remove_switch, id: 703133

    Test setup failed: critical/chats/test_public_chat_browsing.py:23: in prepare_devices
        self.sign_in = SignInView(self.drivers[0])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    Class TestWalletMultipleDevice:

    1. test_wallet_send_asset_from_drawer, id: 727230

    Test setup failed: critical/test_wallet.py:24: in prepare_devices
        self.sign_in_1, self.sign_in_2 = SignInView(self.drivers[0]), SignInView(self.drivers[1])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    2. test_wallet_send_eth, id: 727229

    Test setup failed: critical/test_wallet.py:24: in prepare_devices
        self.sign_in_1, self.sign_in_2 = SignInView(self.drivers[0]), SignInView(self.drivers[1])
    ../views/sign_in_view.py:194: in __init__
        self.enter_seed_phrase_next_button = Button(self.driver, translation_id="enter-seed-phrase")
    ../views/base_element.py:427: in __init__
        super(Button, self).__init__(driver, **kwargs)
    ../views/base_element.py:36: in __init__
        self.set_locator()
    ../views/base_element.py:45: in set_locator
        text = transl[self.translation_id]
     'enter-seed-phrase'
    



    nix: ldflags are expected to be a list
    
    chore: fix deprecated vendorSha256
    
    nix: bump jdk to 17 & pass java version to gradle
    
    make: set universalApk false in run-android target
    
    nix: fix --replace deprecation warning
    
      - NixOS/nixpkgs#260776
    
    ios: update pod and gem lockfiles
    
    nix: bump nodejs to 20.12.2
    
    nix: add Xcode 16.0 to allowed versions
    
    chore: upgrade rn-image-crop-picker lib
    
    nix: full path for missing dep in node_modules sh
    @status-im-auto
    Copy link
    Member

    100% of end-end tests have passed

    Total executed tests: 7
    Failed tests: 0
    Expected to fail tests: 0
    Passed tests: 7
    

    Passed tests (7)

    Click to expand

    Class TestOneToOneChatMultipleSharedDevicesNewUi:

    1. test_1_1_chat_non_latin_messages_stack_update_profile_photo, id: 702745
    Device sessions

    Class TestWalletMultipleDevice:

    1. test_wallet_send_asset_from_drawer, id: 727230
    2. test_wallet_send_eth, id: 727229

    Class TestWalletOneDevice:

    1. test_wallet_add_remove_regular_account, id: 727231
    Device sessions

    Class TestCommunityMultipleDeviceMerged:

    1. test_community_message_edit, id: 702843
    Device sessions

    Class TestCommunityOneDeviceMerged:

    1. test_community_copy_and_paste_message_in_chat_input, id: 702742
    Device sessions

    2. test_restore_multiaccount_with_waku_backup_remove_switch, id: 703133
    Device sessions

    @status-im-auto
    Copy link
    Member

    100% of end-end tests have passed

    Total executed tests: 51
    Failed tests: 0
    Expected to fail tests: 0
    Passed tests: 51
    

    Passed tests (51)

    Click to expand

    Class TestWalletMultipleDevice:

    1. test_wallet_send_asset_from_drawer, id: 727230
    2. test_wallet_send_eth, id: 727229

    Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:

    1. test_1_1_chat_delete_via_long_press_relogin, id: 702784
    Device sessions

    2. test_1_1_chat_is_shown_message_sent_delivered_from_offline, id: 702783
    Device sessions

    3. test_1_1_chat_mute_chat, id: 703496
    Device sessions

    Class TestWalletOneDevice:

    1. test_wallet_add_remove_regular_account, id: 727231
    Device sessions

    Class TestDeepLinksOneDevice:

    1. test_links_open_universal_links_from_chat, id: 704613
    Device sessions

    2. test_links_deep_links_profile, id: 702775
    Device sessions

    3. test_deep_links_communities, id: 739307
    Device sessions

    Class TestActivityMultipleDevicePRTwo:

    1. test_activity_center_mentions, id: 702957
    Device sessions

    2. test_activity_center_admin_notification_accept_swipe, id: 702958
    Device sessions

    Class TestActivityMultipleDevicePR:

    1. test_activity_center_reply_read_unread_delete_filter_swipe, id: 702947
    Device sessions

    Class TestCommunityMultipleDeviceMerged:

    1. test_community_several_images_send_reply, id: 703194
    Device sessions

    2. test_community_one_image_send_reply, id: 702859
    Device sessions

    3. test_community_emoji_send_copy_paste_reply, id: 702840
    Device sessions

    4. test_community_mark_all_messages_as_read, id: 703086
    Device sessions

    5. test_community_contact_block_unblock_offline, id: 702894
    Device sessions

    6. test_community_edit_delete_message_when_offline, id: 704615
    Device sessions

    7. test_community_message_delete, id: 702839
    Device sessions

    8. test_community_message_send_check_timestamps_sender_username, id: 702838
    Device sessions

    9. test_community_links_with_previews_github_youtube_twitter_gif_send_enable, id: 702844
    Device sessions

    10. test_community_message_edit, id: 702843
    Device sessions

    11. test_community_unread_messages_badge, id: 702841
    Device sessions

    Class TestActivityCenterContactRequestMultipleDevicePR:

    1. test_add_contact_field_validation, id: 702777
    Device sessions

    2. test_activity_center_contact_request_accept_swipe_mark_all_as_read, id: 702851
    Device sessions

    3. test_activity_center_contact_request_decline, id: 702850
    Device sessions

    Class TestCommunityMultipleDeviceMergedTwo:

    1. test_community_markdown_support, id: 702809
    Device sessions

    2. test_community_hashtag_links_to_community_channels, id: 702948
    Device sessions

    3. test_community_mentions_push_notification, id: 702786
    Device sessions

    4. test_community_leave, id: 702845
    Device sessions

    5. test_community_join_when_node_owner_offline, id: 703629
    Device sessions

    Class TestCommunityOneDeviceMerged:

    1. test_restore_multiaccount_with_waku_backup_remove_switch, id: 703133
    Device sessions

    2. test_community_copy_and_paste_message_in_chat_input, id: 702742
    Device sessions

    3. test_community_discovery, id: 703503
    Device sessions

    4. test_community_undo_delete_message, id: 702869
    Device sessions

    5. test_community_navigate_to_channel_when_relaunch, id: 702846
    Device sessions

    6. test_community_mute_community_and_channel, id: 703382
    Device sessions

    Class TestOneToOneChatMultipleSharedDevicesNewUi:

    1. test_1_1_chat_emoji_send_reply_and_open_link, id: 702782
    Device sessions

    2. test_1_1_chat_text_message_delete_push_disappear, id: 702733
    Device sessions

    3. test_1_1_chat_push_emoji, id: 702813
    Device sessions

    4. test_1_1_chat_non_latin_messages_stack_update_profile_photo, id: 702745
    Device sessions

    5. test_1_1_chat_edit_message, id: 702855
    Device sessions

    6. test_1_1_chat_send_image_save_and_share, id: 703391
    Device sessions

    7. test_1_1_chat_pin_messages, id: 702731
    Device sessions

    8. test_1_1_chat_message_reaction, id: 702730
    Device sessions

    Class TestGroupChatMultipleDeviceMergedNewUI:

    1. test_group_chat_pin_messages, id: 702732
    Device sessions

    2. test_group_chat_mute_chat, id: 703495
    Device sessions

    3. test_group_chat_send_image_save_and_share, id: 703297
    Device sessions

    4. test_group_chat_reactions, id: 703202
    Device sessions

    5. test_group_chat_join_send_text_messages_push, id: 702807
    Device sessions

    6. test_group_chat_offline_pn, id: 702808
    Device sessions

    @mariia-skrypnyk
    Copy link

    Hey @siddarthkay !

    Did smoke, all good.
    PR can be merged.

    @siddarthkay siddarthkay merged commit 1ee16da into develop Sep 20, 2024
    6 checks passed
    @siddarthkay siddarthkay deleted the use-nixpkgs-unstable-gradle-8-8 branch September 20, 2024 09:31
    @yakimant
    Copy link
    Member

    @siddarthkay, missed that.

    Could we have gradle from unstable and everything else from stable?

    @siddarthkay
    Copy link
    Contributor Author

    @yakimant : The easiest way to do that in existing setup could be to update our fork of nixpkgs from 24.05 release branch and only update gradle 8.8 in that repo and use that commit here.

    @jakubgs
    Copy link
    Member

    jakubgs commented Oct 1, 2024

    @siddarthkay not true. You can have however many nixpkgs version as you want. It's just a variable.

    @siddarthkay
    Copy link
    Contributor Author

    ah then what you suggest is indeed easier @jakubgs

    @siddarthkay
    Copy link
    Contributor Author

    An attempt is here -> #21374

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    Archived in project
    Development

    Successfully merging this pull request may close these issues.

    9 participants