Skip to content

LCORE-1935: Fix the lightspeed-stack-0-6 Konflux component pipelines#1884

Merged
tisnik merged 1 commit into
lightspeed-core:mainfrom
syedriko:syedriko-lightspeed-stack-0-6-fix
Jun 11, 2026
Merged

LCORE-1935: Fix the lightspeed-stack-0-6 Konflux component pipelines#1884
tisnik merged 1 commit into
lightspeed-core:mainfrom
syedriko:syedriko-lightspeed-stack-0-6-fix

Conversation

@syedriko

@syedriko syedriko commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Description

Fix the image repo in the lightspeed-stack-0-6 Konflux component pipelines

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement
  • Benchmarks improvement

Tools used to create PR

Identify any AI code assistants used in this PR (for transparency and review context)

  • Assisted-by: (e.g., Claude, CodeRabbit, Ollama, etc., N/A if not used)
  • Generated by: (e.g., tool name and version; N/A if not used)

Related Tickets & Documents

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

Summary by CodeRabbit

  • Chores
    • CI/CD pipeline now publishes images to an updated repository path and includes adjusted build defaults.
  • New Features
    • Multi-architecture builds now include an additional ARM target; image build entrypoint updated.
    • Pipeline now adds an additional image tag based on commit shorthand.
  • Security / Tests
    • Security scan workflow expanded with new Coverity checks and updated SAST tasks and arguments.
  • Reliability
    • Pipeline and task timeouts set to 4 hours each.

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

This PR updates two Tekton pipeline YAMLs: restructures pipeline parameters/defaults and annotations; changes build/prefetch wiring and output-image/defaults; updates multiple task bundle references; adds Coverity SAST tasks and gates their execution; adjusts SAST task inputs/args; updates tag application and rpms scan bundles; and sets 4h timeouts.

Changes

Tekton pipeline configuration and SAST updates

Layer / File(s) Summary
Pipeline metadata and parameter reshaping
.tekton/lightspeed-stack-0-6-pull-request.yaml, .tekton/lightspeed-stack-0-6-push.yaml
Annotation formatting and many pipeline spec.params are reformatted/reordered or clarified (including path-context, dockerfile, image-expires-after, buildah-format, privileged-nested, enable-package-registry-proxy, sast-target-dirs) and the removed enable-cache-proxy wiring is handled.
Build/prefetch task wiring and image defaults
.tekton/lightspeed-stack-0-6-pull-request.yaml, .tekton/lightspeed-stack-0-6-push.yaml
output-image defaults and build defaults (dockerfile, build-platforms) are updated; prefetch inputs expanded; prefetch-dependencies and buildah-remote-oci-ta task bundle references and parameter wiring are updated to include additional artifacts/platforms.
SAST task updates, Coverity addition, tag and rpms changes
.tekton/lightspeed-stack-0-6-pull-request.yaml, .tekton/lightspeed-stack-0-6-push.yaml
SAST task bundle digests are updated; sast-snyk-check receives additional ARGS and source artifact inputs; new Coverity tasks (sast-coverity-check, coverity-availability-check) added with conditional gating; sast-unicode-check, sast-shell-check updates; apply-tags adds ADDITIONAL_TAGS with short commit; rpms-signature-scan bundle updated.
Pipeline and task timeout settings
.tekton/lightspeed-stack-0-6-pull-request.yaml, .tekton/lightspeed-stack-0-6-push.yaml
Introduces timeouts section setting pipeline: 4h and tasks: 4h in both pipeline YAMLs.

🎯 4 (Complex) | ⏱️ ~45 minutes

Suggested labels: ok-to-test

Suggested reviewers:

  • tisnik
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the main change: fixing the lightspeed-stack-0-6 Konflux component pipelines, which aligns with the core modifications to the two pipeline YAML files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@syedriko syedriko force-pushed the syedriko-lightspeed-stack-0-6-fix branch from 1b79b79 to f330e68 Compare June 9, 2026 17:28
@syedriko syedriko changed the title Fix the lightspeed-stack-0-6 Konflux component pipelines LCORE-1935: Fix the lightspeed-stack-0-6 Konflux component pipelines Jun 9, 2026

@tisnik tisnik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tisnik tisnik merged commit 0f5c230 into lightspeed-core:main Jun 11, 2026
32 of 34 checks passed
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.

2 participants