Skip to content

Fix: consolidate multiple kserve test workflows into a single job#3254

Merged
google-oss-prow[bot] merged 14 commits into
kubeflow:masterfrom
madmecodes:refactor-kserve-test-workflow
Nov 17, 2025
Merged

Fix: consolidate multiple kserve test workflows into a single job#3254
google-oss-prow[bot] merged 14 commits into
kubeflow:masterfrom
madmecodes:refactor-kserve-test-workflow

Conversation

@madmecodes

Copy link
Copy Markdown
Contributor

Pull Request Template for Kubeflow Manifests

✏️ Summary of Changes

Fixes the improper merge of 3 KServe test workflows by consolidating them into a single job that installs the stack once and runs all tests sequentially.

📦 Dependencies

none

🐛 Related Issues

none

✅ Contributor Checklist

  • I have tested these changes with kustomize. See Installation Prerequisites.
  • All commits are signed-off to satisfy the DCO check.
  • I have considered adding my company to the adopters page to support Kubeflow and help the community, since I expect help from the community for my issue (see 1. and 2.).

You can join the CNCF Slack and access our meetings at the Kubeflow Community website. Our channel on the CNCF Slack is here #kubeflow-platform.

Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
@madmecodes

Copy link
Copy Markdown
Contributor Author

shall i also merge all the 3 tests file:

  • kserve_test.sh
  • knative_authentication_test.sh
  • kserve_jwt_authentication_test.sh

@juliusvonkohout

Copy link
Copy Markdown
Member

shall i also merge all the 3 tests file:

* kserve_test.sh

* knative_authentication_test.sh

* kserve_jwt_authentication_test.sh

Yes please, simple and minimalistic.

Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
@google-oss-prow google-oss-prow Bot added size/XL and removed size/L labels Oct 10, 2025
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Comment thread tests/kserve_test.sh Outdated
@juliusvonkohout juliusvonkohout changed the title Fix: consolidate kserve test workflows into job Fix: consolidate multiple kserve test workflows into a single job Oct 10, 2025
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
@google-oss-prow google-oss-prow Bot added size/L and removed size/XL labels Oct 10, 2025
Comment thread tests/kserve_test.sh Outdated
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
@google-oss-prow google-oss-prow Bot added size/XL and removed size/L labels Oct 19, 2025
@juliusvonkohout

Copy link
Copy Markdown
Member

maybe @kunal-511 can help here as well and review.

Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
Comment thread tests/kserve_test.sh Outdated
Comment thread tests/kserve_test.sh Outdated
Comment thread tests/kserve_test.sh
Comment thread tests/kserve_test.sh Outdated
Comment thread tests/kserve_test.sh
Comment thread tests/kserve_test.sh Outdated
Comment thread tests/kserve_test.sh Outdated
…ture

Signed-off-by: madmecodes <ayushguptadev1@gmail.com>
@juliusvonkohout

Copy link
Copy Markdown
Member

Thank you very much

/lgtm
/approve

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: juliusvonkohout

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

The pull request process is described here

Details 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

@google-oss-prow google-oss-prow Bot merged commit 2342bcc into kubeflow:master Nov 17, 2025
9 checks passed
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.

2 participants