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

os: add package_size to TransferRequest #90

Merged
merged 5 commits into from
Oct 10, 2022

Conversation

brianneville
Copy link
Contributor

This will allow the target to know whether an OS package is
going to exceed the maximum possible size before the
client starts streaming any bytes to the target.
Currently, if the client was to transfer a larger-than-allowed
OS package, the target would not be able to detect this until
midway through the transfer_content stream.

This will allow the target to know whether an OS package is
 going to exceed the maximum possible size before the
 client starts streaming any bytes to the target.
Currently, if the client was to transfer a larger-than-allowed
 OS package, the target would not be able to detect this until
 midway through the transfer_content stream.
@coveralls
Copy link

coveralls commented Jul 20, 2022

Pull Request Test Coverage Report for Build 2718230154

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 2503260217: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

os/os.proto Show resolved Hide resolved
Version incremented following the addition of the
 package_size field to the TransferRequest type
os/os.proto Outdated Show resolved Hide resolved
os/os.proto Show resolved Hide resolved
@samribeiro samribeiro mentioned this pull request Jul 20, 2022
@samribeiro samribeiro linked an issue Jul 20, 2022 that may be closed by this pull request
@samribeiro samribeiro merged commit c3391aa into openconfig:main Oct 10, 2022
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.

os.proto Install RPC
5 participants