Skip to content

Conversation

@TylerGillson
Copy link
Contributor

@TylerGillson TylerGillson commented Aug 14, 2025

What issue type does this pull request address? (keep at least one, remove the others)
/kind bugfix

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
resolves #

Fixes an issue where devspace logs a misleading success message and continues when kind load docker-image returns an error.

Sample logs:

  build:brush-local Image: "public.ecr.aws/<redacted>" with ID "sha256:02b339649f779955643eda965cc9680ca5889b1c159e14b1b18cd53556cd943f" not yet present on node "kind-hub-806960-control-plane", loading...
  build:brush-local ERROR: command "docker save -o /tmp/images-tar2283861265/images.tar public.ecr.aws/<redacted>" failed with error: exit status 1
  build:brush-local 
  build:brush-local Command Output: Error response from daemon: reference does not exist
  build:brush-local error during image load to kind cluster: error executing 'kind load docker-image --name kind-hub-806960 public.ecr.aws/<redacted>': 
  build:brush-local Image loaded to kind cluster
  ...

Please provide a short message that should be published in the DevSpace release notes
Correctly propagate error if kind load docker-image fails.

What else do we need to know?
Nothing

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
@netlify
Copy link

netlify bot commented Aug 14, 2025

Deploy Preview for devspace-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d5165e6
🔍 Latest deploy log https://app.netlify.com/projects/devspace-docs/deploys/689e203650d87c0008f24e61
😎 Deploy Preview https://deploy-preview-2988--devspace-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Tyler Gillson <tyler.gillson@gmail.com>
@zerbitx zerbitx merged commit 17faf2e into devspace-sh:main Aug 21, 2025
18 checks 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