LUCENE-10463: increment java version to 17 in smoke tester (#748)

This commit is contained in:
Tomoko Uchida 2022-03-15 19:54:54 +09:00 committed by GitHub
parent 25c4310bd5
commit b6c1024f55
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 30 additions and 30 deletions

View File

@ -143,10 +143,10 @@ def checkJARMetaData(desc, jarFile, gitRevision, version):
'Implementation-Vendor: The Apache Software Foundation', 'Implementation-Vendor: The Apache Software Foundation',
'Specification-Title: Lucene Search Engine:', 'Specification-Title: Lucene Search Engine:',
'Implementation-Title: org.apache.lucene', 'Implementation-Title: org.apache.lucene',
'X-Compile-Source-JDK: 11', 'X-Compile-Source-JDK: 17',
'X-Compile-Target-JDK: 11', 'X-Compile-Target-JDK: 17',
'Specification-Version: %s' % version, 'Specification-Version: %s' % version,
'X-Build-JDK: 11.', 'X-Build-JDK: 17.',
'Extension-Name: org.apache.lucene'): 'Extension-Name: org.apache.lucene'):
if type(verify) is not tuple: if type(verify) is not tuple:
verify = (verify,) verify = (verify,)
@ -610,20 +610,20 @@ def verifyUnpacked(java, artifact, unpackPath, gitRevision, version, testArgs):
validateCmd = './gradlew --no-daemon check -p lucene/documentation' validateCmd = './gradlew --no-daemon check -p lucene/documentation'
print(' run "%s"' % validateCmd) print(' run "%s"' % validateCmd)
java.run_java11(validateCmd, '%s/validate.log' % unpackPath) java.run_java17(validateCmd, '%s/validate.log' % unpackPath)
print(" run tests w/ Java 11 and testArgs='%s'..." % testArgs) print(" run tests w/ Java 17 and testArgs='%s'..." % testArgs)
java.run_java11('./gradlew --no-daemon test %s' % testArgs, '%s/test.log' % unpackPath) java.run_java17('./gradlew --no-daemon test %s' % testArgs, '%s/test.log' % unpackPath)
print(" compile jars w/ Java 11") print(" compile jars w/ Java 17")
java.run_java11('./gradlew --no-daemon jar -Dversion.release=%s' % version, '%s/compile.log' % unpackPath) java.run_java17('./gradlew --no-daemon jar -Dversion.release=%s' % version, '%s/compile.log' % unpackPath)
testDemo(java.run_java11, isSrc, version, '11') testDemo(java.run_java17, isSrc, version, '17')
if java.run_java17: if java.run_java18:
print(" run tests w/ Java 17 and testArgs='%s'..." % testArgs) print(" run tests w/ Java 18 and testArgs='%s'..." % testArgs)
java.run_java17('./gradlew --no-daemon test %s' % testArgs, '%s/test.log' % unpackPath) java.run_java18('./gradlew --no-daemon test %s' % testArgs, '%s/test.log' % unpackPath)
print(" compile jars w/ Java 17") print(" compile jars w/ Java 18")
java.run_java17('./gradlew --no-daemon jar -Dversion.release=%s' % version, '%s/compile.log' % unpackPath) java.run_java18('./gradlew --no-daemon jar -Dversion.release=%s' % version, '%s/compile.log' % unpackPath)
testDemo(java.run_java17, isSrc, version, '17') testDemo(java.run_java18, isSrc, version, '18')
print(' confirm all releases have coverage in TestBackwardsCompatibility') print(' confirm all releases have coverage in TestBackwardsCompatibility')
confirmAllReleasesAreTestedForBackCompat(version, unpackPath) confirmAllReleasesAreTestedForBackCompat(version, unpackPath)
@ -632,9 +632,9 @@ def verifyUnpacked(java, artifact, unpackPath, gitRevision, version, testArgs):
checkAllJARs(os.getcwd(), gitRevision, version) checkAllJARs(os.getcwd(), gitRevision, version)
testDemo(java.run_java11, isSrc, version, '11') testDemo(java.run_java17, isSrc, version, '17')
if java.run_java17: if java.run_java18:
testDemo(java.run_java17, isSrc, version, '17') testDemo(java.run_java18, isSrc, version, '18')
testChangesText('.', version) testChangesText('.', version)
@ -910,7 +910,7 @@ def crawl(downloadedFiles, urlString, targetDir, exclusions=set()):
sys.stdout.write('.') sys.stdout.write('.')
def make_java_config(parser, java17_home): def make_java_config(parser, java18_home):
def _make_runner(java_home, version): def _make_runner(java_home, version):
print('Java %s JAVA_HOME=%s' % (version, java_home)) print('Java %s JAVA_HOME=%s' % (version, java_home))
if cygwin: if cygwin:
@ -924,16 +924,16 @@ def make_java_config(parser, java17_home):
def run_java(cmd, logfile): def run_java(cmd, logfile):
run('%s; %s' % (cmd_prefix, cmd), logfile) run('%s; %s' % (cmd_prefix, cmd), logfile)
return run_java return run_java
java11_home = os.environ.get('JAVA_HOME') java17_home = os.environ.get('JAVA_HOME')
if java11_home is None: if java17_home is None:
parser.error('JAVA_HOME must be set') parser.error('JAVA_HOME must be set')
run_java11 = _make_runner(java11_home, '11') run_java17 = _make_runner(java17_home, '17')
run_java17 = None run_java18 = None
if java17_home is not None: if java18_home is not None:
run_java17 = _make_runner(java17_home, '17') run_java18 = _make_runner(java18_home, '18')
jc = namedtuple('JavaConfig', 'run_java11 java11_home run_java17 java17_home') jc = namedtuple('JavaConfig', 'run_java17 java17_home run_java18 java18_home')
return jc(run_java11, java11_home, run_java17, java17_home) return jc(run_java17, java17_home, run_java18, java18_home)
version_re = re.compile(r'(\d+\.\d+\.\d+(-ALPHA|-BETA)?)') version_re = re.compile(r'(\d+\.\d+\.\d+(-ALPHA|-BETA)?)')
revision_re = re.compile(r'rev-([a-f\d]+)') revision_re = re.compile(r'rev-([a-f\d]+)')
@ -955,8 +955,8 @@ def parse_config():
help='GIT revision number that release was built with, defaults to that in URL') help='GIT revision number that release was built with, defaults to that in URL')
parser.add_argument('--version', metavar='X.Y.Z(-ALPHA|-BETA)?', parser.add_argument('--version', metavar='X.Y.Z(-ALPHA|-BETA)?',
help='Version of the release, defaults to that in URL') help='Version of the release, defaults to that in URL')
parser.add_argument('--test-java17', metavar='java17_home', parser.add_argument('--test-java18', metavar='java18_home',
help='Path to Java17 home directory, to run tests with if specified') help='Path to Java home directory, to run tests with if specified')
parser.add_argument('--download-only', action='store_true', default=False, parser.add_argument('--download-only', action='store_true', default=False,
help='Only perform download and sha hash check steps') help='Only perform download and sha hash check steps')
parser.add_argument('url', help='Url pointing to release to test') parser.add_argument('url', help='Url pointing to release to test')
@ -983,7 +983,7 @@ def parse_config():
if c.local_keys is not None and not os.path.exists(c.local_keys): if c.local_keys is not None and not os.path.exists(c.local_keys):
parser.error('Local KEYS file "%s" not found' % c.local_keys) parser.error('Local KEYS file "%s" not found' % c.local_keys)
c.java = make_java_config(parser, c.test_java17) c.java = make_java_config(parser, c.test_java18)
if c.tmp_dir: if c.tmp_dir:
c.tmp_dir = os.path.abspath(c.tmp_dir) c.tmp_dir = os.path.abspath(c.tmp_dir)