878 Commits (copilot/fix-analyze-go-job)

Author	SHA1	Message	Date
Matthieu MOREL	dc0e3f10c3	chore(pkg): fix perfsprint linter issues part 6 ... Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	1d2d63cc43	chore(pkg): enable perfsprint linter ... #### Description enable perfsprint linter in pkg/cmd Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	63f03c0f5c	chore(pkg): enable perfsprint linter ... #### Description enable perfsprint linter in pkg/action Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	c25c988cfb	chore(pkg): enable perfsprint linter ... #### Description enable perfsprint linter in pkg/registry Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	0fecfd04c2	chore(pkg): enable perfsprint linter ... #### Description enable perfsprint linter in pkg/cmd Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	6c2cb2f547	chore(pkg): enable perfsprint linter ... #### Description enable perfsprint linter in pkg/cmd Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Mads Jensen	36cb3a2fe9	Replace unneeded use of t.Fatalf with t.Fatal ... Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	6 months ago
George Jenkins	be7cdb69e5	Merge pull request #31806 from mmorel-35/errorlint-pkg-4-15f432a ... fix(pkg): errorlint linter	6 months ago
George Jenkins	3dfc4f3cf6	Merge pull request #31858 from helm/gjenkins/go_fix ... chore: `go fix` fixes	6 months ago
MrJack	300f71b1eb	feat(history): add rollback revision column to helm history output ... Signed-off-by: MrJack <36191829+biagiopietro@users.noreply.github.com>	6 months ago
George Jenkins	3ad9ccdb19	Merge pull request #31563 from Mujib-Ahasan/dry-run-fix ... fix: --dry-run=server now respect generateName	6 months ago
George Jenkins	92b64e87ad	chore: fixes ... Signed-off-by: George Jenkins <gvjenkins@gmail.com>	6 months ago
Joe Julian	947b429cdd	Merge pull request #31801 from mmorel-35/modernize-pkg-4-062b248 ... chore(pkg): fix modernize linter	6 months ago
Joe Julian	78f5af72be	Merge pull request #31802 from mmorel-35/modernize-pkg-5-a3c8fdb ... chore(pkg): fix modernize linter	6 months ago
Joe Julian	70899a6b3f	Merge pull request #31578 from manute/main ... fix(action): crd resources can be empty	6 months ago
Joe Julian	56d0d9374a	Merge pull request #31810 from mmorel-35/errorlint-pkg-5-dda0145 ... fix(pkg): errorlint linter	6 months ago
Matthieu MOREL	259f76a849	fix(pkg): errorlint linter ... errorlint linter in pkg/cmd Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	4d0ae7f33a	fix(pkg): errorlint linter ... #### Description errorlint linter in pkg Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	c8989d984f	fix(pkg): errorlint linter ... #### Description errorlint linter in pkg Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Manuel Alonso	9817a68618	fix(install): check nil for restClientGetter and fix tests ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	6 months ago
Matthieu MOREL	5cc2e55714	chore(pkg): fix modernize linter ... #### Description fix modernize linter in pkg/chart/v2/lint/rules Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	111d4e6e0e	chore(pkg): fix modernize linter ... #### Description fix modernize linter in pkg/chart/common/util Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matheus Pimenta	c1cc625323	chore(defaults): server-side apply SDK defaults should always match the CLI defaults ... Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>	7 months ago
Manuel Alonso	7c089bed26	Merge branch 'main' into main ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	7 months ago
Manuel Alonso	6501ef490a	chore(refactor): better testing and functionality for installing crd ... Signed-off-by: Manuel Alonso <m.alonso@Manuels-MacBook-Pro.local>	7 months ago
Matheus Pimenta	59ece92bed	pkg/kube: introduce support for custom kstatus readers ... Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>	7 months ago
Manuel Alonso	0f949a92c1	fix(test): fix tests and check nil for restclient ... Signed-off-by: Manuel Alonso <m.alonso@Manuels-MacBook-Pro.local>	7 months ago
Manuel Alonso Gonzalez	561410ae1d	fix(test): merge fix correctly ... Signed-off-by: Manuel Alonso Gonzalez <434575+manute@users.noreply.github.com> Signed-off-by: Manuel Alonso Gonzalez <m.alonso@Manuels-MacBook-Pro.local>	7 months ago
Evans Mungai	e751a70e84	fix(release): fix test compilation error ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	7 months ago
Evans Mungai	05f25a3834	Merge pull request #31579 from benoittgt/delete-hook-after-log-retrieval ... fix: Delay test hook deletion until after logs are retrieved with `helm test release --logs`	7 months ago
Manuel Alonso	df8ec771f9	Merge branch 'main' into main ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	7 months ago
Evans Mungai	03b535d6b7	Merge branch 'main' into em/check-ownership-before-delete ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	7 months ago
George Jenkins	fa41d60d73	Merge pull request #31239 from StephanieHhnbrg/test-actions-pkg ... Add tests to action package to improve coverage	8 months ago
Manuel Alonso	00f0a48a7d	fix(install): add more tests and check nil file data ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	8 months ago
Manuel Alonso	0357e8d0f7	fix(test): no check empty resources ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	8 months ago
Manuel Alonso	52235cc0bf	fix(install): check lenght and file nil, add tests ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	8 months ago
Manuel Alonso	268593bf2e	fix(action): crd resources can be empty ... Signed-off-by: Manuel Alonso <434575+manute@users.noreply.github.com>	8 months ago
Mujib Ahasan	170911459b	fix: casing issue fixed ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	8 months ago
Mujib Ahasan	94860626ce	fix: error handled correctly ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	8 months ago
Mujib Ahasan	12e8b715aa	fix: doc string added ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	8 months ago
Terry Howe	5d2ab10caa	Merge pull request #12564 from hiddeco/supersede-on-uninstall ... fix(uninstall): supersede deployed releases	8 months ago
Terry Howe	2597088a20	Merge pull request #31586 from calvinbui/main ... fix: use length check for MetaDependencies instead of nil comparison	8 months ago
Robert Sirchia	4cf7d8d025	Merge pull request #31641 from wangjingcun/main ... chore: fix some comments to improve readability	8 months ago
Terry Howe	d28853e206	Merge pull request #31624 from atombrella/feature/sloglint ... Enable the `sloglint` linter	8 months ago
wangjingcun	858cf31583	chore: fix some comments to improve readability ... Signed-off-by: wangjingcun <wangjingcun@aliyun.com>	8 months ago
Mads Jensen	a18e59e465	Enable the sloglint linter ... Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	8 months ago
George Jenkins	3165e5463a	Merge pull request #31528 from benoittgt/31423-git-version ... fix: preserve vendor suffixes in KubeVersion.GitVersion	8 months ago
Stephanie Hohenberg	a109ac2f06	fixes comment in install.go ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	9 months ago
Stephanie Hohenberg	2f598ffc85	fixes tests after merge ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	9 months ago
Stephanie Hohenberg	bb9356e182	fixes lint issue ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	9 months ago
Stephanie Hohenberg	8cf4ad73c3	updates tests after rebase from master ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	9 months ago
Stephanie Hohenberg	31131cff9b	Add tests to action package to improve coverage ... Signed-off-by: Stephanie Hohenberg <stephanie.hohenberg@gmail.com>	9 months ago
Mujib Ahasan	115409976b	update: test coverage added for helper function validateNameAndGenerateName ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	9 months ago
Mujib Ahasan	522d2fe615	update: helper function added for the business logic ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	9 months ago
George Jenkins	dfa1e3c3c5	Merge pull request #31562 from atombrella/feature/modernize ... Use modernize to use newer Golang features.	9 months ago
Evans Mungai	f552950b05	Add ownership verification to uninstall action ... This commit adds ownership verification to the uninstall action. It checks that the resource to be deleted is owned by the chart before deleting it. Fixes: #31333 Signed-off-by: Evans Mungai <mbuevans@gmail.com>	9 months ago
Benoit Tigeot	45c5f3aaca	Deal with golint warning with private executeShutdownFunc ... ``` Error: pkg/action/release_testing.go:60:57: unexported-return: exported method Run returns unexported type action.executeShutdownFunc, which can be annoying to use (revive) func (r *ReleaseTesting) Run(name string) (ri.Releaser, executeShutdownFunc, error) { ``` Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	9 months ago
Calvin Bui	b33d4ae0c0	Use length check for MetaDependencies instead of nil comparison ... Signed-off-by: Calvin Bui <3604363+calvinbui@users.noreply.github.com>	9 months ago
Benoit Tigeot	70fc5f97e2	Code review ... Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	9 months ago
Evans Mungai	6d5b5aab32	Additional logging for ownership verification in dry-run mode ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	9 months ago
Evans Mungai	906f87ce1d	feat: add ownership verification before deleting resources during uninstall ... Fixes #31333 Helm uninstall now verifies resource ownership before deletion by checking Helm labels and annotations. Resources not owned by the release being uninstalled are skipped with warnings, preventing accidental deletion of resources belonging to other releases. Signed-off-by: Evans Mungai <mbuevans@gmail.com>	9 months ago
Mujib Ahasan	6769fb6fb6	generateName is also considered in logic ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	9 months ago
Michelle Fernandez Bieber	6bb5bcc212	Update pkg/action/hooks.go ... Co-authored-by: Marco Lecheler <marco@task.media> Signed-off-by: Michelle Fernandez Bieber <37021266+mfbieber@users.noreply.github.com>	9 months ago
Michelle Fernandez Bieber	d9301441f4	added check for nil shutdown ... Signed-off-by: Michelle Fernandez Bieber <mfbieber@gmail.com>	9 months ago
Michelle Fernandez Bieber	7a61ebf013	cleaned up empty line ... Signed-off-by: Michelle Fernandez Bieber <mfbieber@gmail.com>	9 months ago
Michelle Fernandez Bieber	10714772be	updated comment and made defer of shutdown function return errors as before and not the possible shutdown error ... Signed-off-by: Michelle Fernandez Bieber <mfbieber@gmail.com>	9 months ago
Michelle Fernandez Bieber	7a557580e1	added shutdown hook that is executed after the logs have been retrieved ... Signed-off-by: Michelle Fernandez Bieber <mfbieber@gmail.com>	9 months ago
Mujib Ahasan	b357bcae86	update: business logic respected for skipping object missing name ... Signed-off-by: Mujib Ahasan <ahasanmujib8@gmail.com>	9 months ago
Mads Jensen	6cceead399	Use modernize to use newer Golang features. ... The rangeint suggestion is faster. The maps.Copy and slices.Contains are just syntatic sugar. Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	9 months ago
Matt Farina	61d289c119	Merge pull request #31518 from gjenkins8/gjenkins/fix_update_create ... fix: Use server-side apply for object create during update	9 months ago
Matt Farina	698fcd2290	Merge pull request #31515 from gjenkins8/gjenkins/copy_adopted ... fix: Copy adopted resource info	9 months ago
Benoit Tigeot	ce273eea48	fix: preserve vendor suffixes in KubeVersion.GitVersion ... Helm 3.19.0 introduced a regression where vendor-specific suffixes (e.g., -gke.1245000, -eks-4096722, +) are stripped from .Capabilities.KubeVersion.GitVersion, breaking charts that detect managed Kubernetes platforms. The root cause was using k8sversion.ParseGeneric().String() which intentionally discards vendor suffixes. The fix stores both the full version (with vendor suffix) and a normalized version. String() returns the normalized version for constraint checking (e.g., ">= 1.21.0"), while Version/GitVersion preserve the full string for template access. Fixes #31423 Related to #31063, #31078 Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	9 months ago
George Jenkins	18616e6ce9	fix: Use server-side apply for object create during update ... Signed-off-by: George Jenkins <gvjenkins@gmail.com>	9 months ago
George Jenkins	855ebb6884	Copy adopted resource info ... Signed-off-by: George Jenkins <gvjenkins@gmail.com>	9 months ago
promalert	d8c4040336	refactor: use strings.Builder to improve performance ... Signed-off-by: promalert <promalert@outlook.com>	9 months ago
Scott Rigby	0ee89d2d4e	Merge pull request #31418 from tisonkun/typo ... chore: fix typo of public field	9 months ago
Hidde Beydals	d158708fbf	fix(rollback): `errors.Is` instead of string comp ... Signed-off-by: Hidde Beydals <hidde@hhh.computer>	9 months ago
Hidde Beydals	2f1ecc7100	fix(uninstall): supersede deployed releases ... This ensures that when `helm uninstall` is run with `--keep-history` any release in a `deployed` state other than the last release (e.g. due to a failed upgrade) is being marked as `superseded`. As a by-effect, running `helm upgrade` on a release which has been uninstalled after an upgrade failure now no longer works. But instead fails with a `"<name>" has no deployed releases` error. Which is the (likely) intended behavior, and prevents other side-effects like rolling back to a release version which happened before the uninstall if `--atomic` (or `--rollback-on-failure`) was provided. Signed-off-by: Hidde Beydals <hidde@hhh.computer>	9 months ago
George Jenkins	384a3f0ff8	Merge pull request #31385 from porridge/test-unknown-policy ... test: protect unknown hook delete policies	10 months ago
tison	90d01915f9	for all other similar cases ... Signed-off-by: tison <wander4096@gmail.com>	10 months ago
Matt Farina	ca8eae9361	Reproducible chart archive builds ... Building the same chart into an archive multiple times will have the same sha256 hash. Perviously, the time in the headers for a file was time.Now() which changed each time. The time is now collected from the operating system when the file is loaded and this time is used. Fixes: #3612 Signed-off-by: Matt Farina <matt.farina@suse.com>	10 months ago
Scott Rigby	f4c5220d99	Merge pull request #31411 from banjoh/em/reinstate-logger-param ... feat: reinstate logger parameter to actions package	10 months ago
tison	0d6de28bf4	chore: fix typo of public field ... Signed-off-by: tison <wander4096@gmail.com>	10 months ago
Evans Mungai	2ddeb50fa6	Set default logger in Configuration constructor ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	a112bf5aa6	Remove non-reachable code ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Robert Sirchia	25ad74f5a7	Merge pull request #31337 from rachelvweber/rawo/fixingWaitStrategy ... Fixing rollback and uninstall client WaitStrategy	10 months ago
Evans Mungai	aed687eaa1	Add config options to NewConfiguration() ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	b6eca1c0f1	Refactor logging functionality to use slog.Handler ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	5ab4ca5490	Embed logging functionality to DRY code ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	9c32e34d60	Add logger to sql driver and ensure storage has logger ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	50e43f4017	nil logger should be handled by discard handler ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	7a5816b106	Self review changes ... Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
Evans Mungai	b1d4dc680d	feat: reinstate logger parameter to actions package ... Fixes: #31399 Signed-off-by: Evans Mungai <mbuevans@gmail.com>	10 months ago
George Jenkins	7dd2484aed	Merge pull request #31295 from TerryHowe/fix-make-helm-list-show-all-by-default ... Fix make helm list show all by default	10 months ago
Scott Rigby	f0530309f0	Merge pull request #31302 from TerryHowe/fix-helm-verify-signature ... fix: helm verify Run signature	10 months ago
Terry Howe	c3aa5c51e6	chore: clean up for comments ... Signed-off-by: Terry Howe <terrylhowe@gmail.com>	10 months ago
Terry Howe	8a73d97984	fix: make helm list show all by default ... Signed-off-by: Terry Howe <terrylhowe@gmail.com>	10 months ago
Marcin Owsiany	269a32a6d2	test: protect unknown hook delete policies ... Signed-off-by: Marcin Owsiany <porridge@redhat.com>	10 months ago
Matt Farina	fbf02e494e	Merge pull request #30980 from gjenkins8/gjenkins/cleanup_kubeclient_interfaces ... cleanup: Remove/consolidate redundant kube client Interfaces	10 months ago
Matt Farina	a229bf6529	Ensure the release interface is used in actions instead of the type ... Signed-off-by: Matt Farina <matt.farina@suse.com>	10 months ago