Skip to content

feat(buildsettings): add dockerBuild support in CreateBuildSettingsInfo#5780

Open
fskhiri wants to merge 8 commits intomasterfrom
fskhiri/dockerBuild
Open

feat(buildsettings): add dockerBuild support in CreateBuildSettingsInfo#5780
fskhiri wants to merge 8 commits intomasterfrom
fskhiri/dockerBuild

Conversation

@fskhiri
Copy link
Copy Markdown
Contributor

@fskhiri fskhiri commented May 6, 2026

Description

dockerBuild was falling through to the default branch, emitting a warning and returning empty buildSettingsInfo. Add it as a first-class supported tool alongside kanikoExecute and cnbBuild

Checklist

  • Tests
  • Documentation
  • Inner source library needs updating

dockerBuild was falling through to the default branch, emitting a
warning and returning empty buildSettingsInfo. Add it as a first-class
supported tool alongside kanikoExecute and cnbBuild.
@fskhiri fskhiri requested a review from a team as a code owner May 6, 2026 09:37
Comment thread documentation/docs/steps/dockerBuild.md
@fskhiri fskhiri requested review from a team and fabienfritz May 6, 2026 10:45
kanikoExecute is Jenkins/Kubernetes-centric and requires a sidecar
container. Users running on GitHub Actions have a simpler native
alternative (dockerBuild via Docker BuildKit). Add a visible note at
the top of the doc so they discover it before going deep into Kaniko
setup.
@fskhiri fskhiri requested a review from a team as a code owner May 6, 2026 11:46
@fskhiri
Copy link
Copy Markdown
Contributor Author

fskhiri commented May 6, 2026

/it-go

Copy link
Copy Markdown

@And-tab-8889 And-tab-8889 left a comment

Choose a reason for hiding this comment

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

Overall the change is correct and minimal. Four things worth addressing before merge — see inline comments below, plus one general note:

cnbBuild.md cross-reference missing. kanikoExecute.md gets a note pointing GHA users toward dockerBuild, but cnbBuild.md does not. Since all three are container image build alternatives in GHA pipelines, the same note belongs there too.

Comment thread pkg/buildsettings/buildSettings.go
Comment thread pkg/buildsettings/buildSettings_test.go
Comment thread documentation/docs/steps/dockerBuild.md
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.

3 participants