src: refactor cleanup logic to expose buildkitd.log

[adityamaru]

Previosuly, we only killed the buildkitd process and unmounted if builderInfo was non null. This was wrong cause we could have setup builkdkitd, but failed after that step. This would then rely on the last ditch effort by the post action to cleanup. We now change the proc kill and unmount to happen on any build error.

---

Important

Refactor cleanup logic in main.ts to ensure buildkitd is killed and unmounted on any build error, and increase buildkitd socket timeout in setup_builder.ts.

• Behavior:
  • Refactor cleanup logic in main.ts to kill buildkitd and unmount on any build error, not just when builderInfo is non-null.
  • Expose buildkitd.log contents on build error in main.ts.
• Timeout:
  • Increase buildkitd socket timeout from 5 to 10 seconds in setup_builder.ts.

^{This description was created by for 54bc4e0. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 54bc4e0 in 31 seconds

More details

• Looked at 111 lines of code in 2 files
• Skipped 2 files when reviewing.
• Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_9Fy97SQ3ozypIVP1

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.