Disable the d2 warmup flaky test and deprecated jcenter (#1015)

brycezhongqing · web-flow · commit 0d2233adecc4 · 2024-08-12T20:06:57.000-07:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,9 @@ and what APIs have changed, if applicable.
 
 ## [Unreleased]
 
+## [29.58.3] - 2024-08-12
+- Disable the warmUp flaky unit test
+
 ## [29.58.2] - 2024-08-06
 - Add try/catch logic for INDIS xds stream initialization
 
@@ -5716,7 +5719,8 @@ patch operations can re-use these classes for generating patch messages.
 
 ## [0.14.1]
 
-[Unreleased]: https://github.com/linkedin/rest.li/compare/v29.58.2...master
+[Unreleased]: https://github.com/linkedin/rest.li/compare/v29.58.3...master
+[29.58.3]: https://github.com/linkedin/rest.li/compare/v29.58.2...v29.58.3
 [29.58.2]: https://github.com/linkedin/rest.li/compare/v29.58.1...v29.58.2
 [29.58.1]: https://github.com/linkedin/rest.li/compare/v29.58.0...v29.58.1
 [29.58.0]: https://github.com/linkedin/rest.li/compare/v29.57.2...v29.58.0
diff --git a/build.gradle b/build.gradle
@@ -16,7 +16,7 @@
 
 buildscript {
   repositories {
-    jcenter()
+    mavenCentral()
   }
   dependencies {
     classpath 'org.jfrog.buildinfo:build-info-extractor-gradle:4.32.0'
diff --git a/d2/src/test/java/com/linkedin/d2/balancer/util/WarmUpLoadBalancerTest.java b/d2/src/test/java/com/linkedin/d2/balancer/util/WarmUpLoadBalancerTest.java
@@ -128,7 +128,7 @@ public void testMakingWarmUpRequests() throws URISyntaxException, InterruptedExc
     Assert.assertEquals(VALID_FILES.size(), requestCount.get());
   }
 
-  @Test(timeOut = 10_000)
+  @Ignore("ingore this flaky test")
   public void testDeletingFilesAfterShutdown() throws InterruptedException, ExecutionException, TimeoutException
   {
     createDefaultServicesIniFiles();
diff --git a/gradle-plugins/src/integTest/groovy/com/linkedin/pegasus/gradle/PegasusPluginCacheabilityTest.groovy b/gradle-plugins/src/integTest/groovy/com/linkedin/pegasus/gradle/PegasusPluginCacheabilityTest.groovy
@@ -33,7 +33,7 @@ class PegasusPluginCacheabilityTest extends Specification {
     |}
     |
     |repositories { 
-    |  jcenter()
+    |  mavenCentral()
     |}
     |
     |dependencies {
diff --git a/gradle-plugins/src/integTest/groovy/com/linkedin/pegasus/gradle/PegasusPluginIntegrationTest.groovy b/gradle-plugins/src/integTest/groovy/com/linkedin/pegasus/gradle/PegasusPluginIntegrationTest.groovy
@@ -126,7 +126,7 @@ class PegasusPluginIntegrationTest extends Specification {
     |}
     |
     |repositories {
-    |  jcenter()
+    |  mavenCentral()
     |}
     |
     |dependencies {
diff --git a/gradle.properties b/gradle.properties
@@ -1,4 +1,4 @@
-version=29.58.2
+version=29.58.3
 group=com.linkedin.pegasus
 org.gradle.configureondemand=true
 org.gradle.parallel=true

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`
`17`	`17`	`buildscript {`
`18`	`18`	`repositories {`
`19`		`- jcenter()`
	`19`	`+ mavenCentral()`
`20`	`20`	`}`
`21`	`21`	`dependencies {`
`22`	`22`	`classpath 'org.jfrog.buildinfo:build-info-extractor-gradle:4.32.0'`
Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ public void testMakingWarmUpRequests() throws URISyntaxException, InterruptedExc`
`128`	`128`	`Assert.assertEquals(VALID_FILES.size(), requestCount.get());`
`129`	`129`	`}`
`130`	`130`
`131`		`- @Test(timeOut = 10_000)`
	`131`	`+ @Ignore("ingore this flaky test")`
`132`	`132`	`public void testDeletingFilesAfterShutdown() throws InterruptedException, ExecutionException, TimeoutException`
`133`	`133`	`{`
`134`	`134`	`createDefaultServicesIniFiles();`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ class PegasusPluginCacheabilityTest extends Specification {`
`33`	`33`	`\|}`
`34`	`34`	`\|`
`35`	`35`	`\|repositories {`
`36`		`- \| jcenter()`
	`36`	`+ \| mavenCentral()`
`37`	`37`	`\|}`
`38`	`38`	`\|`
`39`	`39`	`\|dependencies {`
Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ class PegasusPluginIntegrationTest extends Specification {`
`126`	`126`	`\|}`
`127`	`127`	`\|`
`128`	`128`	`\|repositories {`
`129`		`- \| jcenter()`
	`129`	`+ \| mavenCentral()`
`130`	`130`	`\|}`
`131`	`131`	`\|`
`132`	`132`	`\|dependencies {`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-version=29.58.2`
	`1`	`+version=29.58.3`
`2`	`2`	`group=com.linkedin.pegasus`
`3`	`3`	`org.gradle.configureondemand=true`
`4`	`4`	`org.gradle.parallel=true`