Skip to content

Make remote push the primary flow - #63

Open
chruffins wants to merge 4 commits into
mainfrom
hypeship/docker-like-push-ux
Open

Make remote push the primary flow#63
chruffins wants to merge 4 commits into
mainfrom
hypeship/docker-like-push-ux

Conversation

@chruffins

@chruffins chruffins commented Aug 12, 2026

Copy link
Copy Markdown

summary

  • make hypeman push SOURCE TARGET the primary remote-registry flow
  • wait for direct pushes by default; keep push create detached for compatibility and add --detach
  • make structured formats return the final direct-push result unless detached
  • render one live status line in terminals and plain newline status in CI/pipes
  • validate source/target references and required target tags before creating jobs
  • keep local Docker-daemon uploads via hypeman push local IMAGE [TARGET] plus the one-argument legacy form
  • add --password-stdin, push ls, and push inspect

tests

  • GOCACHE=/tmp/hypeman-go-build go test ./...

Note

Medium Risk
CLI contract changes (default remote vs local push) can break existing automation; registry credentials and push orchestration are sensitive though logic remains client-side.

Overview
hypeman push SOURCE TARGET is now the main way to export a cached Hypeman image to a remote registry. It blocks until the job finishes, polling status and showing Docker-style progress on stderr; --detach (and legacy push create) still queue and print the job ID only.

Local Docker → Hypeman uploads move to hypeman push local; a single argument on push still runs the old local upload for script compatibility. Local pushes gain stricter base-URL checks, optional HTTP/insecure registry parsing, byte progress, and digest/size output.

Remote pushes add --password-stdin, shared registry flags on the parent command, reference validation (target must have an explicit tag), and ls / inspect aliases for listing and inspecting jobs.

Reviewed by Cursor Bugbot for commit 6ce0768. Bugbot is set up for automated code reviews on this repo. Configure here.

@chruffins chruffins changed the title Improve Docker-like push UX Make remote push the primary flow Aug 14, 2026
@chruffins
chruffins marked this pull request as ready for review August 14, 2026 15:12

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0a20e7e. Configure here.

Comment thread pkg/cmd/pushcmd.go Outdated
Comment thread pkg/cmd/push.go
Comment thread pkg/cmd/pushcmd.go
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