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

[MiqGenericMountSession] Support @byte_count in #download_single #395

Conversation

NickLaMuro
Copy link
Member

Updates MiqGenericMountSession#download_single to support the byte_count attribute. If passed, it will only download the specified number of bytes to the destination.

Since we are using IO.copy_stream, this is fairly straight forward, and works with basically any IO-like object.:

@NickLaMuro NickLaMuro force-pushed the support-byte-count-for-mount-filesystems branch from 9416cd2 to 6b83b7e Compare November 2, 2018 23:50
@NickLaMuro NickLaMuro force-pushed the support-byte-count-for-mount-filesystems branch from 6b83b7e to 85e1a97 Compare November 3, 2018 00:19
@miq-bot
Copy link
Member

miq-bot commented Nov 3, 2018

Checked commit NickLaMuro@85e1a97 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍰

@carbonin carbonin self-assigned this Nov 5, 2018
@carbonin carbonin added this to the Sprint 98 Ending Nov 5, 2018 milestone Nov 5, 2018
@carbonin carbonin merged commit 63c7f1a into ManageIQ:master Nov 5, 2018
simaishi pushed a commit that referenced this pull request Nov 5, 2018
…filesystems

[MiqGenericMountSession] Support @byte_count in #download_single

(cherry picked from commit 63c7f1a)
@simaishi
Copy link
Contributor

simaishi commented Nov 5, 2018

Hammer backport details:

$ git log -1
commit a8322dddff752e2155f31e7ba070d7946aeed2ff
Author: Nick Carboni <[email protected]>
Date:   Mon Nov 5 09:55:15 2018 -0500

    Merge pull request #395 from NickLaMuro/support-byte-count-for-mount-filesystems
    
    [MiqGenericMountSession] Support @byte_count in #download_single
    
    (cherry picked from commit 63c7f1a7c104a867507e43f436e252e4928bc46c)

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

Successfully merging this pull request may close these issues.

4 participants