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

backport/geode 10257/upgrade java #7710

Conversation

demery-pivotal
Copy link
Contributor

onichols-pivotal and others added 20 commits May 9, 2022 00:17
Remove likely-to-become-outdated copies of release scripts to ensure
they are not run by accident from a branch (they should always be run
from develop).

Also remove CODEOWNERS to avoid the confusion of GitHub showing owner
names like on develop, but codeowner reviews not actually being
required (due to lack of branch protection or minimum review count on
support branches)
As part of the Geode Release Process, the build number must
be rolled forward so work can begin on the next release
…and (apache#7666) (apache#7671)

* Update documentation

Co-authored-by: Dave Barnes <daveba@vmware.com>
(cherry picked from commit d4b80d2)
…udeBuild projects (apache#7600)

* Extract DependencyConstraints plugin from buildSrc
* Extract japicmp plugin customization from buildSrc
* Test isolation from buildSrc
* Disable jacoco on all test facets by default

(cherry picked from commit 509c0c6)
springfox swagger is no longer maintained, so springdoc is the best choice going forward.

(cherry picked from commit 9fbd35a)
…pache#7687)

* GEODE-10307: Doc changes needed for enable security-manager property
* GEODE-10289: Argument file for JDK 17

The argument file was generated on Linux using OpenJDK 17.0.2

* Add arg file to assembly_content.txt
* GEODE-10288: Define JDK 8, 11, 17 homes for upgrade tests (apache#7675)

* GEODE-10288: Fix property assignment syntax (apache#7678)
* GEODE-10302: Increase upgrade test timeout to 4h (apache#7683)

* GEODE-10302: Increase call stack timeout for upgrade tests (apache#7691)

To be 3h45m, 15 minutes shy of the newly increased task timeout.
…7679)

 * During putIfAbsent retry, comparing invalid token value when
   putIfAbsent of a null value instead.

(cherry picked from commit 4f4af2a)
Currently, upgrade tests upgrade from an old version of Geode to the
current version, both running on the test JVM's version of Java.

This commit enhances most upgrade tests so that they also upgrade from
an old Java version to a newer one, both running the current version of
Geode.

The new `VmConfiguration` class represents a configuration for a Geode
JVM, specifying both the Java version and the Geode version.

The new `VmConfigurations` class offers two factory methods to produce
lists of candidate configurations:
- `VmConfigurations.upgrades()` produces a list of "upgrade"
  configurations useful for most upgrade tests. Each upgrade
  configuration specifies either and old version of Geode or an old
  version of Java, but not both.
- `VmConfigurations.all()` produces a list of upgrades plus a
  configuration representing the current version of Geode and the test
  JVM's version of Java.

`VmConfigurations` also includes factory methods to create predicates to
filter configurations.

(cherry picked from commit e835c8c)
@demery-pivotal demery-pivotal added jdk17 add this label to get Linux JDK17 PR checks too jdk8 add this label to get Linux JDK8 PR checks too labels May 19, 2022
@demery-pivotal demery-pivotal requested a review from Bill as a code owner May 19, 2022 17:44
@demery-pivotal demery-pivotal added the windows add this label to get Windows JDK11 PR checks too label May 19, 2022
@demery-pivotal demery-pivotal deleted the backport/geode-10257/upgrade-java branch May 19, 2022 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jdk8 add this label to get Linux JDK8 PR checks too jdk17 add this label to get Linux JDK17 PR checks too windows add this label to get Windows JDK11 PR checks too
Projects
None yet