Skip to content

Inject eventing-integration images references #3543

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 13, 2025

Conversation

pierDipi
Copy link
Member

@pierDipi pierDipi commented Mar 11, 2025

Depends on eventing-integrations images in Konflux Quay repository

Signed-off-by: Pierangelo Di Pilato <[email protected]>
Comment on lines +6789 to +6805
apiVersion: v1
kind: ConfigMap
metadata:
name: eventing-integrations-images
namespace: knative-eventing
labels:
app.kubernetes.io/version: "v20250220-98eb037"
data:
aws-s3-sink: gcr.io/knative-nightly/aws-s3-sink@sha256:c14c49821ccf59b8584a13523ea04a164c418fdb899e584809a2d4cb5de1ef76
aws-sqs-source: gcr.io/knative-nightly/aws-sqs-source@sha256:03c355c8d91db9dbc97b076734a61c0ccc283f84b67e03541fec675c9e5f899d
aws-sns-sink: gcr.io/knative-nightly/aws-sns-sink@sha256:4c8972ccea48ac2823bbc6e44f3b0b9a1daeb0b5727c08fe25e8992a0c9673c2
timer-source: gcr.io/knative-nightly/timer-source@sha256:4270869170ab6a0d717d7a904f47fdd7899b1fc8d9a1457b68de5ff8e667a4ed
aws-ddb-streams-source: gcr.io/knative-nightly/aws-ddb-streams-source@sha256:9670cc0ab31113d244176f3490082e5ea155d64ab7d3dcaeb5027e40cb9c9859
aws-s3-source: gcr.io/knative-nightly/aws-s3-source@sha256:1d6bdd165da87126cfb6a9f9e90e7343a6a4a768415627ef060d2727e46b4f59
log-sink: gcr.io/knative-nightly/log-sink@sha256:15be39524a6467367c5d21a15642a972047d929555f79b4d26f0188060cc03cd
aws-sqs-sink: gcr.io/knative-nightly/aws-sqs-sink@sha256:d6d962fc3d6af2ec0456319785ed97f22399fbeff562f8bb1a5bce71be1b6970
---
Copy link
Member Author

Choose a reason for hiding this comment

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

this will be removed after we merge openshift-knative/eventing#1191 (that one depends on this PR)

@pierDipi pierDipi marked this pull request as ready for review March 12, 2025 14:04
@openshift-ci openshift-ci bot requested review from matzew and skonto March 12, 2025 14:04
knative.dev/networking v0.0.0-20241022012959-60e29ff520dc
knative.dev/operator v0.43.1
knative.dev/operator v0.43.3-0.20250312021337-6b0f076ba7ae
Copy link
Member

Choose a reason for hiding this comment

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

did you get this via make generated-files and its call to update deps?

Copy link
Member Author

Choose a reason for hiding this comment

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

these 2:

go get -u knative.dev/[email protected]
make generated-files

Copy link
Member

@matzew matzew left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

Copy link
Contributor

openshift-ci bot commented Mar 12, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matzew, pierDipi

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

@pierDipi
Copy link
Member Author

/override "CSV revision check / CSV component revision alignment check"

Copy link
Contributor

openshift-ci bot commented Mar 12, 2025

@pierDipi: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • CSV revision check / CSV component revision alignment check

Only the following failed contexts/checkruns were expected:

  • CSV component revision alignment check
  • Konflux kflux-prd-rh02 / serverless-openshift-kn-operator-136-on-pull-request
  • Update deps
  • ci/prow/417-aws-ovn-images
  • ci/prow/417-azure-images
  • ci/prow/417-gcp-images
  • ci/prow/417-hypershift-images
  • ci/prow/417-images
  • ci/prow/417-operator-e2e
  • ci/prow/417-osd-images
  • ci/prow/417-single-node-images
  • ci/prow/417-test-upgrade
  • ci/prow/417-upstream-e2e
  • ci/prow/417-upstream-e2e-kafka
  • ci/prow/417-vsphere-images
  • ci/prow/ocp4.17-lp-rosa-classic-images
  • ci/prow/ocp4.18-lp-interop-images
  • pull-ci-openshift-knative-serverless-operator-main-417-aws-ovn-images
  • pull-ci-openshift-knative-serverless-operator-main-417-azure-images
  • pull-ci-openshift-knative-serverless-operator-main-417-gcp-images
  • pull-ci-openshift-knative-serverless-operator-main-417-hypershift-images
  • pull-ci-openshift-knative-serverless-operator-main-417-images
  • pull-ci-openshift-knative-serverless-operator-main-417-operator-e2e
  • pull-ci-openshift-knative-serverless-operator-main-417-osd-images
  • pull-ci-openshift-knative-serverless-operator-main-417-single-node-images
  • pull-ci-openshift-knative-serverless-operator-main-417-test-upgrade
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e-kafka
  • pull-ci-openshift-knative-serverless-operator-main-417-vsphere-images
  • pull-ci-openshift-knative-serverless-operator-main-ocp4.17-lp-rosa-classic-images
  • pull-ci-openshift-knative-serverless-operator-main-ocp4.18-lp-interop-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override "CSV revision check / CSV component revision alignment check"

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.

@pierDipi
Copy link
Member Author

/test serverless-openshift-kn-operator-136-on-pull-request

Copy link
Contributor

openshift-ci bot commented Mar 12, 2025

@pierDipi: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test 414-images
/test 414-operator-e2e
/test 414-test-upgrade
/test 417-aws-ovn-images
/test 417-azure-images
/test 417-gcp-images
/test 417-hypershift-images
/test 417-images
/test 417-operator-e2e
/test 417-osd-images
/test 417-single-node-images
/test 417-test-upgrade
/test 417-vsphere-images
/test 418-images
/test 418-operator-e2e
/test 418-test-upgrade
/test ocp4.17-lp-rosa-classic-images
/test ocp4.18-lp-interop-images

The following commands are available to trigger optional jobs:

/test 414-kitchensink-e2e
/test 414-kitchensink-upgrade
/test 414-mesh-e2e
/test 414-mesh-upgrade
/test 414-test-soak
/test 414-ui-e2e
/test 414-upstream-e2e
/test 414-upstream-e2e-kafka
/test 417-kitchensink-e2e
/test 417-kitchensink-upgrade
/test 417-mesh-e2e
/test 417-mesh-upgrade
/test 417-test-soak
/test 417-ui-e2e
/test 417-upstream-e2e
/test 417-upstream-e2e-kafka
/test 418-kitchensink-e2e
/test 418-kitchensink-upgrade
/test 418-mesh-e2e
/test 418-mesh-upgrade
/test 418-test-soak
/test 418-ui-e2e
/test 418-upstream-e2e
/test 418-upstream-e2e-kafka

Use /test all to run the following jobs that were automatically triggered:

pull-ci-openshift-knative-serverless-operator-main-417-aws-ovn-images
pull-ci-openshift-knative-serverless-operator-main-417-azure-images
pull-ci-openshift-knative-serverless-operator-main-417-gcp-images
pull-ci-openshift-knative-serverless-operator-main-417-hypershift-images
pull-ci-openshift-knative-serverless-operator-main-417-images
pull-ci-openshift-knative-serverless-operator-main-417-operator-e2e
pull-ci-openshift-knative-serverless-operator-main-417-osd-images
pull-ci-openshift-knative-serverless-operator-main-417-single-node-images
pull-ci-openshift-knative-serverless-operator-main-417-test-upgrade
pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e
pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e-kafka
pull-ci-openshift-knative-serverless-operator-main-417-vsphere-images
pull-ci-openshift-knative-serverless-operator-main-ocp4.17-lp-rosa-classic-images
pull-ci-openshift-knative-serverless-operator-main-ocp4.18-lp-interop-images

In response to this:

/test serverless-openshift-kn-operator-136-on-pull-request

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.

@pierDipi
Copy link
Member Author

/override "CSV component revision alignment check"

Copy link
Contributor

openshift-ci bot commented Mar 12, 2025

@pierDipi: Overrode contexts on behalf of pierDipi: CSV component revision alignment check

In response to this:

/override "CSV component revision alignment check"

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.

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 6d20f62 and 2 for PR HEAD 468e854 in total

@creydr
Copy link
Member

creydr commented Mar 13, 2025

/override "Konflux kflux-prd-rh02"

Copy link
Contributor

openshift-ci bot commented Mar 13, 2025

@creydr: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • Konflux kflux-prd-rh02

Only the following failed contexts/checkruns were expected:

  • Update deps
  • ci/prow/417-aws-ovn-images
  • ci/prow/417-azure-images
  • ci/prow/417-gcp-images
  • ci/prow/417-hypershift-images
  • ci/prow/417-images
  • ci/prow/417-operator-e2e
  • ci/prow/417-osd-images
  • ci/prow/417-single-node-images
  • ci/prow/417-test-upgrade
  • ci/prow/417-upstream-e2e
  • ci/prow/417-upstream-e2e-kafka
  • ci/prow/417-vsphere-images
  • ci/prow/ocp4.17-lp-rosa-classic-images
  • ci/prow/ocp4.18-lp-interop-images
  • pull-ci-openshift-knative-serverless-operator-main-417-aws-ovn-images
  • pull-ci-openshift-knative-serverless-operator-main-417-azure-images
  • pull-ci-openshift-knative-serverless-operator-main-417-gcp-images
  • pull-ci-openshift-knative-serverless-operator-main-417-hypershift-images
  • pull-ci-openshift-knative-serverless-operator-main-417-images
  • pull-ci-openshift-knative-serverless-operator-main-417-operator-e2e
  • pull-ci-openshift-knative-serverless-operator-main-417-osd-images
  • pull-ci-openshift-knative-serverless-operator-main-417-single-node-images
  • pull-ci-openshift-knative-serverless-operator-main-417-test-upgrade
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e-kafka
  • pull-ci-openshift-knative-serverless-operator-main-417-vsphere-images
  • pull-ci-openshift-knative-serverless-operator-main-ocp4.17-lp-rosa-classic-images
  • pull-ci-openshift-knative-serverless-operator-main-ocp4.18-lp-interop-images
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

In response to this:

/override "Konflux kflux-prd-rh02"

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.

@pierDipi pierDipi merged commit 4537ccf into openshift-knative:main Mar 13, 2025
25 of 28 checks passed
@pierDipi pierDipi deleted the inject-ei-images branch March 13, 2025 10:49
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.

4 participants