Skip to content

feat(cli): Add feast projects delete command (closes #5095)#6318

Merged
ntkathole merged 5 commits into
feast-dev:masterfrom
mailtoboggavarapu-coder:feat/expose-project-delete-cli-v2
May 6, 2026
Merged

feat(cli): Add feast projects delete command (closes #5095)#6318
ntkathole merged 5 commits into
feast-dev:masterfrom
mailtoboggavarapu-coder:feat/expose-project-delete-cli-v2

Conversation

@mailtoboggavarapu-coder

@mailtoboggavarapu-coder mailtoboggavarapu-coder commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #5095

Exposes project deletion through the CLI by adding a new feast projects delete <name> subcommand to the existing projects group.

What's changed

Added project_delete command to sdk/python/feast/cli/projects.py:

Usage: feast projects delete [OPTIONS] NAME

  Delete a project and all its resources from the registry.

Options:
  -y, --yes  Skip confirmation prompt and delete immediately.
  --help     Show this message and exit.

Design notes

  • Uses store.registry.delete_project(name, commit=True) via the lazy-init registry property (the abstract method exists in BaseRegistry and is implemented in all concrete registries)
  • Adds an interactive confirmation prompt with click.confirm — can be skipped with --yes / -y for scripting
  • Catches both FeastObjectNotFoundException and ProjectNotFoundException so the CLI exits cleanly (exit code 1) for either variant raised by concrete registry implementations
  • Follows existing CLI patterns in projects.py

Example

# Interactive (prompts for confirmation)
feast projects delete my-project

# Non-interactive
feast projects delete my-project --yes

Checklist

  • Added delete command to projects_cmd group
  • Confirmation prompt with --yes bypass flag
  • Error handling for non-existent projects
  • Follows existing CLI patterns in projects.py

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

@mailtoboggavarapu-coder

Copy link
Copy Markdown
Contributor Author

Thanks for the detailed review, @devin-ai-integration! Fixed both issues in commit 21f7f6a:

  1. project_listutils.py_object_to_proto + broken tag parsing: Updated to match master exactly: uses utils.tags_list_to_dict(tags) for tag filtering and prints tabulated output via tabulate.
  2. project_current — simplified incorrectly: Fixed to match master (calls store.get_project(name=None) with try/except and YAML-formatted output).

The only new code is the project_delete command itself.

@mailtoboggavarapu-coder

Copy link
Copy Markdown
Contributor Author

Hi team! Friendly ping on this PR — all 18 CI checks are passing and there are no conflicts with the base branch.

This adds a feast projects delete command (closes #5095) with an interactive confirmation prompt and a --yes / -y flag for scripting. Happy to iterate on any feedback!

@mailtoboggavarapu-coder

Copy link
Copy Markdown
Contributor Author

Hi team! 👋 Friendly ping on feast #6318 — this adds feast projects delete <name> to the CLI (closes #5095).

Status as of today:

  • ✅ DCO passing
  • ✅ Devin Review completed
  • ✅ Branch synced with master
  • ⏳ 11 workflows awaiting maintainer CI approval (fork policy)

The only blocker is maintainer CI approval + code review. Happy to address any feedback!

@ntkathole ntkathole force-pushed the feat/expose-project-delete-cli-v2 branch from cfe215b to 9f45a4b Compare May 1, 2026 04:44
@ntkathole

Copy link
Copy Markdown
Member

@mailtoboggavarapu-coder I think it's better to include delete_project method in sdk/python/feast/feature_store.py

See #6212

Let's add it to feature_store.py and remove context passing in cli

@mailtoboggavarapu-coder

Copy link
Copy Markdown
Contributor Author

Hi @ntkathole 👋 — all unit tests, smoke tests, and most integration tests are now passing after the indentation fix. Two checks are failing but neither is caused by this PR's changes:

  1. linterdetect-secrets is flagging infra/feast-operator/bundle/manifests/openlineage-secret_v1_secret.yaml, a file this PR does not touch (pre-existing repo issue).

  2. pr-integration-tests — exited with code 2 (pytest "interrupted") after 16+ minutes, indicating a cloud infrastructure/timeout flake rather than a test assertion failure. All other integration test workflows (DuckDB, Ray, remote-rbac, registry REST API, local) passed.

Could you please re-run the pr-integration-tests job? It appears to be a transient infrastructure failure unrelated to this change. Thank you! 🙏

Exposes project deletion via the CLI. The new `feast projects delete <name>`
command calls store.registry.delete_project() which is already implemented
in all concrete registries.

- Adds interactive confirmation prompt with --yes/-y flag for scripting
- Catches both FeastObjectNotFoundException and ProjectNotFoundException
- Exits with code 1 for non-existent projects

Signed-off-by: Venkateswarlu Boggavarapu <mailtoboggavarapu@gmail.com>
…tation

Addresses Devin Review findings:
- Fix project_list: use utils.tags_list_to_dict() instead of manual tag.split(), and print tabulated output matching master
- Fix project_current: call store.get_project(name=None) with error handling instead of store.project

Both functions now match master branch exactly. Only project_delete is new code from this PR.

Signed-off-by: Venkateswarlu Boggavarapu <mailtoboggavarapu@gmail.com>
Refactor project listing and deletion commands in CLI.
Added a method to delete a project from the registry.
@ntkathole ntkathole force-pushed the feat/expose-project-delete-cli-v2 branch from 2babaa0 to 5d6419a Compare May 6, 2026 04:04
@ntkathole ntkathole merged commit 1a4b96c into feast-dev:master May 6, 2026
25 of 28 checks passed
franciscojavierarceo pushed a commit that referenced this pull request Jun 13, 2026
# [0.64.0](v0.63.0...v0.64.0) (2026-06-13)

### Bug Fixes

* Add async_supported property to RedisOnlineStore ([9b088fe](9b088fe))
* Add missing feast init templates to operator CRD and enhance persistence documentation ([1941d4d](1941d4d))
* Allow to publish from reference branch ([5458ec8](5458ec8))
* API calls list ([4203eb7](4203eb7))
* **bigquery:** Enable list inference for parquet loads in offline_write_batch ([9243497](9243497)), closes [#5845](#5845)
* Bump grpcio dependencies ([07b4782](07b4782))
* **compute-engine/local:** Honor field_mapping on join keys in dedup + join nodes ([#6395](#6395)) ([bd01824](bd01824))
* **dynamodb:** Avoid tag race condition by using diff-based tag updates ([#6479](#6479)) ([bad2b7d](bad2b7d)), closes [#6418](#6418)
* **dynamodb:** Fix mypy type for _build_projection_expression return ([217b4da](217b4da))
* Fix intermittent async test failures for DynamoDB and Redis ([63c5eb1](63c5eb1))
* Fix mongodb blog title ([57d28d4](57d28d4))
* Fix shared SQL registry crash - avoid unnecessary UDF deserialization in proto cache building ([ac588d7](ac588d7))
* Fix SparkRetrievalJob.persist() failing for SparkSource ([209d7cd](209d7cd))
* Fixed formatting and image for mongo blog ([#6377](#6377)) ([f8389fb](f8389fb))
* Fixes for ray source ([7f592a4](7f592a4))
* **go:** skip registry refresh when cache_ttl_seconds <= 0 ([97ed40c](97ed40c))
* Handle array of strings columns in Athena materialization ([#6324](#6324)) ([4ed0278](4ed0278))
* make milvus VARCHAR max_length configurable, remove hardcoded 512 limit ([3b98c22](3b98c22))
* **operator:** Set appProtocol: grpc on registry gRPC Service ([#6367](#6367)) ([c9ae2b4](c9ae2b4))
* PyJWT 2.10+ added validation that rejects empty HMAC keys ([e756ffe](e756ffe))
* RemoteOnlineStore sends all features in a single HTTP request ([8f187dd](8f187dd))
* Remove registry proto dump to enforce RBAC and add permission checks to Commit/Refresh RPCs ([328431f](328431f))
* Remove selector migration job - no longer needed ([51c325e](51c325e))
* replace broken .claude skill symlink with correct relative path ([4541690](4541690))
* Replace selector label strip patch with migration Job for upgrade-safe selector uniqueness ([00dea50](00dea50))
* Scope feature view name conflict check to current project in file-based registry ([#6369](#6369)) ([a4fde83](a4fde83)), closes [#6209](#6209)
* **snowflake:** Stop double-quoting connection identifiers ([#6462](#6462)) ([e914d59](e914d59))
* **spark:** S3/GCS PyArrow filesystem resolution for staging paths ([#6442](#6442)) ([ae50414](ae50414))
* **trino:** Clean up temporary entity tables after retrieval ([#6381](#6381)) ([d86b13d](d86b13d)), closes [#6306](#6306)
* Update go-feature-server base image to Go 1.25 and fix operator Dockerfile COPY permissions ([86ef0bc](86ef0bc))

### Features

* [Backend] Data Quality Monitoring with native compute, multi-backend support, REST API, CLI ([#6202](#6202)) ([5458c37](5458c37))
* Add apache flink compute engine ([#6476](#6476)) ([9636d6a](9636d6a))
* Add demo noteboooks for users ([e362173](e362173))
* Add enabled/disabled toggle for feature views ([#6401](#6401)) ([5f1fa0d](5f1fa0d)), closes [#6395](#6395)
* Add Label View to init template ([ec272d5](ec272d5))
* Add mTLS support to remote registry gRPC client ([#6474](#6474)) ([c9602d8](c9602d8))
* Add Prometheus gauges for FeatureStore installation telemetry ([#6354](#6354)) ([1b681b7](1b681b7))
* Adds registry REST API endpoints for managing entities, data sources, and feature views ([#6413](#6413)) ([f77bd1d](f77bd1d))
* Allow CRUD on entities, data sources, and feature views from UI ([#6412](#6412)) ([2321c07](2321c07))
* Allow default openlineage configuration ([#6467](#6467)) ([276b6df](276b6df))
* **bigquery:** Support DATE-type event timestamp columns ([#6362](#6362)) ([753dee5](753dee5)), closes [#2530](#2530)
* **cli:** Add `feast projects delete` command (closes [#5095](#5095)) ([#6318](#6318)) ([1a4b96c](1a4b96c))
* Data Quality Monitoring added in feast UI ([#6422](#6422)) ([fa271be](fa271be))
* **dynamodb:** Use ProjectionExpression when requested_features is set ([0adc906](0adc906)), closes [#6058](#6058)
* Enhance DataSource and FeatureView modals with error handling and submission states ([96d7169](96d7169))
* Expose registry endpoints on feature server for MCP access ([f77981c](f77981c))
* Feast First-Class LabelView Implementation ([#6292](#6292)) ([c0e7e5d](c0e7e5d))
* Feast-MLflow Integration ([#6235](#6235)) ([7279c75](7279c75))
* Operational metrics for offline store and SOX metrics for both ([#6340](#6340)) ([65b1b80](65b1b80))
* Pre-compute feature service ([8011550](8011550))
* REST API-backed UI for RBAC compatibility and per-page lazy loading ([#6414](#6414)) ([6ae80af](6ae80af))
* Support non-string map key types ([#6382](#6382)) ([#6383](#6383)) ([728aa2e](728aa2e))
* Update FeatureStore CRD with DRA Fields ([01241e4](01241e4))

### Performance Improvements

* Cache feature view resolution in get_online_features to reduce per-request overhead ([55c2f18](55c2f18))
* Optimize feature serving latency with batched async Redis, cached checks fix ([103809a](103809a))
* Replace MessageToDict with optimized custom dict builder ([#6015](#6015)) ([9902064](9902064))
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.

Expose project deletion to CLI

2 participants