Skip to content

Red Hat Konflux kflux-prd-rh03 update coreos-assembler#4125

Merged
joelcapitao merged 1 commit into
mainfrom
konflux-coreos-assembler
Jun 11, 2025
Merged

Red Hat Konflux kflux-prd-rh03 update coreos-assembler#4125
joelcapitao merged 1 commit into
mainfrom
konflux-coreos-assembler

Conversation

@red-hat-konflux-kflux-prd-rh03
Copy link
Copy Markdown
Contributor

Pipelines as Code configuration proposal

To start the PipelineRun, add a new comment with content /ok-to-test

For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun

To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Please follow the block sequence indentation style introduced by the proprosed PipelineRuns YAMLs, or keep using consistent indentation level through your customized PipelineRuns. When different levels are mixed, it will be changed to the proposed style.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 4, 2025

Hi @red-hat-konflux-kflux-prd-rh03[bot]. Thanks for your PR.

I'm waiting for a coreos member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@joelcapitao
Copy link
Copy Markdown
Member

/retest

@joelcapitao
Copy link
Copy Markdown
Member

/ok-to-test

joelcapitao
joelcapitao previously approved these changes Jun 4, 2025
Comment thread .tekton/coreos-assembler-push.yaml
@joelcapitao joelcapitao enabled auto-merge (rebase) June 4, 2025 15:01
@joelcapitao
Copy link
Copy Markdown
Member

/retest

Signed-off-by: red-hat-konflux-kflux-prd-rh03 <konflux@no-reply.konflux-ci.dev>
@joelcapitao joelcapitao force-pushed the konflux-coreos-assembler branch from 6ee0058 to b8f7412 Compare June 5, 2025 06:29
@jlebon
Copy link
Copy Markdown
Member

jlebon commented Jun 5, 2025

Is that pipeline definition synced from anywhere/derived from any higher-level configuration? If so, where does that live?

@joelcapitao
Copy link
Copy Markdown
Member

joelcapitao commented Jun 11, 2025

Is that pipeline definition synced from anywhere/derived from any higher-level configuration? If so, where does that live?

This PR is a consequence of applying the YAML files of [1] with oc/kubectl.
I basically ran oc apply -k coreos-assembler locally before submitting the MR.
Now that's it's merged, ArgoCD takes care of the synchronization automatically.

As you can see in [2], we requested Konflux to send the PR with this annotation (i.e more information in [3], in another context but you get the idea).
We could remove this annotation and submit the tekton files manually.

When done automatically, Pipeline-as-Code (one of the Konflux component) pulls the pipeline definition set in [4] from [5].
In this case, it's [6].
Also, all the tasks defined in those pipelines live in the same repo at [7].
So you can see this repo as the source of truth of Konflux for the default Tekton pipelines and tasks.

As you can see, the *-pull-request.yaml and *-push.yaml files are quite the same except the trigger definition. We could simplify this later by defining the whole pipeline in a single file, and calling it as ref in the two files e.g [8]

[1] https://gitlab.cee.redhat.com/releng/konflux-release-data/-/merge_requests/6797
[2] https://gitlab.cee.redhat.com/releng/konflux-release-data/-/blob/a8c8e079bd6127de01a1a599f5fa6b9415be0018/tenants-config/cluster/kflux-prd-rh03/tenants/coreos-tenant/coreos-assembler/components.yaml#L8
[3] https://konflux-ci.dev/docs/building/reconfiguring-build-pipeline/#reset-the-build-pipeline
[4] https://gitlab.cee.redhat.com/releng/konflux-release-data/-/blob/a8c8e079bd6127de01a1a599f5fa6b9415be0018/tenants-config/cluster/kflux-prd-rh03/tenants/coreos-tenant/coreos-assembler/components.yaml#L9
[5] https://github.com/konflux-ci/build-definitions/tree/main/pipelines
[6] https://github.com/konflux-ci/build-definitions/tree/main/pipelines/docker-build-multi-platform-oci-ta
[7] https://github.com/konflux-ci/build-definitions/tree/main/task
[8] https://github.com/jcapiitao/fcc/tree/main/.tekton

@joelcapitao
Copy link
Copy Markdown
Member

/retest

@joelcapitao
Copy link
Copy Markdown
Member

/override ci/prow/rhcos
as it's failing on coreos/rhel-coreos-config#20.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 11, 2025

@jcapiitao: Overrode contexts on behalf of jcapiitao: ci/prow/rhcos

Details

In response to this:

/override ci/prow/rhcos
as it's failing on coreos/rhel-coreos-config#20.

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.

@joelcapitao joelcapitao merged commit 1508369 into main Jun 11, 2025
6 checks passed
@joelcapitao joelcapitao deleted the konflux-coreos-assembler branch June 11, 2025 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants