Skip to content

RSDK-14228 Add API for package status#873

Open
Benjamin Rewis (benjirewis) wants to merge 5 commits into
viamrobotics:mainfrom
benjirewis:package-status
Open

RSDK-14228 Add API for package status#873
Benjamin Rewis (benjirewis) wants to merge 5 commits into
viamrobotics:mainfrom
benjirewis:package-status

Conversation

@benjirewis

@benjirewis Benjamin Rewis (benjirewis) commented Jun 29, 2026

Copy link
Copy Markdown
Member

RSDK-14228

Defines PackageStatus and adds repeated PackageStatus packages to GetMachineStatusResponse per the package status API scope.

Adds two extra fields bytes_downloaded and total_bytes.

See RDK changes here.

@github-actions github-actions Bot added the safe to test committer is a member of this org label Jun 29, 2026
@viamrobotics-overwatch

Copy link
Copy Markdown

Hey Benjamin Rewis (@benjirewis) — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

Auto-comment from overwatch. Will not re-nudge for 7 days.

@benjirewis Benjamin Rewis (benjirewis) changed the title [WIP] Add API for package status [RSDK-14229] Add API for package status Jul 7, 2026
@benjirewis Benjamin Rewis (benjirewis) changed the title [RSDK-14229] Add API for package status RSDK-14228 Add API for package status Jul 7, 2026
@viamrobotics-overwatch

Copy link
Copy Markdown

Hey Benjamin Rewis (@benjirewis) — CI is green and no reviewer is assigned yet. Could you request one when you have a chance?

Auto-comment from overwatch. Will not re-nudge for 7 days.

Comment on lines +557 to +562
// number of bytes of the package tarball downloaded so far. Only populated
// while state == STATE_DOWNLOADING; equals total_bytes once the download completes.
int64 bytes_downloaded = 7;

// total size of the package tarball in bytes. Zero if the size is unknown.
int64 total_bytes = 8;

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.

If these won't be negative, you can use uint64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test committer is a member of this org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants