Skip to content

Conversation

@Srihari1192
Copy link

@Srihari1192 Srihari1192 commented May 22, 2025

What this PR does / why we need it:

This PR introduces a new end-to-end test case that validates the full flow of a real-time credit scoring demo using Feast operator .
The test ensures that the system can successfully:

  • Apply infrastructure manifests for Redis and Postgres and deploy the Feast FeatureStore custom resource (CR)
  • Verify feast apply and feast materialize-incremental are configured correctly in the CR's CronJob
  • Execute Feast CLI methods inside the online container to validate registered entities, features, data sources, feature views etc
  • Patch and validate the CronJob to run training and prediction via a Python script

Which issue(s) this PR fixes:

closes https://issues.redhat.com/browse/RHOAIENG-15683

  • Misc

@openshift-ci
Copy link

openshift-ci bot commented May 22, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link

openshift-ci bot commented May 22, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Srihari1192
Once this PR has been reviewed and has the lgtm label, please assign tmihalac for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@ntkathole
Copy link
Member

ntkathole commented May 22, 2025

@Srihari1192 These tests can be part of upstream repo

@Srihari1192
Copy link
Author

Srihari1192 commented May 22, 2025

@Srihari1192 These tests can be part of upstream repo

@ntkathole Yes i will cherry-pick this PR and #19 after merging to upstream

@Srihari1192 Srihari1192 marked this pull request as ready for review May 22, 2025 09:48
@Srihari1192 Srihari1192 requested a review from jyejare May 22, 2025 10:46
@openshift-ci
Copy link

openshift-ci bot commented May 22, 2025

@Srihari1192: 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/e2e-in-cluster-presubmit 63a34ce link true /test e2e-in-cluster-presubmit

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.

@Srihari1192
Copy link
Author

closing this PR , will open this changes in upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants