Skip to content
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.

Sign up for GitHub

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

Jump to bottom

JobSink kitchensink test and E2E #2984

Merged
merged 1 commit into from
Nov 28, 2024
Merged

JobSink kitchensink test and E2E #2984

merged 1 commit into from
Nov 28, 2024

Conversation

mgencur
Copy link
Contributor

@mgencur mgencur commented Nov 4, 2024 •
edited
Loading

Related to https://issues.redhat.com/browse/SRVKE-1614

  • Tests JobSink in combination with all resources such as Broker, Channel, KafkaSource, ContainerSource, etc.
  • Adds E2E test TestJobSinkSuccess which tests JobSink in combination with Knative Service. It will be also used for testing in disconnected environment. JobSink brings a new controller (and image). The test suite for s-o covers most images in disconnected env.

Proposed Changes

@openshift-ci openshift-ci bot requested review from creydr and rhuss November 4, 2024 09:40
@openshift-ci openshift-ci bot added the approved label Nov 4, 2024
@mgencur mgencur changed the title JobSink kitchensink test JobSink kitchensink test and E2E Nov 5, 2024
@mgencur mgencur force-pushed the jobsink_kitchensink_test branch from fae7a53 to 294097d Compare November 5, 2024 10:02
@mgencur
Copy link
Contributor Author

mgencur commented Nov 5, 2024

/unhold
All dependent PRs merged.

@mgencur
Copy link
Contributor Author

mgencur commented Nov 6, 2024

/retest

@mgencur
Copy link
Contributor Author

mgencur commented Nov 6, 2024

/test ?

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 6, 2024

@mgencur: The following commands are available to trigger required jobs:

  • /test 413-images
  • /test 413-operator-e2e-aws-413
  • /test 413-test-upgrade-aws-413
  • /test 416-aws-ovn-images
  • /test 416-azure-images
  • /test 416-gcp-images
  • /test 416-hypershift-images
  • /test 416-images
  • /test 416-operator-e2e-aws-416
  • /test 416-osd-images
  • /test 416-single-node-images
  • /test 416-test-upgrade-aws-416
  • /test 416-vsphere-images
  • /test 417-images
  • /test 417-operator-e2e-aws-417
  • /test 417-test-upgrade-aws-417
  • /test ocp4.14-lp-interop-images
  • /test ocp4.14-lp-rosa-classic-images
  • /test ocp4.15-lp-interop-images
  • /test ocp4.15-lp-rosa-classic-images
  • /test ocp4.16-lp-interop-images
  • /test ocp4.17-lp-interop-images
  • /test serverless-p2p-rosa-classic-images

The following commands are available to trigger optional jobs:

  • /test 413-kitchensink-e2e-aws-413
  • /test 413-kitchensink-upgrade-aws-413
  • /test 413-mesh-e2e-aws-413
  • /test 413-mesh-upgrade-aws-413
  • /test 413-test-soak-aws-413
  • /test 413-ui-e2e-aws-413
  • /test 413-upstream-e2e-aws-413
  • /test 413-upstream-e2e-kafka-aws-413
  • /test 416-kitchensink-e2e-aws-416
  • /test 416-kitchensink-upgrade-aws-416
  • /test 416-mesh-e2e-aws-416
  • /test 416-mesh-upgrade-aws-416
  • /test 416-test-soak-aws-416
  • /test 416-ui-e2e-aws-416
  • /test 416-upstream-e2e-aws-416
  • /test 416-upstream-e2e-kafka-aws-416
  • /test 417-kitchensink-e2e-aws-417
  • /test 417-kitchensink-upgrade-aws-417
  • /test 417-mesh-e2e-aws-417
  • /test 417-mesh-upgrade-aws-417
  • /test 417-test-soak-aws-417
  • /test 417-ui-e2e-aws-417
  • /test 417-upstream-e2e-aws-417
  • /test 417-upstream-e2e-kafka-aws-417

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

  • pull-ci-openshift-knative-serverless-operator-main-416-images
  • pull-ci-openshift-knative-serverless-operator-main-416-operator-e2e-aws-416
  • pull-ci-openshift-knative-serverless-operator-main-416-test-upgrade-aws-416
  • pull-ci-openshift-knative-serverless-operator-main-416-upstream-e2e-aws-416
  • pull-ci-openshift-knative-serverless-operator-main-416-upstream-e2e-kafka-aws-416

In response to this:

/test ?

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.

@mgencur mgencur force-pushed the jobsink_kitchensink_test branch from db240da to 3465235 Compare November 27, 2024 12:32
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 27, 2024 •
edited
Loading

@mgencur: The following tests 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/416-operator-e2e-aws-416 db240da link true /test 416-operator-e2e-aws-416
ci/prow/416-upstream-e2e-kafka-aws-416 db240da link false /test 416-upstream-e2e-kafka-aws-416
ci/prow/416-test-upgrade-aws-416 db240da link true /test 416-test-upgrade-aws-416
ci/prow/417-upstream-e2e-aws-417 3465235 link false /test 417-upstream-e2e-aws-417
ci/prow/417-upstream-e2e-kafka-aws-417 3465235 link false /test 417-upstream-e2e-kafka-aws-417

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.

@mgencur
Copy link
Contributor Author

mgencur commented Nov 28, 2024

/retest-required

@maschmid
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Nov 28, 2024
@mgencur
Copy link
Contributor Author

mgencur commented Nov 28, 2024

/cherry-pick release-1.35

@openshift-cherrypick-robot
Copy link
Contributor

@mgencur: once the present PR merges, I will cherry-pick it on top of release-1.35 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.35

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.

@mgencur
Copy link
Contributor Author

mgencur commented Nov 28, 2024

/override ?

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 28, 2024

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

  • ?

Only the following failed contexts/checkruns were expected:

  • Red Hat Konflux / serverless-operator-135-enterprise-contract / pr group jobsink_kitchensink_test
  • Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-ingress-135
  • Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-kn-operator-135
  • Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-metadata-webhook-135
  • Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-must-gather-135
  • Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-openshift-kn-operator-135
  • Update deps
  • ci/prow/417-images
  • ci/prow/417-operator-e2e-aws-417
  • ci/prow/417-test-upgrade-aws-417
  • ci/prow/417-upstream-e2e-aws-417
  • ci/prow/417-upstream-e2e-kafka-aws-417
  • code/snyk (Hybrid Platforms - OpenShift Serverless)
  • license/snyk (Hybrid Platforms - OpenShift Serverless)
  • pull-ci-openshift-knative-serverless-operator-main-417-images
  • pull-ci-openshift-knative-serverless-operator-main-417-operator-e2e-aws-417
  • pull-ci-openshift-knative-serverless-operator-main-417-test-upgrade-aws-417
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e-aws-417
  • pull-ci-openshift-knative-serverless-operator-main-417-upstream-e2e-kafka-aws-417
  • security/snyk (Hybrid Platforms - OpenShift Serverless)
  • 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 ?

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.

@mgencur
Copy link
Contributor Author

mgencur commented Nov 28, 2024

/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / pr group jobsink_kitchensink_test"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-ingress-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-kn-operator-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-metadata-webhook-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-must-gather-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-openshift-kn-operator-135"

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 28, 2024

@mgencur: Overrode contexts on behalf of mgencur: Red Hat Konflux / serverless-operator-135-enterprise-contract / pr group jobsink_kitchensink_test, Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-ingress-135, Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-kn-operator-135, Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-metadata-webhook-135, Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-must-gather-135, Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-openshift-kn-operator-135

In response to this:

/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / pr group jobsink_kitchensink_test"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-ingress-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-kn-operator-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-metadata-webhook-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-must-gather-135"
/override "Red Hat Konflux / serverless-operator-135-enterprise-contract / serverless-openshift-kn-operator-135"

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

/lgtm

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 28, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maschmid, mgencur, 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

@openshift-merge-bot openshift-merge-bot bot merged commit 295622c into openshift-knative:main Nov 28, 2024
22 of 30 checks passed
@openshift-cherrypick-robot
Copy link
Contributor

@mgencur: new pull request created: #3063

In response to this:

/cherry-pick release-1.35

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-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Nov 28, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@creydr creydr Awaiting requested review from creydr

@rhuss rhuss Awaiting requested review from rhuss

Assignees

@maschmid maschmid

@pierDipi pierDipi

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mgencur @maschmid @openshift-cherrypick-robot @pierDipi @openshift-merge-robot