Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add remove command #51

Merged
merged 2 commits into from
Feb 29, 2024
Merged

Add remove command #51

merged 2 commits into from
Feb 29, 2024

Conversation

amisevsk
Copy link
Contributor

Description

Add kit remove. The modelkit to be removed can either be specified via tag or by digest. If a digest is used, that modelkit is removed from storage (bringing any relevant tags with it). If specified via tag, the model is either untagged or deleted, depending on whether other tags refer to the model. Flag -f|--force can be used to force removal via tag even if other tags point at the modelkit.

Remove a modelkit from local storage.

Description:
  Removes a modelkit from storage on the local disk.

  The model to be removed may be specifed either by a tag or by a digest. If
  specified by digest, that modelkit will be removed along with any tags that
  might refer to it. If specified by tag (and the --force flag is not used),
  the modelkit will only be removed if no other tags refer to it; otherwise
  it is only untagged.

Usage:
  kit remove [flags] <reference>

Examples:
  kit remove my-registry.com/my-org/my-repo:my-tag
  kit remove my-registry.com/my-org/my-repo@sha256:<digest>
  kit remove my-registry.com/my-org/my-repo:tag1,tag2,tag3

Flags:
  -f, --force   remove manifest even if other tags refer to it
  -h, --help    help for remove

Global Flags:
      --config string   Config file (default $HOME/.kitops)
  -v, --verbose         Include additional information in output (default false)

Add 'kit remove' to allow deleting models from local storage.
Add flag --force to the remove subcommand. If specified, it will result
in modelkits being deleted even if other tags refer to them.
@amisevsk amisevsk requested a review from gorkem February 28, 2024 19:39
Copy link
Contributor

@gorkem gorkem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@amisevsk amisevsk merged commit 4f21b69 into main Feb 29, 2024
1 check passed
@amisevsk amisevsk deleted the remove branch February 29, 2024 16:06
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