8000 Stop all sub processes spawn by acceptance tests by fab-10 · Pull Request #8284 · hyperledger/besu · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Stop all sub processes spawn by acceptance tests #8284

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
Merged
merged 2 commits into from
Feb 13, 2025
Merged

Stop all sub processes spawn by acceptance tests #8284

merged 2 commits into from
Feb 13, 2025

Conversation

fab-10
Copy link
Contributor
@fab-10 fab-10 commented Feb 12, 2025 •
edited
Loading

PR description

Another attempt to release this fix after #8226 has been reverted here #8236, this time reducing the memory used by Gradle

Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests

@fab-10 fab-10 mentioned this pull request Feb 12, 2025
Closed
@fab-10 fab-10 force-pushed the retry-8226-stop-besu-subprocess-in-ats branch from b45328c to 5689b42 Compare February 12, 2025 18:04
@fab-10
Stop all sub processes spawn by acceptance tests
f7c85f6 
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
@fab-10
Reduce memory used by Gradle
1f98ebf 
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
@fab-10 fab-10 force-pushed the retry-8226-stop-besu-subprocess-in-ats branch from 5689b42 to 1f98ebf Compare February 13, 2025 09:59
@fab-10 fab-10 marked this pull request as ready for review February 13, 2025 10:35
@macfarla macfarla added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to Branch Protection failures Feb 13, 2025
You're not authorized to push to this branch. Visit "About protected branches" for more information.
@macfarla macfarla added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to Branch Protection failures Feb 13, 2025
You're not authorized to push to this branch. Visit "About protected branches" for more information.
@macfarla macfarla added this pull request to the merge queue Feb 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to Branch Protection failures Feb 13, 2025
You're not authorized to push to this branch. Visit "About protected branches" for more information.
@macfarla macfarla merged commit 46e74d1 into hyperledger:main Feb 13, 2025
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@macfarla macfarla macfarla approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@fab-10 @macfarla
0