Skip to content

Conversation

ajanikow
Copy link
Collaborator

No description provided.

@cla-bot cla-bot bot added the cla-signed label Aug 14, 2025
@ajanikow ajanikow requested a review from Copilot August 14, 2025 14:35
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new feature that allows filtering Helm charts by tag annotations. The implementation introduces a GetByTag method to the chart manager repository interface that can retrieve chart versions based on tag values stored in chart annotations.

  • Adds a new GetByTag method to the ChartManagerRepo interface for tag-based chart filtering
  • Implements the method to filter charts by the HelmLabelTag annotation and return the most recent matching version
  • Includes test coverage for the new functionality

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
pkg/util/constants/helm.go Adds the HelmLabelTag constant for chart tag annotations
pkg/util/k8sutil/helm/chart_manager_repo.go Implements the GetByTag method with filtering and sorting logic
pkg/util/k8sutil/helm/chart_manager_test.go Adds test case for the new tag-based chart filtering functionality
CHANGELOG.md Documents the new Chart By Tag filter feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ajanikow ajanikow merged commit 565e4e5 into master Aug 15, 2025
3 checks passed
@ajanikow ajanikow deleted the feature/chart_by_tag branch August 15, 2025 14:05
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