Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ImageBuild: changes out of order error in 0.32.1 #6102

Closed
harjotgill opened this issue Apr 17, 2023 · 2 comments · Fixed by #6104
Closed

ImageBuild: changes out of order error in 0.32.1 #6102

harjotgill opened this issue Apr 17, 2023 · 2 comments · Fixed by #6104
Labels
bug Something isn't working

Comments

@harjotgill
Copy link

harjotgill commented Apr 17, 2023

Expected Behavior

Docker container build fails with a cryptic error. It builds fine with Tilt versions 0.32.0 and earlier.

Current Behavior

Logs
STEP 1/3 — Building Dockerfile: [fluxninja/aperture-operator]
Building Dockerfile for platform linux/amd64:
  # syntax=docker/dockerfile:1
  FROM golang:1.20.3-bullseye AS builder
  
  WORKDIR /src
  COPY --link . .
  
  RUN --mount=type=cache,target=/go/pkg/ \
    --mount=type=cache,target=/root/.cache/go-build/ \
    go mod download
  
  RUN --mount=type=cache,target=/go/pkg/ \
    --mount=type=cache,target=/root/.cache/go-build/ \
    CGO_ENABLED=0 go build \
    -a \
    -o aperture-operator \
    operator/main.go
  
  FROM gcr.io/distroless/static:nonroot
  WORKDIR /
  COPY --from=builder /src/aperture-operator .
  USER 65532:65532
  
  ENTRYPOINT ["/aperture-operator"]


     Building image
     resolve image config for docker.io/docker/dockerfile:1 [done: 429ms]
     docker-image://docker.io/docker/dockerfile:1@sha256:39b85bbfa7536a5feceb7372a0817649ecb2724562a38360f4d6a7782a409b14
     [stage-1 1/3] FROM gcr.io/distroless/static:nonroot@sha256:149531e38c7e4554d4a6725d7d70593ef9f9881358809463800669ac89f3b0ec
     [builder 1/5] FROM docker.io/library/golang:1.20.3-bullseye@sha256:595c9af0430dd84bad33020e7e9e328af4bd1a1aabd46a03b5bf6f252cdbecf3
     [background] read source files 23B
     
     ERROR IN: [background] read source files

Build Failed: ImageBuild: changes out of order: "playground/resources/kong/Dockerfile" "pkg/webhooks/webhooks.go"

Steps to Reproduce

See the above logs or try it out at https:/fluxninja/aperture

Context

tilt doctor Output

$ tilt doctor
...
Tilt: v0.32.1, built 2023-04-14
System: linux-amd64
---
Docker
- Host: unix:///var/run/docker.sock
- Server Version: 20.10.14
- API Version: 1.41
- Builder: 2
- Compose Version: v2.3.3
---
Kubernetes
- Env: kind
- Context: kind-aperture-playground
- Cluster Name: kind-aperture-playground
- Namespace: default
- Container Runtime: containerd
- Version: v1.26.3
- Cluster Local Registry: &RegistryHosting{Host:localhost:5005,HostFromClusterNetwork:ctlptl-registry:5000,HostFromContainerRuntime:,Help:https:/tilt-dev/ctlptl,SingleName:,}
---
Thanks for seeing the Tilt Doctor!
Please send the info above when filing bug reports. 💗

The info below helps us understand how you're using Tilt so we can improve,
but is not required to ask for help.
---
Analytics Settings
--> (These results reflect your personal opt in/out status and may be overridden by an `analytics_settings` call in your Tiltfile)
- User Mode: opt-in
- Machine: 53a99a17a2860a63abaa4507668d32a6
- Repo: U0WktcSBkzdEeB8dlnoRog==

About Your Use Case

@harjotgill harjotgill added the bug Something isn't working label Apr 17, 2023
@harjotgill harjotgill changed the title ImageBuild: changes out of order in 0.32.1 ImageBuild: changes out of order error in 0.32.1 Apr 17, 2023
@ekosov-form3
Copy link

ekosov-form3 commented Apr 17, 2023

Observing a similar issue with v0.32.1. Downgrading to v0.32.0 fixed it.

@nicks
Copy link
Member

nicks commented Apr 17, 2023

thanks for the report! i think i understand the problem

nicks added a commit to nicks/tilt that referenced this issue Apr 17, 2023
the sync protocol should always send over the directory
file info when it sends over files in its subtree, even
if the directory path itself is marked as ignored

fixes tilt-dev#6102
nicks added a commit to nicks/tilt that referenced this issue Apr 17, 2023
the sync protocol should always send over the directory
file info when it sends over files in its subtree, even
if the directory path itself is marked as ignored

fixes tilt-dev#6102

Signed-off-by: Nick Santos <[email protected]>
nicks added a commit that referenced this issue Apr 17, 2023
the sync protocol should always send over the directory
file info when it sends over files in its subtree, even
if the directory path itself is marked as ignored

fixes #6102

Signed-off-by: Nick Santos <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants