[FLPATH-4307] Add koku-onprem tekton pipelines for separate application - #6164
[FLPATH-4307] Add koku-onprem tekton pipelines for separate application#6164ydayagi wants to merge 22 commits into
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds a new Tekton pipeline run configuration for pull requests (koku-onprem-pull_request.yaml) and updates the application label in the push pipeline configuration (koku-onprem-push.yaml) to ensure consistency. The feedback suggests adding the GIT_COMMIT={{ revision }} build argument to the pull request pipeline configuration so that the commit SHA is correctly injected during the build process.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-pr Root cause: Konflux registry credentials not configured for
|
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-pr Root cause: Konflux registry credentials not configured for
|
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-ci / koku Root cause:
|
|
/retest |
1 similar comment
|
/retest |
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-enterprise-contract / koku-onprem Root cause: Konflux pipeline task versions are outdated (infra issue, not PR code)The enterprise contract policy check failed in the Evidence from AssessmentThis failure is not caused by code changes in this PR. The pipeline task bundle for ActionThis requires the team to update the Generated automatically. Review before applying. |
Add pull_request pipeline and update push pipeline to reference the new koku-onprem application instead of koku. Pass GIT_COMMIT build arg for image traceability. Signed-off-by: Yaron Dayagi <ydayagi@redhat.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Yaron Dayagi <ydayagi@redhat.com>
b3881ba to
4ccefa0
Compare
Signed-off-by: Yaron Dayagi <ydayagi@redhat.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Yaron Dayagi <ydayagi@redhat.com>
djnakabaale
left a comment
There was a problem hiding this comment.
hermetic builds should be enabled here
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ce to 5Gi Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…drasil-engine only Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…eto conflict Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-on-pull-request Root cause:
|
…dist available) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…for wheel-only packages - requirements.txt: app dependencies (sdist, no allow_binary) - requirements-build.txt: build tools for compiling sdists (flit_core, etc) - prefetch-binary/requirements.txt: wheel-only packages (yggdrasil-engine, pip, pipenv, etc) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-on-pull-request Root cause: The Konflux Evidence: Failing task: Action: The hashes in Generated automatically. Review before applying. |
…w_binary too) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-on-pull-request Root cause: Missing
|
Move pip, setuptools, pipenv, virtualenv, distlib, filelock, python-discovery to requirements-tools.txt (fetched as sdist). Only yggdrasil-engine remains in prefetch-binary (wheel-only). Add setuptools-scm and vcs-versioning to build deps. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The previous hashes were wheel hashes copied from prefetch-binary. Hermeto needs sdist hashes since these are fetched without allow_binary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
charset-normalizer requires setuptools<82.1 as build dependency. Our prefetched setuptools 83.0.0 is too new. Add 82.0.1 so pip can satisfy the version constraint during hermetic build. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-on-pull-request Root cause: The Docker image build ( Evidence: Classification: Konflux infrastructure issue — the Cachi2 prefetch cache is missing Action: Re-trigger the Generated automatically. Review before applying. |
|
🤖 CI Triager — Diagnosis Check: Red Hat Konflux / koku-onprem-on-pull-request Root cause: The Evidence: Classification: Konflux infrastructure issue — Hermeto's offline cache contains stale/mismatched checksums for build-time dependencies ( Action: Re-trigger the Generated automatically. Review before applying. |
|
/retest |
b01ec6e to
171c8a4
Compare
a818b67 to
be1f348
Compare
|
🤖 CI Triager — Diagnosis Check: Root cause: The Evidence from prior runs: Action: Re-trigger the pipeline (push an empty commit or ask a maintainer to Generated automatically. Review before applying. |
|
🤖 CI Triager — Diagnosis Check: Root cause: The Evidence: Action: Add a Then add the line with its hash to Generated automatically. Review before applying. |
…prefetch Full tree discovery found 10 missing build backends plus 4 transitive deps needed to compile sdist packages in hermetic builds: - poetry-core (click-didyoumean) - Cython 3.1.1 (grpcio, numpy, pandas, pyarrow, scipy, sqlalchemy, statsmodels) - maturin (orjson, pydantic-core) - meson-python, meson (numpy, pandas, scipy) - scikit-build-core (pyarrow) - hatch-fancy-pypi-readme (pydantic) - pybind11, pythran (scipy) - libcst (pyarrow) - pyproject-metadata, ply, gast, beniget (transitive deps) Also explicitly reference requirements-build.txt in the prefetch-input via both requirements_build_files on the sdist entry and requirements_files on the binary entry. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🤖 CI Triager — Diagnosis Check: Root cause: The Enterprise Contract policy check failed with 50 Evidence: Action: This is a pre-existing Konflux/EC infrastructure issue affecting all builds that include binary PyPI packages. No code fix is needed on this PR. The team should:
This failure is unrelated to the diff in this PR. Generated automatically. Review before acting. |
Summary
koku-onprem-pull_request.yamltekton pipeline for PR builds of the on-prem imagekoku-onprem-push.yamlto reference the newkoku-onpremapplication instead ofkokukoku-onpremapplication being created in konflux-release-data to resolve the enterprise contract policy conflict (registry-standardvsapp-interface-standard)Test plan
registry-standardpolicy