Skip to content

List leftover dash processes with watch identity - #1057

Merged
jim80net merged 4 commits into
mainfrom
agent/retro-0905-p1-e-watch-identity
Sep 5, 2026
Merged

jim80net merged 4 commits into
mainfrom
agent/retro-0905-p1-e-watch-identity

Conversation

@jim80net

@jim80net jim80net commented Sep 5, 2026 •

Copy link
Copy Markdown
Owner

Operator summary

Before

After a PATH install, leftover dashboard processes (including version-suffixed binaries) were only visible by inspecting process tables by hand. Status described the new install; old loopback binds stayed up.

Change

flotilla watch identity lists running watch and dash processes: disk hash, executable hash, revision, deleted-inode flag, and dash listen addresses. Version-suffixed flotilla-<build> dash leftovers are included. The command is list-only; it does not kill.

After

An operator can see leftover dashboard binds and deleted inodes without sending signals.

Identifiers

  • Head: 5137a2243673f6b852e8ec69f5039958ec271219
  • Branch: agent/retro-0905-p1-e-watch-identity

Summary

Process admission accepts basename flotilla or a flotilla- prefix. Non-flotilla binaries stay ignored. No kill or signal path.

Independent review: code-clean at this exact successor head. Not installed. A leftover dash bind on the host is not reaped by this PR.


Summary by cubic

Adds the flotilla watch identity command so operators can see leftover dashboard processes and deleted inodes without inspecting process tables by hand.

  • Lists running watch and dash processes with disk and executable SHA-256 hashes, VCS revision, deleted-inode flag, and dash listen addresses.
  • Includes version-suffixed flotilla-<build> dash leftovers.
  • Excludes non-flotilla binaries; the command is read-only and sends no signals.
  • Reports a warning when an executable identity is unreadable or missing, surfacing degraded processes instead of failing.

Written for commit 8910ef2. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai 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.

All reported issues were addressed across 4 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread cmd/flotilla/watch_identity.go Resolved
Comment thread cmd/flotilla/watch_identity.go Resolved

@cubic-dev-ai cubic-dev-ai 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.

All reported issues were addressed across 2 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread cmd/flotilla/watch_identity.go Resolved
Comment thread cmd/flotilla/watch_identity_test.go Resolved
@jim80net
jim80net merged commit 0d27103 into main Sep 5, 2026
6 checks passed
@jim80net
jim80net deleted the agent/retro-0905-p1-e-watch-identity branch September 5, 2026 21:17
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