Kubernetes executor direct tests fail without envtest assets

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#7,013 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

@singhharsh1708 đang làm issue này rồi.

Từ ngày 6/9/2026.

  • #7014 của @rootp1 — đã đóng, không merge

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
76/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go, kubernetes
Lĩnh vực
devops, testing-qa

Hướng nghiên cứu

Bắt đầu với pkg/app/piped/executor/kubernetes/sync_test.go, đặc biệt là TestExecutor_ensureSync và phần thiết lập envtest.Environment, sau đó so sánh việc cung cấp asset trong Makefile và target make test/go. Chạy trực tiếp lệnh go test khi KUBEBUILDER_ASSETS chưa được thiết lập. Hoàn tất khi việc thiếu asset envtest tạo ra thông báo skip hoặc setup rõ ràng, trong khi make test/go vẫn chạy test với các asset đã được cung cấp.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

What happened:

Direct package-level Go tests for the Kubernetes executor are not self-contained. Running go test ./pkg/app/piped/executor/kubernetes -run TestExecutor_ensureSync -count=1 without preinstalled envtest assets fails while starting envtest.Environment with exec: "etcd": executable file not found in $PATH or fork/exec /usr/local/kubebuilder/bin/etcd: no such file or directory.

What you expected to happen:

Direct package tests should either bootstrap envtest prerequisites automatically or skip with a clear message when the assets are unavailable, instead of failing as though the executor logic is broken.

How to reproduce it:

  1. Ensure KUBEBUILDER_ASSETS is unset and no local envtest binaries are installed.
  2. Run go test ./pkg/app/piped/executor/kubernetes -run TestExecutor_ensureSync -count=1.
  3. Observe TestExecutor_ensureSync fail while envtest.Environment starts.
  4. Compare that with make test/go, which installs and exports envtest assets via setup-envtest in Makefile before running the Go tests.

Environment:

  • piped version: repository checkout on 2026-07-11 (upstream/master fetched to d720a26a during validation)
  • control-plane version: Not applicable
  • Others: reproduced locally during repository assessment

Summary

Direct Go tests for the Kubernetes executor rely on envtest binaries that are only provisioned by the Makefile wrapper.

Affected Area

pkg/app/piped/executor/kubernetes/sync_test.go, Makefile

Expected Behaviour

The test should either provision envtest requirements or skip with a clear environment-related message.

Actual Behaviour

The test fails at envtest startup when envtest binaries are not available.

Technical Evidence

  • pkg/app/piped/executor/kubernetes/sync_test.go creates envtest.Environment directly and calls Start().
  • Makefile provisions setup-envtest and exports KUBEBUILDER_ASSETS for make test/go.
  • Reproduced failure:
    • go test ./pkg/app/piped/executor/kubernetes -run TestExecutor_ensureSync -count=1
    • Error: Received unexpected error: unable to start control plane itself: failed to start the controlplane. retried 5 times: exec: "etcd": executable file not found in $PATH

Suspected Root Cause

The test assumes envtest binaries are already available, but that prerequisite is encoded only in the Make target and not in the test itself.

Proposed Direction

Add an explicit envtest asset preflight in the test so it either uses available binaries or skips with a clear message when the environment is missing the required assets.

Acceptance Criteria

  • Direct go test on the package no longer reports a misleading functional failure when envtest assets are missing.
  • make test/go continues to run the test unchanged when envtest assets are provisioned.
  • Contributors can distinguish environmental setup issues from real executor regressions immediately.
Ngôn ngữ chính
Go
Star
1.4k
Fork
391
Merge trung bình
4 ngày 1 giờ
Pull request đã merge (30 ngày)
58

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của pipe-cd/pipecd

Tất cả issue của pipe-cd/pipecd

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.