Skip to content

chore(deps): automated Go dependency update - #6036

Closed
replicated-software-factory[bot] wants to merge 1 commit into
mainfrom
deps/kots-update-go
Closed

chore(deps): automated Go dependency update#6036
replicated-software-factory[bot] wants to merge 1 commit into
mainfrom
deps/kots-update-go

Conversation

@replicated-software-factory

Copy link
Copy Markdown
Contributor

Automated Go dependency update for the kots-update-go workflow.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


ElasticClaw Factory seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@greptile-apps

greptile-apps Bot commented Aug 21, 2026

Copy link
Copy Markdown

Reviews (1): Last reviewed commit: "chore(deps): automated Go dependency upd..." | Re-trigger Greptile

Comment thread pkg/image/online.go

// IsPrivateImageFn is the function used by UpdateInstallationImages to determine if an image is private.
// It defaults to IsPrivateImage but can be overridden in tests to avoid network calls.
var IsPrivateImageFn = IsPrivateImage

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Process-wide mutable test hook

If tests or callers replace IsPrivateImageFn concurrently, UpdateInstallationImages reads the shared variable from up to ten worker goroutines, causing data races or cross-test image-privacy classifications. Scope this override to an invocation or otherwise synchronize it to preserve test isolation.

Knowledge Base Used: Image Registry: Pull, Push, Copy, and Auth

- Ran go get -u ./... to refresh direct and transitive Go dependencies.
- go.mod language/toolchain version bumped to 1.26.7 to satisfy
  github.com/bitnami/sealed-secrets v0.39.1.
- helm.sh/helm/v4 upgraded to 4.2.4; updated pull/base test fixtures to
  match the new helm release serialization.
- Fixed image privacy determinism in pull tests by injecting a mockable
  IsPrivateImageFn so the integration tests no longer depend on live
  Docker Hub privacy checks.
@greptile-apps

greptile-apps Bot commented Aug 23, 2026

Copy link
Copy Markdown

Reviews (2): Last reviewed commit: "chore(deps): automated Go dependency upd..." | Re-trigger Greptile

@xavpaice

xavpaice commented Sep 6, 2026

Copy link
Copy Markdown
Member

Needs rebase and is a substantial change. Closing, we can recreate another one.

@xavpaice xavpaice closed this Sep 6, 2026
@xavpaice
xavpaice deleted the deps/kots-update-go branch September 6, 2026 22:36
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.

2 participants