Skip to content

misc: main sync#385

Merged
prkhrkat merged 13 commits intodevelopfrom
main
Apr 8, 2026
Merged

misc: main sync#385
prkhrkat merged 13 commits intodevelopfrom
main

Conversation

@prkhrkat
Copy link
Copy Markdown
Contributor

@prkhrkat prkhrkat commented Apr 8, 2026

No description provided.

prkhrkat and others added 13 commits February 24, 2026 12:58
- Add BuildxBuilderPodWaitDurationSecs to CommonWorkflowRequest (JSON field)
- Add BuildxK8sClientFactory type+field for testability; default wraps newBuildxK8sClient
- Extract waitForBuilderPods as a package-level helper (testable)
- Add initial pod liveness wait in setupBuildxBuilder after RegisterBuilderPods
  to eliminate false-positive BuilderPodDeletedError from pod startup latency
- executeDockerReBuild: use factory for k8s client, buffered done channel (size 1)
  to prevent goroutine leak, context.WithTimeout instead of time.After
- Fix pre-existing GitManager_test.go broken CloneAndCheckout call signature
- Tests (Groups A-D, F-G): 15 unit tests covering duration computation,
  executeDockerReBuild error paths, waitForBuilderPods, JSON compat
…ings.Contains in tests

- setupBuildxBuilder: replace direct newBuildxK8sClient call with
  impl.k8sClientFactory, making the initial build path consistent with
  executeDockerReBuild and testable without a live k8s cluster
- buildx_rebuild_test.go: replace hand-rolled contains() helper with
  strings.Contains from stdlib
…t-timeout

feat: add env-driven timeout for buildx builder pod readiness
feat: add native sidecar container support in resource tree calls
@prkhrkat prkhrkat merged commit e2d5c63 into develop Apr 8, 2026
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants