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.

Sign up for GitHub

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

Jump to bottom

ci: Bump multiprocess memory #22061

Merged
merged 1 commit into from May 26, 2021
Merged

ci: Bump multiprocess memory #22061

merged 1 commit into from May 26, 2021

Conversation

laanwj
Copy link
Member

@laanwj laanwj commented May 25, 2021 •
edited by DrahtBot

Fixes #22059

@laanwj laanwj added the Tests label May 25, 2021
@maflcko
Copy link
Member

maflcko commented May 25, 2021

Might have been introduced by #21869

ryanofsky
ryanofsky approved these changes May 25, 2021
View reviewed changes
Copy link
Contributor

@ryanofsky ryanofsky left a comment

Choose a reason for hiding this comment

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

Code review ACK fa0bfc5. Thanks for the update, and interesting to know about #21869. It looks like relevant build https://cirrus-ci.com/task/4807455453478912 is succeeding too

fanquake
fanquake approved these changes May 26, 2021
View reviewed changes
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK fa0bfc5

@fanquake fanquake merged commit 2e7c346 into bitcoin:master May 26, 2021
@maflcko maflcko deleted the 2105-ciBumpOOM branch May 26, 2021 05:43
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 27, 2021
@fanquake
Merge bitcoin#22061: ci: Bump multiprocess memory
1735ab3 
fa0bfc5 ci: Bump multiprocess memory (MarcoFalke)

Pull request description:

  Fixes bitcoin#22059

ACKs for top commit:
  ryanofsky:
    Code review ACK fa0bfc5. Thanks for the update, and interesting to know about bitcoin#21869. It looks like relevant build https://cirrus-ci.com/task/4807455453478912 is succeeding too
  fanquake:
    ACK fa0bfc5

Tree-SHA512: f6e49aadf33199ffa7960c8da0b81bdc5ffea61f373e1b0367d000cdbd214614374b9f1a8b3ce9b8270e6d13a24a2029ab07bddb48e44c86dcb687d645e5ef34
@ryanofsky ryanofsky mentioned this pull request Jun 2, 2021
Draft
gwillen pushed a commit to ElementsProject/elements that referenced this pull request Jun 1, 2022
@apoelstra
Merge 2e7c346 into merged_master (Bitcoin PR bitcoin/bitcoin#22061)
bd42a3e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fanquake fanquake fanquake approved these changes

@ryanofsky ryanofsky ryanofsky approved these changes

Assignees
No one assigned
Labels
Tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cirrus CI: Linker OOMKilled
4 participants
@laanwj @maflcko @fanquake @ryanofsky