[Cinder][Ceph] TestVM image format

Bug #1584062 reported by Ivan Kolodyazhny
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Confirmed
Medium
Fuel Sustaining
9.x
Won't Fix
Medium
Registry Administrators

Bug Description

According to bug #1568898 [1] we have to upload TestVM image in raw format in case, when both Cinder and Glance use Ceph back-end

[1] https://bugs.launchpad.net/mos/+bug/1568898

Ivan Kolodyazhny (e0ne)
Changed in mos:
assignee: nobody → MOS Puppet Team (mos-puppet)
milestone: none → 10.0
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :

The TestVM image is about 10-20 MBs in size. Do we really care here, if Cinder automatically performs conversion when needed? (based on the description of https://bugs.launchpad.net/mos/+bug/1568898)

Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

IMO, we have to do it because we test with this image in scale. Also, we have to follow our own guidelines in deployment

Revision history for this message
Dina Belova (dbelova) wrote :

This looks as medium issue to be fixed for test purposes. Due to medium priority, marking as won't fix for 9.0

Changed in mos:
assignee: MOS Puppet Team (mos-puppet) → Alexey Deryugin (velovec)
Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :

Puppet installs cirros image from the debian package http://perestroika-repo-tst.infra.mirantis.net/mos-repos/ubuntu/master/pool/main/c/cirros-testvm/ and that's it. I think this problem should be addressed to MOS Linux team

Changed in mos:
assignee: Alexey Deryugin (velovec) → MOS Linux (mos-linux)
Revision history for this message
Dmitry Teselkin (teselkin-d) wrote :

testvm package have no idea about how the image will be used further, so adding the same image repackaged in another format isn't a solution here. Converting to raw should be done conditionally from puppet.

Changed in mos:
assignee: MOS Linux (mos-linux) → MOS Puppet Team (mos-puppet)
Revision history for this message
Denis Egorenko (degorenko) wrote :

I agree with Ivan: puppet should not convert any existed resource to a new format. From my POV, the most appropriate solution is to fix nailgun (update astute.yaml if ceph is enabled with addition RAW image info) and add RAW image to cirros-testvm package or add new cirros-testvm-raw package.

Otherwise, puppet-convert solution looks like hardcode.

Changed in mos:
assignee: MOS Puppet Team (mos-puppet) → Fuel Sustaining (fuel-sustaining-team)
Changed in mos:
milestone: 9.0 → 10.0
no longer affects: mos/10.0.x
tags: added: area-ceph area-cinder area-library
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.