Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/implement status command #65

Merged
merged 23 commits into from
Oct 15, 2019

Conversation

mcred
Copy link
Collaborator

@mcred mcred commented Oct 14, 2019

Issues Addressed

Fixes #56
Fixes launch issue with Tags vs Digests
Updates to server 0.4.5
Bump version to 0.3.3

Proposed Changes

Use new status API endpoint and format output.

Testing

Tested locally

Derek Smart and others added 23 commits September 18, 2019 10:58
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Github Actions with minor fixes (#11)
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
Signed-off-by: Derek Smart <[email protected]>
@mcred mcred requested a review from a team as a code owner October 14, 2019 22:35
private val n = System.lineSeparator()

/**
* https://stackoverflow.com/a/5599842
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI, you may want to compare the output of this with RsyncExecutor::numberToString() in titan-server. The latter is based on the ZFS implementation, which pays special attention to behavior near boundaries (e.g. 1023MiB instead of 0.99GiB).

@eschrock eschrock merged commit 253a700 into titan-data:master Oct 15, 2019
@mcred mcred deleted the feature/implement-status-command branch October 24, 2019 02:59
mcred referenced this pull request in mcred/titan Oct 29, 2019
mcred referenced this pull request in mcred/titan Oct 31, 2019
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.

implement status command
2 participants