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

Rococo can't sync from scratch: node triggers memory leak #1148

Closed
bLd75 opened this issue Aug 25, 2023 · 8 comments
Closed

Rococo can't sync from scratch: node triggers memory leak #1148

bLd75 opened this issue Aug 25, 2023 · 8 comments
Assignees

Comments

@bLd75
Copy link

bLd75 commented Aug 25, 2023

When syncing Rococo from scratch, at certain chain height the node starts to use resources abnormally until it gets OOM.

I've been testing on several servers (with 32Gb RAM), result is always the same:

  • from block ~1929500, messages like this one start to appear (may not be relevant)
    Allocating slots failed for lease period 134, with: Module(ModuleError { index: 31, error: [2, 0, 0, 0], message: Some("CannotUpgrade") })
  • until block ~3018000, no problem node sync normally
  • from block ~3018000, memory consumption starts increasing and disk IO spiking, other processes are becoming irresponsive but node still syncs
  • 2 hours 30mn later (around block ~3638800), node get OOM and restarts
  • from this point, the node will get OOM every 3 minutes and restart

image

@ggwpez
Copy link
Member

ggwpez commented Aug 25, 2023

Last time I tried syncing Kusama it also needed 64 GB or something like this. This is definitely not good.

@bLd75
Copy link
Author

bLd75 commented Aug 25, 2023

I've set a snapshot of today for whom may need it
https://partage.bldnodes.org/share/rococo-snapshot-20230825.tar.gz

@bkchr
Copy link
Member

bkchr commented Aug 26, 2023

@koute can you please take a look?

@koute koute self-assigned this Aug 26, 2023
@koute
Copy link
Contributor

koute commented Aug 26, 2023

I'll look into this.

@koute
Copy link
Contributor

koute commented Aug 28, 2023

@bLd75 Just for reference, can you paste full exact command-line invocation you're using and the exact commit on which you see this problem?

@bLd75
Copy link
Author

bLd75 commented Aug 28, 2023

Hi @koute I made some new tests and I just realized polkadot-v1.0.0 client manages to sync Rococo correctly.
I was using embedded parachain client on version polkadot-v0.9.43, this is the problematic version.
I'm closing the issue as latest version fixes it, but this would be to communicate with Rococo parachains (most of them don't use latest Polkadot client version) cc @ggwpez .

@bLd75 bLd75 closed this as completed Aug 28, 2023
@ggwpez
Copy link
Member

ggwpez commented Aug 29, 2023

I will test it on a GCP to confirm that its fixed.

@ggwpez
Copy link
Member

ggwpez commented Sep 1, 2023

Yep syncing with 32GB works again in v1.0.0.

serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
bkchr pushed a commit that referenced this issue Apr 10, 2024
* Alter gitlab.

* Use substrate's rustfmt.toml

* cargo +nightly fmt --all

* Fix spellcheck.

* cargo +nightly fmt --all

* format.

* Fix spellcheck and fmt

* fmt?

* Fix spellcheck

Co-authored-by: Tomasz Drwięga <tomasz@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: done
Development

No branches or pull requests

4 participants