-
Notifications
You must be signed in to change notification settings - Fork 6
Closed
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationhelmk8s-reporterpost-mergetesting
Description
PR #528 adds support for custom pod labels to the k8s-reporter Helm chart cronjob. This feature enables pod selection for specific node types like EKS Fargate nodes. Once this PR is merged, we need to complete the following tasks to ensure this feature is properly tested, documented, and released.
Tasks to complete:
A) Testing Requirements Review & Implementation
- Review what additional tests are needed for the new pod labels functionality
- Consider test cases such as:
- No labels (empty/unset)
- Single label vs multiple labels
- YAML formatting and indentation validation
- Invalid label format handling
- Implement agreed-upon test cases
B) Documentation Updates
- Assess whether the following documentation updates are needed:
- Update k8s-reporter Helm chart documentation for the new
podLabelsoption - Add usage examples (especially for EKS Fargate use case)
- Update values.yaml with inline comments
- Check if main Kosli CLI documentation needs updates
- Update k8s-reporter Helm chart documentation for the new
C) Changelog Entry
- Add entry to CHANGELOG.md for the new custom pod labels feature
- Include mention of EKS Fargate compatibility improvement
Acceptance Criteria:
- Appropriate tests are added and passing
- Documentation is updated where needed
- Changelog entry is added
- Feature works as expected with various label configurations
Related to: #528
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
documentationImprovements or additions to documentationImprovements or additions to documentationhelmk8s-reporterpost-mergetesting