Skip to content

Commit 1ef08fa

Browse files
authored
Disable BlockHound reporting for Kubernetes client testing (#6329)
Motivation: It is inconvient and time-consuming to check all CI results due to the false-positive BlockHound reports. Therefore, I propose disabling BlockHound reporting until [the patched version](eclipse-vertx/vert.x#5637) is released. Modifications: - Add `FastThreadLocalRunnable.run` to `allowBlockingCallsInside` for disabling the reports Result: Less noisy CI builds
1 parent 883e8bf commit 1ef08fa

2 files changed

Lines changed: 29 additions & 0 deletions

File tree

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
/*
2+
* Copyright 2025 LY Corporation
3+
*
4+
* LY Corporation licenses this file to you under the Apache License,
5+
* version 2.0 (the "License"); you may not use this file except in compliance
6+
* with the License. You may obtain a copy of the License at:
7+
*
8+
* https://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
12+
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
13+
* License for the specific language governing permissions and limitations
14+
* under the License.
15+
*/
16+
17+
package com.linecorp.armeria.client.kubernetes;
18+
19+
import reactor.blockhound.BlockHound.Builder;
20+
import reactor.blockhound.integration.BlockHoundIntegration;
21+
22+
public class KubernetesTestingBlockHoundIntegration implements BlockHoundIntegration {
23+
@Override
24+
public void applyTo(Builder builder) {
25+
// TODO(ikhoon): Remove this once https://github.com/eclipse-vertx/vert.x/pull/5637 is released.
26+
builder.allowBlockingCallsInside("io.netty.util.concurrent.FastThreadLocalRunnable", "run");
27+
}
28+
}
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
com.linecorp.armeria.client.kubernetes.KubernetesTestingBlockHoundIntegration

0 commit comments

Comments
 (0)