mirror of
https://github.com/helmfile/helmfile.git
synced 2026-10-01 01:43:04 +02:00
* feat: add kubedog-based resource tracking integration Add kubedog tracking as an alternative to Helm's --wait flag with: - Real-time deployment progress tracking - Container log streaming - Fine-grained resource filtering (trackKinds/skipKinds/trackResources) Features: - New pkg/resource package for unified manifest parsing and filtering - New pkg/kubedog package wrapping kubedog library - CLI flags: --track-mode, --track-timeout, --track-logs - Helmfile YAML support for trackMode, trackTimeout, trackLogs, trackKinds, skipKinds, trackResources - Case-insensitive kind matching for filtering - Multi-context support with proper kubeconfig/kubeContext handling Tracking supports: Deployment, StatefulSet, DaemonSet, Job Resource filtering priority (highest to lowest): 1. trackResources - explicit resource whitelist 2. skipKinds - blacklist specific kinds 3. trackKinds - whitelist specific kinds Integration: - Disable Helm --wait when using kubedog tracking - Track after successful Helm sync/apply - Respect release.Namespace as fallback for resources without namespace - Use getKubeContext() for correct cluster targeting Tests: - Unit tests for resource filtering and kubedog options - Integration test with httpbin chart - E2E snapshot tests for YAML serialization - Documentation in docs/advanced-features.md Signed-off-by: yxxhero <aiopsclub@163.com> * fix: address PR #2383 review comments (round 4) 1. resource/filter.go: Skip empty whitelist entries in matchWhitelist - At least one field (kind/name/namespace) must be specified - Prevents matching all resources with empty TrackResources entries 2. config/apply.go: Add ValidateConfig for track-mode validation - Validate --track-mode must be 'helm' or 'kubedog' - Reject invalid values like --track-mode foo 3. config/sync.go: Add ValidateConfig for track-mode validation - Same validation as apply command - Ensures consistent behavior across commands Signed-off-by: yxxhero <aiopsclub@163.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com>
63 lines
1.1 KiB
Go
63 lines
1.1 KiB
Go
package resource
|
|
|
|
import (
|
|
"bytes"
|
|
"errors"
|
|
"fmt"
|
|
"io"
|
|
|
|
"go.uber.org/zap"
|
|
"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
|
|
k8syaml "k8s.io/apimachinery/pkg/util/yaml"
|
|
)
|
|
|
|
func ParseManifest(manifest []byte, defaultNamespace string, logger *zap.SugaredLogger) ([]Resource, error) {
|
|
var resources []Resource
|
|
|
|
decoder := k8syaml.NewYAMLOrJSONDecoder(bytes.NewReader(manifest), 4096)
|
|
|
|
for {
|
|
var obj unstructured.Unstructured
|
|
err := decoder.Decode(&obj)
|
|
if err != nil {
|
|
if errors.Is(err, io.EOF) {
|
|
break
|
|
}
|
|
return nil, fmt.Errorf("failed to decode manifest: %w", err)
|
|
}
|
|
|
|
if len(obj.Object) == 0 {
|
|
continue
|
|
}
|
|
|
|
kind := obj.GetKind()
|
|
if kind == "" {
|
|
if logger != nil {
|
|
logger.Debugf("Skipping resource without kind")
|
|
}
|
|
continue
|
|
}
|
|
|
|
name := obj.GetName()
|
|
if name == "" {
|
|
if logger != nil {
|
|
logger.Debugf("Skipping %s resource without name", kind)
|
|
}
|
|
continue
|
|
}
|
|
|
|
namespace := obj.GetNamespace()
|
|
if namespace == "" {
|
|
namespace = defaultNamespace
|
|
}
|
|
|
|
resources = append(resources, Resource{
|
|
Kind: kind,
|
|
Name: name,
|
|
Namespace: namespace,
|
|
})
|
|
}
|
|
|
|
return resources, nil
|
|
}
|