Skip to content

Conversation

@p-se
Copy link
Contributor

@p-se p-se commented Nov 26, 2025

  • Address review comments
  • InsecureSkipVerify for cloning git repos with fleet.yaml's helm.chart field
  • Refactor code duplication
  • Address review comments

(cherry picked from commit 3ec37b2)

Conflicts resolved:

  • e2e/single-cluster/schedule_test.go
  • e2e/testenv/githelper/git.go
  • internal/bundlereader/loaddirectory.go

Refers to # (TODO create if backport is supposed to be kept/merged)
Backport of #4185

Additional Information

Checklist

  • I have updated the documentation via a pull request in the
    fleet-docs repository.

…ml` file (rancher#4185)

* Address review comments
* InsecureSkipVerify for cloning git repos with fleet.yaml's helm.chart field
* Refactor code duplication
* Address review comments

(cherry picked from commit 3ec37b2)

Conflicts resolved:
	e2e/single-cluster/schedule_test.go
	e2e/testenv/githelper/git.go
	internal/bundlereader/loaddirectory.go
@p-se p-se changed the title Use CABundle when fetching chart with helm.chart field of `fleet.ya… [v0.13] Backport #4185 -- Use CABundle when fetching chart with helm.chart field of fleet.yaml file Nov 26, 2025
* Fix race when creating secrets and Kubernetes jobs

When creating secrets and a Kubernetes Job that uses those secrets in
volumes, it is possible that the job is created before the secret has
been synchronized. If the job required the content of the secret to be
mounted, the file would be missing.

Also fixes usage of `controllerutil.CreateOrUpdate`.

* Apply suggestion from @weyfonk

Co-authored-by: Corentin Néau <[email protected]>

---------

Co-authored-by: Corentin Néau <[email protected]>
(cherry picked from commit 98965d7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant