Skip to content

feat: upload E2E coverage to Codecov for policy-controller-operator#489

Draft
kdacosta0 wants to merge 1 commit into
mainfrom
kdacosta/SECURESIGN-4443-pco-e2e-codecov
Draft

feat: upload E2E coverage to Codecov for policy-controller-operator#489
kdacosta0 wants to merge 1 commit into
mainfrom
kdacosta/SECURESIGN-4443-pco-e2e-codecov

Conversation

@kdacosta0
Copy link
Copy Markdown
Member

Summary

  • Add Codecov CLI upload of e2e-coverage.out to the execute-operator-e2e step in both the bundle (policy-controller-operator-e2e.yaml) and FBC (policy-controller-fbc-e2e.yaml) integration test pipelines.
  • Inject CODECOV_TOKEN via secretKeyRef from the codecov-token Kubernetes secret.
  • Upload is non-fatal — a Codecov outage will never block test results.

Prerequisites

A codecov-token secret must be created in the Konflux tenant namespace before merging:

kubectl create secret generic codecov-token \
  --from-literal=token=<CODECOV_UPLOAD_TOKEN> \
  -n <konflux-tenant-namespace>

Test plan

  • Verify codecov-token secret exists in the Konflux namespace
  • Trigger a policy-controller-operator build and confirm the bundle E2E pipeline uploads coverage
  • Trigger an FBC build and confirm the FBC E2E pipeline uploads coverage
  • Verify coverage appears in Codecov under the e2e-tests flag
  • Verify pipeline still passes when Codecov upload fails (e.g. bad token)

Ref: SECURESIGN-4443

🤖 Generated with Claude Code

@kdacosta0 kdacosta0 marked this pull request as draft May 12, 2026 14:49
Add Codecov upload step to the execute-operator-e2e script in both
bundle and FBC integration test pipelines. After `make e2e-test`
completes, the codecov-cli uploads e2e-coverage.out with the
`e2e-tests` flag. The upload is non-fatal so test results are
never blocked by a Codecov outage.

Requires a `codecov-token` Kubernetes secret in the Konflux namespace.

Ref: SECURESIGN-4443

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kdacosta0 kdacosta0 force-pushed the kdacosta/SECURESIGN-4443-pco-e2e-codecov branch from fb5a295 to facc891 Compare May 13, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant