Skip to content

Shipwright: DockerBuildStrategy Type Migration#128

Merged
jmontleon merged 9 commits into
migtools:shipwrightfrom
psrvere:docker-to-buildah
Sep 11, 2025
Merged

Shipwright: DockerBuildStrategy Type Migration#128
jmontleon merged 9 commits into
migtools:shipwrightfrom
psrvere:docker-to-buildah

Conversation

@psrvere

@psrvere psrvere commented Aug 28, 2025

Copy link
Copy Markdown
Contributor

This PR includes migration of BuildConfig's DockerBuildStrategy Type to Shipwright equivalent specs. Following is the field level status:

Field Names Status
From ✅ (replaces first FROM in docker file)
PullSecret ✅ (via Build resource, generates service account, maybe opt in?)
NoCache ⬆️ (Support --no-cache flag)
Env
ForcePull ⬆️ (Support --pull=always flag)
DockerfilePath
BuildArgs
ImageOptimizationPolicy ⬆️ (Support --squash flag)
Volumes ✅ (Mount paths not supported)

⬆️ Upstream RFE

Please refer to detailed notes here

@psrvere psrvere changed the base branch from main to shipwright September 10, 2025 05:29
  - For docker to buildah migration, this commit adds handling FROM field. This field is passed as "from" arg to shipwirght which then would be picked up as "--from" flag while executing buildah strategy

Signed-off-by: Prateek Rathore <prathore@redhat.com>
Assisted-by: Cursor
Signed-off-by: Prateek Rathore <prathore@redhat.com>
  - added validaton logic

Signed-off-by: Prateek Rathore <prathore@redhat.com>
Signed-off-by: Prateek Rathore <prathore@redhat.com>
Signed-off-by: Prateek Rathore <prathore@redhat.com>
  - for docker to buildah strategy migration

Signed-off-by: Prateek Rathore <prathore@redhat.com>
Signed-off-by: Prateek Rathore <prathore@redhat.com>
Assisted-by: Cursor
Signed-off-by: Prateek Rathore <prathore@redhat.com>
Assisted-by: Cursor
Signed-off-by: Prateek Rathore <prathore@redhat.com>
Assited-by: Cursor
@psrvere psrvere changed the title BuildConfig Docker to Shipwright Buildah Migration Shipwright: DockerBuildStrategy Type Migration Sep 10, 2025
@psrvere psrvere marked this pull request as ready for review September 10, 2025 05:58
@jmontleon jmontleon merged commit 99b1342 into migtools:shipwright Sep 11, 2025
1 check passed
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