Skip to content

fix: remove catalog delete-by-type command - #247

Open
jeff-schnitter wants to merge 1 commit into
mainfrom
remove-delete-by-type
Open

jeff-schnitter wants to merge 1 commit into
mainfrom
remove-delete-by-type

Conversation

@jeff-schnitter

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the catalog delete-by-type CLI command, which allowed bulk-deleting all entities of a given type
  • Removes the corresponding test file and test data fixtures
  • Motivated by a customer incident where this command was used accidentally to delete all their entities

Test plan

  • Verify cortex catalog delete-by-type is no longer available as a subcommand
  • Run just test-all to confirm no regressions

Notes

The test_perf_rate_limiting.py test calls the underlying API directly (not via the CLI command) for cleanup — that is intentional internal test infrastructure and is unaffected by this change.

The next version will be 1.46.2 (patch bump from the fix: commit prefix).

🤖 Generated with Claude Code

… entity deletion

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant