Basic support for image conversion

[OpenStack Infra]

https://review.openstack.org/159129
commit 1b144f4c12fd6da58d7c48348bf7bab5873388e9
Author: Flavio Percoco <email address hidden>
Date: Wed Feb 25 14:50:06 2015 +0100

    Basic support for image conversion

    This patch adds a task for image conversion. It uses `qemu-img` to
    convert images. This tool has support for several image formats.

    The current implementation converts images if and only if the operator
    has configured glance to do so. That is, the `convert_to_format` option
    has been set.

    There are few things about this patch that should be improved by
    follow-up patches. The first one is the fact that it relies on the
    entry_points order for task execution. Although this works, it is not
    the most flexible/controllable way to do it. The second thing is that it
    relies on the aforementioned configuration option to enable/disable the
    task (as in, it becomes a non-op). There should be an explicit way to
    enable/disable tasks.

    Since both things mentioned in the previous paragraph affect the task
    management in general, I've decided to leave the fix for a follow-up
    patch.

    DocImpact
    SecurityImpact

    Partially-implements blueprint: new-upload-workflow
    Partially-implements blueprint: basic-import-conversion

    Change-Id: I0196a6f327c0147f897ae051ee60a8cb11b8fd40

[Alexandra Settle (alexandra-settle)]

Flavio - could you please detail what is the documentation impact here, and what the necessary change is?

Thank you

[Anne Gentle (annegentle)]

Trying to do a little triage here. The code patch mentions "convert_to_format" but I don't find it on the docs site anywhere. I also checked out the juno, kilo, and liberty branches of openstack-manuals, and don't see convert_to_format. But since the config ref is auto-generated, I an surprised not to find it. Any other thoughts on this?

[Alexandra Settle (alexandra-settle)]

I see what you see :( nothing. Hoping for a bit of response from Flavio to confirm. Adding glance PTL Brian.

[Brian Rosmaita (brian-rosmaita)]

I think we have some conceptual information about tasks (in glance/doc/source/tasks.rst), but nothing about how to configure/use them on the admin side. The tasks API is admin-only by default, and may be impacted by the image import refactor happening during Pike. So probably best to do nothing about this for now.

[Alexandra Settle (alexandra-settle)]

On that note, I think I'll close this for now. It's not valid - we can revisit later.