Skip to content

8214003: Limit default test jobs based on memory size#1307

Closed
gdams wants to merge 1 commit intoopenjdk:pr/1302from
gdams:mem_size
Closed

8214003: Limit default test jobs based on memory size#1307
gdams wants to merge 1 commit intoopenjdk:pr/1302from
gdams:mem_size

Conversation

@gdams
Copy link
Member

@gdams gdams commented Aug 3, 2022

This patch was partially backported in abcacae. This PR adds the remaining patch to make/RunTests.gmk.

This backport is a prerequisite to backporting openjdk/jdk@753c58b


Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue

Integration blocker

 ⚠️ Dependency #1302 must be integrated first

Issue

  • JDK-8214003: Limit default test jobs based on memory size

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk11u-dev pull/1307/head:pull/1307
$ git checkout pull/1307

Update a local copy of the PR:
$ git checkout pull/1307
$ git pull https://git.openjdk.org/jdk11u-dev pull/1307/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 1307

View PR using the GUI difftool:
$ git pr show -t 1307

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk11u-dev/pull/1307.diff

@bridgekeeper
Copy link

bridgekeeper bot commented Aug 3, 2022

👋 Welcome back gdams! A progress list of the required criteria for merging this PR into pr/1302 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk openjdk bot changed the title Backport 1804d6ce65816a529489e3058d9a035d8f772f61 8214003: Limit default test jobs based on memory size Aug 3, 2022
@openjdk
Copy link

openjdk bot commented Aug 3, 2022

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels Aug 3, 2022
@mlbridge
Copy link

mlbridge bot commented Aug 3, 2022

Webrevs

@RealCLanger
Copy link
Contributor

Hm, this one also rectifies things after the JDK-8212028 backport. Again, JDK-8214003 was already backported. Can you meld these changes into #1302 (and adapt JDK-8291794 accordingly)

@gdams gdams closed this Aug 3, 2022
@gdams gdams deleted the mem_size branch August 3, 2022 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

2 participants