Skip to content

Commit 2a44c4a

Browse files
authored
Merge branch 'master' into quintenp/netty
2 parents ffb1fb4 + d15fd4f commit 2a44c4a

3 files changed

Lines changed: 9 additions & 20 deletions

File tree

common/scala/build.gradle

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,7 @@ dependencies {
198198
api("ch.qos.logback:logback-core:1.2.13")
199199
api("io.netty:netty-codec:${gradle.netty.version}")
200200
api("io.netty:netty-common:${gradle.netty.version}")
201+
api("com.fasterxml.jackson.core:jackson-databind:${gradle.jackson.version}")
201202
}
202203
}
203204

@@ -206,10 +207,10 @@ configurations {
206207
exclude group: 'commons-logging'
207208
exclude group: 'log4j'
208209
}
209-
all {
210+
configureEach {
210211
resolutionStrategy.dependencySubstitution {
211212
// CVE-2025-12183, CVE-2025-66566: org.lz4:lz4-java relocated to at.yawk.lz4 transitive dependency of kafka-clients
212-
substitute module('org.lz4:lz4-java') using module('at.yawk.lz4:lz4-java:1.10.3')
213+
substitute module("org.lz4:lz4-java") using module("at.yawk.lz4:lz4-java:1.10.3")
213214
}
214215
}
215216
}

settings.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,4 +109,5 @@ gradle.ext.grpc = [version : '1.75.0']
109109
gradle.ext.curator = [version : '5.7.0']
110110
gradle.ext.kube_client = [version: '4.10.3']
111111

112+
gradle.ext.jackson = [version: '2.21.1']
112113
gradle.ext.netty = [version : '4.1.132.Final']

tests/src/test/scala/org/apache/openwhisk/core/containerpool/v2/test/InvokerHealthManagerTests.scala

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ package org.apache.openwhisk.core.containerpool.v2.test
1919

2020
import org.apache.pekko.actor.FSM.{CurrentState, SubscribeTransitionCallBack, Transition}
2121
import org.apache.pekko.actor.{ActorRef, ActorRefFactory, ActorSystem}
22-
import org.apache.pekko.testkit.{ImplicitSender, TestActor, TestFSMRef, TestKit, TestProbe}
22+
import org.apache.pekko.testkit.{ImplicitSender, TestFSMRef, TestKit, TestProbe}
2323
import common.StreamLogging
2424
import org.apache.openwhisk.common.InvokerState.{Healthy, Offline, Unhealthy}
2525
import org.apache.openwhisk.common.{Enable, GracefulShutdown, RingBuffer}
@@ -331,26 +331,13 @@ class InvokerHealthManagerTests
331331
instanceId.tags,
332332
instanceId.dedicatedNamespaces).serialize))
333333

334-
val mockHealthActionProxy = TestProbe()
335-
336-
mockHealthActionProxy.setAutoPilot((sender, msg) =>
337-
msg match {
338-
case _: Initialize =>
339-
(1 to InvokerHealthManager.bufferSize - InvokerHealthManager.bufferErrorTolerance) foreach { _ =>
340-
sender ! HealthMessage(true)
341-
}
342-
TestActor.KeepRunning
343-
344-
case GracefulShutdown =>
345-
TestActor.KeepRunning
346-
})
347-
348-
fsm.underlyingActor.healthActionProxy = Some(mockHealthActionProxy.ref)
349-
350334
fsm ! Enable
335+
(1 to InvokerHealthManager.bufferSize - InvokerHealthManager.bufferErrorTolerance) foreach { _ =>
336+
fsm ! HealthMessage(true)
337+
}
351338

352339
probe.expectMsg(Transition(fsm, Offline, Unhealthy))
353-
probe.expectMsg(10.seconds, Transition(fsm, Unhealthy, Healthy))
340+
probe.expectMsg(Transition(fsm, Unhealthy, Healthy))
354341

355342
dataManagementService.expectMsg(
356343
UpdateDataOnChange(

0 commit comments

Comments
 (0)