Skip to content

Add Tekton Results & S3 integration for osde2e#513

Open
YiqinZhang wants to merge 22 commits intoopenshift:tekton-resultsfrom
YiqinZhang:tekton-results
Open

Add Tekton Results & S3 integration for osde2e#513
YiqinZhang wants to merge 22 commits intoopenshift:tekton-resultsfrom
YiqinZhang:tekton-results

Conversation

@YiqinZhang
Copy link
Contributor

@YiqinZhang YiqinZhang commented Dec 3, 2025

Introduces Tekton Results for enhanced observability, S3 storage for test artifacts, and Loki integration for log aggregation.

Changes

New Files

  • osde2e-tekton-task.yml - Main test Task definition
  • upload-to-s3-task.yml - S3 upload Task for test results
  • osde2e-pipeline.yml - Pipeline orchestrating test execution and result upload
  • e2e-tekton-template.yml - OpenShift Template for easy PipelineRun creation
  • setup-complete-tekton-stack.sh - Automated setup script
  • run-with-credentials.sh - Interactive test runner with credential management
  • view-pipeline-logs.sh - Log viewer utility
  • tekton-results-api.sh - Results API query helper

Documentation (doc/)

  • MANUAL-SETUP-GUIDE.md - Complete step-by-step manual setup
  • REQUIRED-CONFIG-FILES.md - Configuration file reference
  • TROUBLESHOOTING.md - Common issues and solutions
  • QUICK-REFERENCE.md - Command cheat sheet
  • STORAGE-GUIDE.md - Storage architecture overview
  • QUERY-RESULTS-GUIDE.md - Methods to retrieve test results

Optional Components

  • ClusterLogForwarder.yaml - Log forwarding to Loki
  • loki-s3-policy.json - IAM policy reference

Key Features

  • Tekton Results integration - Persistent storage of run metadata in PostgreSQL
  • S3 upload - Automatic upload of test logs and JUnit XML with pre-signed URLs
  • Structured results - PASS/FAIL status, test summary, and logs captured as Task results
  • Dynamic configuration - Auto-detection of Loki Operator channel and LokiStack sizing

SDCICD-1672

@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Dec 3, 2025
@YiqinZhang YiqinZhang changed the title Add Tekton Results integration for osde2e Add Tekton Results & S3 integration for osde2e Dec 3, 2025
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 3, 2025
@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@YiqinZhang YiqinZhang force-pushed the tekton-results branch 2 times, most recently from 2dd3cb3 to 2d2edc2 Compare December 4, 2025 15:39
Bumps ubi9/ubi-minimal from 9.7-1763362218 to 9.7-1764794109.

---
updated-dependencies:
- dependency-name: ubi9/ubi-minimal
  dependency-version: 9.7-1764794109
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@YiqinZhang YiqinZhang force-pushed the tekton-results branch 2 times, most recently from e74503d to 548d1cf Compare December 16, 2025 15:56
@ritmun
Copy link
Contributor

ritmun commented Dec 17, 2025

/hold

needs rebase.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants