Skip to content

Conversation

@chmeliik
Copy link
Member

@chmeliik chmeliik commented Jun 5, 2024

STONEBLD-2339

All the images built in build-definitions CI now get released to the
quay.io/konflux-ci organization.

Replace the relevant redhat-appstudio[-tekton-catalog] references in
this repo with konflux-ci[/tekton-catalog] references.

Signed-off-by: Adam Cmiel acmiel@redhat.com

STONEBLD-2339

All the images built in build-definitions CI now get released to the
quay.io/konflux-ci organization.

Replace the relevant redhat-appstudio[-tekton-catalog] references in
this repo with konflux-ci[/tekton-catalog] references.

Signed-off-by: Adam Cmiel <acmiel@redhat.com>
@openshift-ci openshift-ci bot requested review from adambkaplan and mkosiarc June 5, 2024 12:05
Copy link
Member

@mmorhun mmorhun left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Member

@zregvart zregvart left a comment

Choose a reason for hiding this comment

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

/lgtm

I double checked the quay.io/konflux-ci/tekton-catalog/data-acceptable-bundles:latest image and it looks okay to me, it has all the current OCI references, and misses some git references currently (which we can add as needed using hack/trusted-tasks-update.sh from build-definitions

@openshift-ci
Copy link

openshift-ci bot commented Jun 5, 2024

@chmeliik: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/appstudio-hac-e2e-tests 87e89e3 link false /test appstudio-hac-e2e-tests

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

Copy link
Contributor

@robnester-rh robnester-rh left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci
Copy link

openshift-ci bot commented Jun 5, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chmeliik, mmorhun, rcerven, robnester-rh, zregvart

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 9872ba8 into redhat-appstudio:main Jun 5, 2024
enarha added a commit to enarha/pipeline-service that referenced this pull request Jul 10, 2024
The change was wrongly introduced in the infra-deployments deploy.yaml
files, making the updates from pipeline-service to override the change.
ref: redhat-appstudio/infra-deployments#3848
enarha added a commit to enarha/pipeline-service that referenced this pull request Jul 15, 2024
The change was wrongly introduced in the infra-deployments deploy.yaml
files, making the updates from pipeline-service to override the change.
ref: redhat-appstudio/infra-deployments#3848
enarha added a commit to openshift-pipelines/pipeline-service that referenced this pull request Jul 15, 2024
The change was wrongly introduced in the infra-deployments deploy.yaml
files, making the updates from pipeline-service to override the change.
ref: redhat-appstudio/infra-deployments#3848
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.

5 participants