FS driver: chunk size is hard-coded

Bug #1792869 reported by Lucian Petrut on 2018-09-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
glance_store
Low
Lucian Petrut

Bug Description

At the moment, the filesystem driver uses a hardcoded 64 KB size, used when reading/writing image.

This can be extremely inefficient, especially when file shares are used. For this reason, the chunk size should be configurable, similar to the other glance store drivers.

Changed in glance:
assignee: nobody → Cyril Roelandt (cyril-roelandt)
assignee: Cyril Roelandt (cyril-roelandt) → nobody
Cyril Roelandt (cyril-roelandt) wrote :

Could you assign yourself to the bug, and move it to glance_store rather than glance?

Lucian Petrut (petrutlucian94) wrote :
affects: glance → glance-store
Changed in glance-store:
assignee: nobody → Lucian Petrut (petrutlucian94)
status: New → Incomplete
status: Incomplete → In Progress
Lucian Petrut (petrutlucian94) wrote :

@Cyril: Done, thanks for mentioning this. I wasn't aware that glance_store bugs are handled separately, sorry about that.

Changed in glance-store:
importance: Undecided → Low

Reviewed: https://review.openstack.org/603023
Committed: https://git.openstack.org/cgit/openstack/glance_store/commit/?id=ea1cad2bedabc72604d935daeb359a1e6180d8f0
Submitter: Zuul
Branch: master

commit ea1cad2bedabc72604d935daeb359a1e6180d8f0
Author: Lucian Petrut <email address hidden>
Date: Thu Sep 13 14:17:54 2018 +0300

    Filesystem driver: add chunk size config option

    At the moment, the filesystem driver uses a hardcoded 64 KB chunk
    size, used when reading/writing image files.

    This can be extremely inefficient, especially when file shares are
    used.

    For this reason, this change will make it configurable.

    DocImpact

    Change-Id: I1559c03308d36ecf9305c7a72ad658ecd1f2dc76
    Closes-Bug: #1792869

Changed in glance-store:
status: In Progress → Fix Released

This issue was fixed in the openstack/glance_store 0.28.0 release.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers