Skip to content

fix: clean up partial build and publish failures#22

Merged
jmgilman merged 1 commit into
masterfrom
feat/partial-failure-cleanup
May 14, 2026
Merged

fix: clean up partial build and publish failures#22
jmgilman merged 1 commit into
masterfrom
feat/partial-failure-cleanup

Conversation

@jmgilman
Copy link
Copy Markdown
Contributor

Summary

  • remove newly-created IncusOS outputs after later variant build failures
  • reject pre-existing output paths before build to avoid unrecoverable overwrites
  • delete draft imgsrv artifacts after safe pre-publish AddArtifact cleanup points
  • avoid cleanup after PublishVersion because server state is ambiguous once the POST is sent

Verification

  • proto run go -- test -count=1 ./internal/providers/incusos ./internal/publish ./internal/cli
  • proto run go -- test -count=1 -tags integration ./cmd/imgcli
  • moon run root:check --summary minimal
  • git diff --check master

Stacked on #21.

Base automatically changed from feat/multi-variant-incusos to master May 14, 2026 05:36
@jmgilman jmgilman force-pushed the feat/partial-failure-cleanup branch from 2f55fc2 to d75a113 Compare May 14, 2026 05:37
@jmgilman jmgilman merged commit bba56c8 into master May 14, 2026
2 checks passed
@jmgilman jmgilman deleted the feat/partial-failure-cleanup branch May 14, 2026 05:41
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.

1 participant