Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

Commit 77e755f

Browse files
authored
Merge pull request #5351 from liubin/fix/5348-dont-dump-containerd-log
cri-containerd: avoid printing containerd log twice
2 parents 4abd915 + 7a10ed1 commit 77e755f

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

integration/containerd/cri/integration-tests.sh

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,6 @@ err_report() {
162162
fi
163163
}
164164

165-
trap err_report ERR
166165

167166
check_daemon_setup() {
168167
info "containerd(cri): Check daemon works with runc"
@@ -475,7 +474,6 @@ main() {
475474

476475
info "containerd(cri): Running cri-integration"
477476

478-
TestContainerSwap
479477

480478
passing_test="TestContainerStats|TestContainerRestart|TestContainerListStatsWithIdFilter|TestContainerListStatsWithIdSandboxIdFilter|TestDuplicateName|TestImageLoad|TestImageFSInfo|TestSandboxCleanRemove"
481479

@@ -500,6 +498,12 @@ main() {
500498
CONTAINERD_CONFIG_FILE="$CONTAINERD_CONFIG_FILE_TEMP" \
501499
make GO_BUILDTAGS="no_btrfs" -e cri-integration
502500

501+
# trap error for print containerd log,
502+
# containerd's `cri-integration` will print the log itself.
503+
trap err_report ERR
504+
505+
TestContainerSwap
506+
503507
# TODO: runtime-rs doesn't support memory update currently
504508
if [ "$KATA_HYPERVISOR" != "dragonball" ]; then
505509
TestContainerMemoryUpdate 1

0 commit comments

Comments
 (0)