Test Verification Report: OCPBUGS-78310

JIRAOCPBUGS-78310: Warning about [controller-runtime] log.SetLogger(...) was never called when using hcp CLI
PRHyperShift #8955: OCPBUGS-78310: fix(cli): initialize controller-runtime logger in product-cli
Commit8ea96d7f8c (main) + PR #8955 diff applied
Clusterbrcox-azure-dev-hc (Azure, 4.22.0) on AWS dev mgmt cluster
Date2026-07-13
TesterBryan Cox
1/1
Unit Tests Passing
30/30
CI Jobs Passed
6/6
Scenarios Verified
6/6
Regression Checks

Summary

CategoryStatus
Build (make product-cli)PASS
Unit Tests (go test ./product-cli/...)PASS
Lint / Verify / CIPASS
Scenario 1: Warning eliminated (create & destroy paths)PASS
Scenario 2: Structured JSON log output (RFC3339)PASS
Scenario 3: Platform create command — no regression (real AWS HostedCluster created)PASS
Scenario 4: Destroy command — no regressionPASS
Scenario 5: Basic CLI functionality (version, --help)PASS
Scenario 6: Before/after binary comparisonPASS

Regression Checklist

CheckResultNotes
AWS cluster create works end-to-endPASSReal AWS HostedCluster created with fixed binary: VPC, subnets, IAM roles, DNS zones, Kubernetes resources — 52 JSON log lines, zero warnings
Destroy commands work without regressionPASSAzure and AWS destroy paths exercised; no warnings, correct error messages
Log format matches hypershift-operator/control-plane-operatorPASSJSON-encoded, RFC3339 timestamps; same zap.JSONEncoder + zapcore.RFC3339TimeEncoder pattern
Signal handling still works (SIGINT)PASSSIGINT handler unchanged; ctrl.SetLogger() added before cobra command setup
Help output identical before/afterPASSdiff confirms identical --help output
Version command worksPASSClient and server version displayed correctly; no warnings

CI Job Results (30/30 passed)

Prow Jobs

JobStatusLink
e2e-aksPASSProw
e2e-awsPASSProw
e2e-aws-upgrade-hypershift-operatorPASSProw
e2e-azure-v2-self-managedPASSProw
e2e-kubevirt-aws-ovn-reducedPASSProw
e2e-v2-awsPASSProw
e2e-v2-gkePASSProw
imagesPASSProw
okd-scos-imagesPASSProw
securityPASSProw
verify-depsPASSProw

GitHub Actions

JobStatusLink
LintPASSGHA
VerifyPASSGHA
Unit Tests (cmd-support)PASSGHA
Unit Tests (cpo-hostedcontrolplane)PASSGHA
Unit Tests (cpo-other)PASSGHA
Unit Tests (other)PASSGHA
Unit Tests (hypershift-operator)PASSGHA
CodespellPASSGHA
GitlintPASSGHA
CodeRabbitPASSReview completed
Codecov (patch)PASSCodecov
Codecov (project)PASSCodecov

Konflux

JobStatusLink
hypershift-cli-mce-50-on-pull-requestPASSKonflux
hypershift-operator-main-on-pull-requestPASSKonflux
hypershift-release-mce-50-on-pull-requestPASSKonflux

Full CI history: Prow PR History | GitHub Checks