Skip to content

Conversation

@timmilesdw
Copy link
Contributor

Added d8 system package scan <repository-name> command that creates a PackageRepositoryOperation CR to trigger a full scan.

Usage:

  • d8 system package scan my-repo
  • d8 system package scan my-repo --timeout 10m
  • d8 system package scan my-repo --name custom-name
  • d8 system package scan my-repo --dry-run

Features:

  • Validates PackageRepository exists before creating operation
  • Shell tab-completion for repository names
  • --dry-run to preview without creating
  • Auto-generated name with manual marker and nanosecond timestamp
  • deckhouse.io/created-by: deckhouse-cli annotation
  • Unit tests included

@timmilesdw timmilesdw requested a review from LazarenkoA January 30, 2026 15:08
@timmilesdw timmilesdw marked this pull request as ready for review January 30, 2026 15:08
@timmilesdw timmilesdw requested a review from ldmonster as a code owner January 30, 2026 15:08
@ldmonster ldmonster requested a review from ipaqsa February 2, 2026 14:22
@timmilesdw timmilesdw force-pushed the feat/packagerepository-scan-task branch from 9793f5f to 84ce805 Compare February 3, 2026 08:21
Signed-off-by: Timur Tuktamyshev <timur.tuktamyshev@flant.com>
@timmilesdw timmilesdw force-pushed the feat/packagerepository-scan-task branch from 84ce805 to 3ffbd66 Compare February 3, 2026 11:28
return nil, cobra.ShellCompDirectiveError
}

var names []string

Choose a reason for hiding this comment

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

can initialize the slice size

return nil, cobra.ShellCompDirectiveError
}

ctx, cancel := context.WithTimeout(context.Background(), constants.DefaultAPITimeout)

Choose a reason for hiding this comment

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

maybe context.WithTimeout(cmd.Context(), constants.DefaultAPITimeout) ?

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.

4 participants