Skip to content

SIGKILL (OOM?) for documenting web-sys 0.3.46 #1197

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

Closed
alexcrichton opened this issue Dec 1, 2020 · 4 comments
Closed

SIGKILL (OOM?) for documenting web-sys 0.3.46 #1197

alexcrichton opened this issue Dec 1, 2020 · 4 comments
Labels
C-limits-increase Category: A crate needs a limits increase to build successfully

Comments

@alexcrichton
Copy link
Member

Crate name: web-sys
Build failure link: https://docs.rs/crate/web-sys/0.3.46/builds/321002

Additional details:

Is it possible to figure out why SIGKILL was received on this build? I think that OOM is the obvious suspect since this is a huge crate with all of its documentation enabled (which we do for docs-rs). I wanted to ideally confirm though that this was OOM-killed before trying to do something else.

Otherwise, if the memory limit can't be raised, I suspect the next step is to open a bug on rustdoc itself!

@jyn514
Copy link
Member

jyn514 commented Dec 1, 2020

Is it possible to figure out why SIGKILL was received on this build?

Not without running cargo doc or a local docs.rs instance, unfortunately - I'd love to have it though: #878.

I can definitely raise the memory limit in the meantime, it will probably be a while before rustdoc's memory consumption improves (although if the cause is different here than in rust-lang/rust#79103 I'd love to hear about it 🙂)

@jyn514 jyn514 added the C-limits-increase Category: A crate needs a limits increase to build successfully label Dec 1, 2020
@jyn514
Copy link
Member

jyn514 commented Dec 1, 2020

$  command time -v cargo doc --all-features
	Maximum resident set size (kbytes): 3117768

Yup, this just hit the limit. I'll bump it to 5 GB for now, that should be enough for a while.

@jyn514 jyn514 changed the title SIGKILL (OOM?) for documenting web-sys SIGKILL (OOM?) for documenting web-sys 0.3.46 Dec 1, 2020
@jyn514
Copy link
Member

jyn514 commented Dec 1, 2020

Requeued and rebuilt: https://docs.rs/crate/web-sys/0.3.46

@jyn514 jyn514 closed this as completed Dec 1, 2020
@alexcrichton
Copy link
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-limits-increase Category: A crate needs a limits increase to build successfully
Projects
None yet
Development

No branches or pull requests

2 participants