| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Add support for --take-ownership parameter (#742) * Add support for --take-ownership parameter Fix #731 Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> * Fix tests Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> * Add tests Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> * fix lint Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> * Add test for util Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> --------- Signed-off-by: Guillaume Perrin <guillaume28.perrin@gmail.com> | 1 年前 | |
Helm v4 support (#872) * Update packages for Helm v4 | 9 个月前 | |
fix normalieze manifests issue (#585) * fix normalieze manifests issue Signed-off-by: yxxhero <aiopsclub@163.com> * refactor: unexport normalize helper and harden tests - Rename ContentNormalizeManifests to unexported normalizeContent (avoids widening the public API; name avoids clash with the normalizeManifests bool parameter) - Move the test into an internal package (manifest) to access the unexported helper - Add an error-path case (sequence cannot unmarshal into map) - Use require.NoError/require.Error and direct []byte comparison Signed-off-by: yxxhero <aiopsclub@163.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com> | 2 个月前 | |
fix normalieze manifests issue (#585) * fix normalieze manifests issue Signed-off-by: yxxhero <aiopsclub@163.com> * refactor: unexport normalize helper and harden tests - Rename ContentNormalizeManifests to unexported normalizeContent (avoids widening the public API; name avoids clash with the normalizeManifests bool parameter) - Move the test into an internal package (manifest) to access the unexported helper - Add an error-path case (sequence cannot unmarshal into map) - Use require.NoError/require.Error and direct []byte comparison Signed-off-by: yxxhero <aiopsclub@163.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com> | 2 个月前 | |
fix: reduce memory consumption by eliminating redundant string copies (#965) * fix: reduce memory consumption by eliminating redundant string copies Changes manifest.Parse and parseContent to accept []byte instead of string, avoiding a full copy of the entire manifest when converting from the raw helm output bytes. Raw byte slices are released after parsing to allow GC reclamation before diff computation begins. Also adds a content length ratio check in rename detection to skip obviously different resources before computing expensive full diffs. Closes #915 Signed-off-by: yxxhero <aiopsclub@163.com> * fix: remove ineffectual nil assignments flagged by linter The releaseManifest and installManifest nil assignments were flagged by ineffassign since the variables are not read afterward. The main memory optimization comes from the []byte signature change. Signed-off-by: yxxhero <aiopsclub@163.com> * fix: address PR review comments for reduce memory consumption - Replace append copy in Parse with io.MultiReader to avoid manifest copy - Extract length ratio constants for named constants Add unit tests for length ratio heuristic - Restructure cmd flows to parse into locals before diff release raw bytes earlier Signed-off-by: yxxhero <aiopsclub@163.com> * fix: address all PR review comments - Nil out raw manifest byte slices after parsing in upgrade/release/rollback/revision to allow GC reclamation before diff computation - Fix unkeyed Options struct literals in diff_test.go (4 locations) to use keyed field names for resilience against struct changes - Skip length-ratio filter for Secrets in contentSearch() since Secret raw content lengths before redaction/decoding are unreliable rename predictors - Extract empty-content check before the Secret kind branch (DRY) Agent-Logs-Url: https://github.com/databus23/helm-diff/sessions/c2d2d0b3-3840-48a8-89f0-4b6913c83973 Co-authored-by: yxxhero <11087727+yxxhero@users.noreply.github.com> * fix: suppress ineffassign warnings on GC-hint nil assignments Add //nolint:ineffassign comments on the nil assignments that are intentionally written to allow GC to reclaim raw manifest bytes before the diff computation begins. The ineffassign linter correctly identifies these as never-read-after-assignment, but the intent is a deliberate GC hint for long-running diff operations. Agent-Logs-Url: https://github.com/databus23/helm-diff/sessions/aa2df3f5-57bb-4939-ac45-3f3612deac60 Co-authored-by: yxxhero <11087727+yxxhero@users.noreply.github.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> | 5 个月前 | |
feat: omit 'app.kubernetes.io/managed-by' label on three-way-diff (#1044) * feat: omit 'app.kubernetes.io/managed-by' label on three-way-diff * Remove labels if empty in util.go * Update util.go * refactor: avoid nested if for annotations/labels tidy Address review feedback to not use 'if in if'. Extract a deleteFromMap helper that uses a comma-ok type assertion (early return) and a final empty-check, so the two conditionals are sequential rather than nested. Apply it to both the existing annotations tidy and the new labels tidy (omitting app.kubernetes.io/managed-by) for consistency. Also add test coverage for label omission and the empty-label cleanup path. Signed-off-by: yxxhero <aiopsclub@163.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com> Co-authored-by: yxxhero <aiopsclub@163.com> | 25 天前 | |
feat: omit 'app.kubernetes.io/managed-by' label on three-way-diff (#1044) * feat: omit 'app.kubernetes.io/managed-by' label on three-way-diff * Remove labels if empty in util.go * Update util.go * refactor: avoid nested if for annotations/labels tidy Address review feedback to not use 'if in if'. Extract a deleteFromMap helper that uses a comma-ok type assertion (early return) and a final empty-check, so the two conditionals are sequential rather than nested. Apply it to both the existing annotations tidy and the new labels tidy (omitting app.kubernetes.io/managed-by) for consistency. Also add test coverage for label omission and the empty-label cleanup path. Signed-off-by: yxxhero <aiopsclub@163.com> --------- Signed-off-by: yxxhero <aiopsclub@163.com> Co-authored-by: yxxhero <aiopsclub@163.com> | 25 天前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 1 年前 | ||
| 9 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 5 个月前 | ||
| 25 天前 | ||
| 25 天前 |