feat: migrate to testcontainers-go v0.42.0 and upgrade dependencies (#146)

* feat: migrate to testcontainers-go v0.42.0 and upgrade all major dependencies

- Delete compose_provider.go (601 lines) — custom DockerContainer/DockerProvider
  re-implementation replaced by native testcontainers-go APIs
- Delete compose_listener.go (85 lines) — Docker event listener no longer needed;
  containers are accessible after stack.Up() via stack.ServiceContainer()
- Rewrite compose.go to use compose.NewDockerComposeWith + stack.Up/ServiceContainer
- Rewrite cleanup/compose.go to use stack.Down
- Upgrade testcontainers-go v0.11.1 → v0.42.0
- Upgrade docker/docker v20.10.7 → v28.5.2
- Upgrade k8s.io/* v0.22.2 → v0.35.3
- Upgrade sigs.k8s.io/kind v0.27.0 → v0.31.0
- Upgrade spf13/cobra v1.8.0 → v1.10.2
- Remove docker-compose v1 binary install from CI (v0.42.0 uses Docker Compose v2 plugin)
- Update action.yaml Go version to 1.26

Closes #139

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: add multi-platform CI build matrix and CGO_ENABLED support

Split Build workflow into lint-and-test + build-platform (ubuntu/windows/macos)
matrix jobs. Enable CGO for native builds. Update Dockerfile with build-essential
for CGO. Change `make build` to build for current OS only (CI matrix handles
cross-platform).

Based on #139

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: remove compose.Wait(true) to avoid requiring all containers healthy

compose.Wait(true) maps to `docker compose up --wait` which requires ALL
containers to remain running and healthy. This is too strict for e2e tests
where some containers may not have healthchecks or may exit after init.
Port readiness is already handled by WaitForService() with per-port
wait strategies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: remove containerd image store disable workaround

No longer needed — testcontainers-go v0.42.0 uses Docker Compose v2
plugin which fully supports the containerd image store.

Reverts the workaround added in 8c21e43 (#140).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add manual port waiting after compose up

WaitForService strategies only execute when compose.Wait(true) is passed,
but compose.Wait(true) is too strict (requires all containers healthy).
Instead, manually wait for each port via TCP dial after Up() returns,
matching the old code's behavior of waitPortUntilReady().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: restore DOCKER_API_VERSION negotiation in action.yaml

Docker SDK v28.5.2 defaults to API v1.51, but GitHub Actions runners
may only support up to v1.48. Set DOCKER_API_VERSION to the daemon's
supported version to ensure compatibility.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add logging to port waiting and try immediately before polling

Add INFO-level logs for port wait start/completion and DEBUG-level for
retry attempts. Also try connecting immediately before the first ticker
interval to avoid unnecessary 500ms delay when ports are already ready.
Log when a service has no ports to wait for.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: wait for Docker healthcheck before port readiness check

After stack.Up(), wait for each container's Docker healthcheck to pass
(if defined) before doing TCP port checks. This matches the old behavior
where docker-compose up -d would respect depends_on: condition: service_healthy.

Also restored internal container port check (/proc/net/tcp + nc + /dev/tcp)
from the old WaitPort implementation to align with original behavior.

The wait sequence is now:
1. stack.Up(ctx) — start containers
2. waitForHealthy — poll Docker health status until healthy (if healthcheck defined)
3. waitForPort — external TCP dial + internal container check
4. Export env vars

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: only wait for healthcheck on services with ports

The old code only waited for services that had ports: in docker-compose.yml.
Services without ports were completely skipped. The healthcheck wait was
incorrectly running on ALL services, causing timeouts on services like
init containers or senders whose healthchecks depend on other services.

Now: services without ports get log streaming only. Services with ports
get healthcheck wait + port wait + env export (matching old behavior).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: remove healthcheck waiting, use only TCP+internal port check

The old code never checked Docker healthcheck status. It only did
TCP dial + internal /proc/net/tcp check. Many compose files use
healthchecks with tools (nc) not available in the container image,
causing Docker to mark containers as unhealthy even when the
application is running fine.

Remove waitForHealthy entirely to match old behavior exactly:
port-level waiting only, no Docker healthcheck dependency.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: restore build target to build all platforms for release

The build target must build all platforms (windows/linux/darwin) because
release-bin depends on it. Each platform build uses CGO_FLAG to enable
CGO only for native builds.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: build only current platform per CI matrix runner

Each matrix runner (ubuntu/windows/macos) now builds only its own
platform instead of all three redundantly. The `make build` target
is preserved for release which needs all platforms.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* ci: simplify build matrix by moving OS-to-target mapping into matrix include

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: kezhenxu94 <kezhenxu94@apache.org>

Author: 3 people

.github/workflows/build.yaml

@@ -5,9 +5,9 @@
 # the Apache License, Version 2.0 (the "License"); you may
 # not use this file except in compliance with the License.
 # You may obtain a copy of the License at
-# 
+#
 #     http://www.apache.org/licenses/LICENSE-2.0
-# 
+#
 # Unless required by applicable law or agreed to in writing,
 # software distributed under the License is distributed on an
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
@@ -17,16 +17,14 @@
 #
 
 name: Build
-
 on:
   pull_request:
   push:
     branches:
       - main
-
 jobs:
-  build:
-    name: Build
+  lint-and-test:
+    name: Lint and Test
     runs-on: ubuntu-latest
     steps:
       - name: Set up Go
@@ -36,23 +34,51 @@ jobs:
           cache-dependency-path: |
             go.sum
         id: go
-
       - name: Check out code into the Go module directory
         uses: actions/checkout@v4
-
       - name: Check License
         uses: apache/skywalking-eyes@ec88b7d850018c8983f87729ea88549e100c5c82
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-
       - name: Lint Codes
         run: make lint
-
       - name: Test
-        run: make test
-
+        run: CGO_ENABLED=1 make test
+  build-platform:
+    name: Build (${{ matrix.os }})
+    strategy:
+      matrix:
+        include:
+          - os: ubuntu-latest
+            target: linux
+          - os: windows-latest
+            target: windows
+          - os: macos-latest
+            target: darwin
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: 1.26
+          cache-dependency-path: |
+            go.sum
+        id: go
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v4
       - name: Build
-        run: make build
-
+        run: make ${{ matrix.target }}
       - name: Build Docker Image
+        if: matrix.os == 'ubuntu-latest'
         run: make docker
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    needs:
+      - lint-and-test
+      - build-platform
+    steps:
+      - name: Check out code into the Go module directory
+        uses: actions/checkout@v4
+      - name: Result
+        run: echo "Lint and Test, Build (ubuntu-latest), Build (windows-latest), Build (macos-latest) completed successfully!"

.github/workflows/e2e-test.yaml

@@ -44,16 +44,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Build e2e
-        run: make linux
-
-      - name: Install docker-compose
-        shell: bash
-        run: |
-          if ! command docker-compose 2>&1 > /dev/null; then
-            echo "Installing docker-compose"
-            sudo curl -L "https://github.com/docker/compose/releases/download/1.29.2/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose
-            sudo chmod +x /usr/local/bin/docker-compose
-          fi
+        run: CGO_ENABLED=1 make linux
 
       - name: Run E2E Test
         uses: apache/skywalking-infra-e2e@main

Dockerfile

@@ -5,9 +5,9 @@
 # the Apache License, Version 2.0 (the "License"); you may
 # not use this file except in compliance with the License.
 # You may obtain a copy of the License at
-# 
+#
 #     http://www.apache.org/licenses/LICENSE-2.0
-# 
+#
 # Unless required by applicable law or agreed to in writing,
 # software distributed under the License is distributed on an
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
@@ -22,7 +22,10 @@ WORKDIR /e2e
 
 COPY . .
 
-RUN make linux
+# Install build dependencies for CGO
+RUN apt-get update && apt-get install -y build-essential && rm -rf /var/lib/apt/lists/*
+
+RUN CGO_ENABLED=1 make linux
 
 FROM golang:1.26 AS bin
 

Makefile

@@ -25,6 +25,10 @@ GO := GO111MODULE=on go
 GO_PATH = $(shell $(GO) env GOPATH)
 GOARCH ?= $(shell $(GO) env GOARCH)
 GOOS ?= $(shell $(GO) env GOOS)
+HOST_GOOS = $(shell $(GO) env GOOS)
+HOST_GOARCH = $(shell $(GO) env GOARCH)
+# CGO is enabled only for native builds; cross-compilation requires target SDK
+CGO_FLAG = $(shell if [ "$(os)" = "$(HOST_GOOS)" ] && [ "$(GOARCH)" = "$(HOST_GOARCH)" ]; then echo 1; else echo 0; fi)
 GO_BUILD = $(GO) build
 GO_TEST = $(GO) test
 GO_LINT = $(GO_PATH)/bin/golangci-lint
@@ -57,7 +61,7 @@ windows: PROJECT_SUFFIX=.exe
 .PHONY: $(PLATFORMS)
 $(PLATFORMS):
 	mkdir -p $(OUT_DIR)
-	GOOS=$(os) GOARCH=$(GOARCH) $(GO_BUILD) $(GO_BUILD_FLAGS) -ldflags "$(GO_BUILD_LDFLAGS)" -o $(OUT_DIR)/$(os)/$(PROJECT)$(PROJECT_SUFFIX) cmd/e2e/main.go
+	GOOS=$(os) GOARCH=$(GOARCH) CGO_ENABLED=$(CGO_FLAG) $(GO_BUILD) $(GO_BUILD_FLAGS) -ldflags "$(GO_BUILD_LDFLAGS)" -o $(OUT_DIR)/$(os)/$(PROJECT)$(PROJECT_SUFFIX) cmd/e2e/main.go
 
 .PHONY: build
 build: windows linux darwin

action.yaml

@@ -31,35 +31,14 @@ inputs:
 runs:
   using: "composite"
   steps:
-    - name: Disable containerd image store
+    - name: Set Docker API version
       shell: bash
       run: |
-        DAEMON_JSON="/etc/docker/daemon.json"
-        if [ -f "$DAEMON_JSON" ]; then
-          sudo jq '. + {"features": {"containerd-snapshotter": false}}' "$DAEMON_JSON" \
-            | sudo tee "${DAEMON_JSON}.tmp" > /dev/null
-          sudo mv "${DAEMON_JSON}.tmp" "$DAEMON_JSON"
-        else
-          echo '{"features": {"containerd-snapshotter": false}}' \
-            | sudo tee "$DAEMON_JSON" > /dev/null
-        fi
-        sudo systemctl restart docker
-        docker version
-        docker info
         echo "DOCKER_API_VERSION=$(docker version --format '{{.Server.APIVersion}}')" >> "$GITHUB_ENV"
-    - name: Install docker-compose
-      shell: bash
-      if: runner.os != 'Windows'
-      run: |
-        if ! command docker-compose 2>&1 > /dev/null; then
-          echo "Installing docker-compose"
-          sudo curl -L "https://github.com/docker/compose/releases/download/1.29.2/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose
-          sudo chmod +x /usr/local/bin/docker-compose
-        fi
     - name: Set up Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v5
       with:
-        go-version: 1.24
+        go-version: 1.26
         cache-dependency-path: ${{ github.action_path }}/go.sum
     - shell: bash
       run: make -C $GITHUB_ACTION_PATH install DESTDIR=/usr/local/bin