Skip to content

feat: Add support for topo ps returning processing domains#274

Merged
Luke-Parkin merged 1 commit into
mainfrom
ps-domains
May 27, 2026
Merged

feat: Add support for topo ps returning processing domains#274
Luke-Parkin merged 1 commit into
mainfrom
ps-domains

Conversation

@Luke-Parkin
Copy link
Copy Markdown
Member

Changes

  • Topo has many examples which deploy to heterogenous devices using remoteproc runtime
  • This PR adds support to topo ps for displaying which processing domain each container is deployed to.

Signed-off-by: luke <luke.parkin@arm.com>
@Luke-Parkin Luke-Parkin requested a review from a team as a code owner May 26, 2026 16:01
Copy link
Copy Markdown
Contributor

@yejseo01 yejseo01 left a comment

Choose a reason for hiding this comment

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

minor comment

}

const PSTemplate = `{{if .}}Image Status Address
const PSTemplate = `{{if .}}Image Status Processing Domain Address
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what does PS stand for? Maybe not the best to abbreviate it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think it is best to abbreviate it as that's what everyone calls it 🤷 - I think it's recognisable enough tbh

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it means process status

Copy link
Copy Markdown
Contributor

@yejseo01 yejseo01 left a comment

Choose a reason for hiding this comment

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

LGTM

@Luke-Parkin Luke-Parkin merged commit ec8edb6 into main May 27, 2026
5 checks passed
@Luke-Parkin Luke-Parkin deleted the ps-domains branch May 27, 2026 16:07
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.

2 participants