FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: Repair Chronon operator validation and Pixi CI setup by franciscojavierarceo · Pull Request #6947 · feast-dev/feast · GitHub

Repository navigation

fix: Repair Chronon operator validation and Pixi CI setup - #6947

Merged
franciscojavierarceo merged 3 commits into
feast-dev:masterfrom
franciscojavierarceo:codex/fix-chronon-operator-types
Oct 5, 2026
Merged

franciscojavierarceo merged 3 commits into
feast-dev:masterfrom
franciscojavierarceo:codex/fix-chronon-operator-types

Conversation

franciscojavierarceo commented Oct 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Chronon registration in #6188 added Python store types without adding them to the operator's persistence allowlists. A FeatureStore using type: chronon is rejected, and the Python/operator data-source consistency suite fails for both online and offline stores.

Add chronon to both persistence enums and validation lists in the v1 and v1alpha1 APIs. Regenerate the CRD, installer, and bundle schemas, and refresh the secret baseline's shifted line numbers.

Ray, DuckDB, and registration CI also stop before tests because the earlier GLIDE extra change left Feast's local-package metadata stale in pixi.lock. Synchronize the GLIDE requirement and CI-extra membership. No resolved package versions change.

Validation:

  • Reproduced two missing-chronon failures with make -C infra/feast-operator test-datasources; all three checks pass after the fix.
  • make -C infra/feast-operator build-installer and make -C infra/feast-operator bundle pass.
  • All 24 generated online/offline persistence enums include Chronon.
  • Scoped pre-commit checks and git diff --check pass.
  • Pixi 0.75.0 lock --dry-run --check confirms the updated lockfile is current.
  • Final-commit CI passes: Linux/macOS Python unit matrix, Go/UI unit tests, lint/security/smoke, local integration including operator data-source parity, Ray, DuckDB, registry REST, remote RBAC, and operator tests including end-to-end, previous-version, and upgrade coverage.
  • Both local and cloud registration jobs pass on the rerun that checks out the updated PR merge ref.

Signed-off-by: Francisco Javier Arceo <arceofrancisco@gmail.com>

codecov-commenter commented Oct 5, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

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

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6947   +/-   ##
=======================================
  Coverage   49.02%   49.02%           
=======================================
  Files         433      433           
  Lines       54308    54308           
  Branches     7910     7910           
=======================================
  Hits        26625    26625           
  Misses      25799    25799           
  Partials     1884     1884           
Flag Coverage Δ
go-feature-server 30.58% <ø> (ø)
python-unit 50.41% <ø> (ø)

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 dd059c6...fec855e. 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.

Signed-off-by: Francisco Javier Arceo <arceofrancisco@gmail.com>
Signed-off-by: Francisco Javier Arceo <arceofrancisco@gmail.com>
franciscojavierarceo changed the title fix: Allow Chronon persistence in the Feast operator fix: Repair Chronon operator validation and Pixi CI setup Oct 5, 2026
franciscojavierarceo merged commit 3710f78 into feast-dev:master Oct 5, 2026
27 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL