Skip to content

test: Add cron job check for e2e test runTestDeploySimpleCRFunc - #6705

Open
Srihari1192 wants to merge 1 commit into
feast-dev:masterfrom
Srihari1192:addCronJobTestCheck
Open

test: Add cron job check for e2e test runTestDeploySimpleCRFunc#6705
Srihari1192 wants to merge 1 commit into
feast-dev:masterfrom
Srihari1192:addCronJobTestCheck

Conversation

@Srihari1192

@Srihari1192 Srihari1192 commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

  • Add checkIfCronJobExists helper in feast-operator e2e utils
  • Validate CronJob feast-<featureStoreName> exists in validateTheFeatureStoreCustomResource before FeatureStore Ready checks
  • Covers runTestDeploySimpleCRFunc and the remote-registry deploy path that share this validation

Which issue(s) this PR fixes:

Checks

  • I've made sure the tests are passing.
  • My commits are signed off (git commit -s)
  • My PR title follows conventional commits format

Testing Strategy

  • Unit tests
  • Integration tests
  • Manual tests
  • Testing is not required for this change

Misc

@Srihari1192
Srihari1192 force-pushed the addCronJobTestCheck branch from 9b2714d to d9b9caa Compare August 5, 2026 09:12
@codecov-commenter

codecov-commenter commented Aug 5, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.77%. Comparing base (e8edab5) to head (b54c7ba).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6705   +/-   ##
=======================================
  Coverage   46.77%   46.77%           
=======================================
  Files         414      414           
  Lines       50191    50191           
  Branches     7181     7181           
=======================================
  Hits        23475    23475           
  Misses      25077    25077           
  Partials     1639     1639
Flag Coverage Δ
go-feature-server 30.58% <ø> (ø)
python-unit 48.10% <ø> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e8edab5...b54c7ba. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Srihari1192 Srihari1192 changed the title test:Add cron job check for e2e test runTestDeploySimpleCRFunc test: Add cron job check for e2e test runTestDeploySimpleCRFunc Aug 5, 2026
@Srihari1192
Srihari1192 marked this pull request as ready for review August 5, 2026 10:17
@Srihari1192
Srihari1192 requested a review from a team as a code owner August 5, 2026 10:17
ntkathole
ntkathole reviewed Aug 5, 2026
View reviewed changes
Comment thread infra/feast-operator/test/utils/test_util.go
@Srihari1192
Srihari1192 requested a review from ntkathole August 6, 2026 05:27
@Srihari1192
test:Add cron job check for e2e test runTestDeploySimpleCRFunc
b54c7ba 
Signed-off-by: Srihari <svenkata@redhat.com>
@Srihari1192
Srihari1192 force-pushed the addCronJobTestCheck branch from d9b9caa to b54c7ba Compare August 6, 2026 07:33
This was referenced Aug 6, 2026
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ntkathole ntkathole ntkathole approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Srihari1192 @codecov-commenter @ntkathole