search

status-im / status-go

The Status module that consumes go-ethereum
https://status.im
Mozilla Public License 2.0
728 stars 247 forks source link

fix_: update defaultGorushURL #6011

Closed igor-sirotin closed 1 month ago

igor-sirotin commented 1 month ago

The old gorush instance was decomissioned today:

It was used with old eth.prod fleet which was also decomissioned a few weeks back: https://github.com/status-im/infra-eth-cluster/commit/bb56c3cc

The correct new address is https://gorush.infra.status.im/

This fixes the failing MessengerPushNotificationSuite tests.

Ideally, we should run a gorush server locally. It's fairly simple to do with docker. But it's probably not worth it.

status-im-auto commented 1 month ago

Jenkins Builds

:grey_question: Commit :hash: Finished (UTC) Duration Platform Result
:heavy_check_mark: e2a73185 #1 2024-10-29 16:20:52 ~3 min macos :package:zip
:heavy_check_mark: e2a73185 #1 2024-10-29 16:21:47 ~4 min tests-rpc :page_facing_up:log
:heavy_check_mark: e2a73185 #1 2024-10-29 16:22:28 ~5 min ios :package:zip
:heavy_check_mark: e2a73185 #1 2024-10-29 16:22:29 ~5 min macos :package:zip
:heavy_check_mark: e2a73185 #1 2024-10-29 16:22:49 ~5 min linux :package:zip
:heavy_check_mark: e2a73185 #1 2024-10-29 16:23:04 ~6 min android :package:aar
:heavy_multiplication_x: e2a73185 #1 2024-10-29 16:27:32 ~10 min windows :package:zip
:heavy_check_mark: e2a73185 #1 2024-10-29 16:51:35 ~34 min tests :page_facing_up:log
codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 60.59%. Comparing base (eea527a) to head (e2a7318). Report is 2 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #6011 +/- ## ============================================ + Coverage 13.07% 60.59% +47.52% ============================================ Files 797 812 +15 Lines 107755 109323 +1568 ============================================ + Hits 14087 66244 +52157 + Misses 91867 35298 -56569 - Partials 1801 7781 +5980 ``` | [Flag](https://app.codecov.io/gh/status-im/status-go/pull/6011/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im) | Coverage Δ | | |---|---|---| | [functional](https://app.codecov.io/gh/status-im/status-go/pull/6011/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im) | `13.02% <ø> (-0.05%)` | :arrow_down: | | [unit](https://app.codecov.io/gh/status-im/status-go/pull/6011/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im) | `59.95% <ø> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files with missing lines](https://app.codecov.io/gh/status-im/status-go/pull/6011?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im) | Coverage Δ | | |---|---|---| | [protocol/pushnotificationserver/server.go](https://app.codecov.io/gh/status-im/status-go/pull/6011?src=pr&el=tree&filepath=protocol%2Fpushnotificationserver%2Fserver.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im#diff-cHJvdG9jb2wvcHVzaG5vdGlmaWNhdGlvbnNlcnZlci9zZXJ2ZXIuZ28=) | `72.28% <ø> (+72.28%)` | :arrow_up: | ... and [645 files with indirect coverage changes](https://app.codecov.io/gh/status-im/status-go/pull/6011/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=status-im)