expand image checksum type support about baremetal-operator HOT 10 CLOSED

cc @juliakreger @dtantsur

from baremetal-operator.

if translated directly to ironic's image_checksum it is assumed to be md5. Two different fields are used for other checksums: 'image_os_hash_algo' and 'image_os_hash_value'. FWIW, it requires a stein IPA ramdisk. The URL support, @dtantsur might be best to doublecheck because I think it has more support, but I don't remember offhand.

from baremetal-operator.

It would be nice to revive this as in combination with https://storyboard.openstack.org/#!/story/2006936 we could deploy the OKD images directly (they are shipped as xz compressed qcow2 images, with an sha256sum)

I think specifying the type of checksum in the existing field e.g sha256:xxxx seems reasonable and keeps the interface terse but still flexible - any objections to that approach?

We could retain the existing behavior by default by assuming md5 when no prefix is given?

@dtantsur can you confirm if image_os_hash_value works with the URL support we now rely on?

from baremetal-operator.

Hi, I have a question. Is the plan to still keep "image_checsum" in addition to a combo of "image_os_hash_algo:image_os_hash_value" or completely replace "image_checksum" with the latter?

from baremetal-operator.

Hi, I have a question. Is the plan to still keep "image_checsum" in addition to a combo of "image_os_hash_algo:image_os_hash_value" or completely replace "image_checksum" with the latter?

Perhaps @juliakreger can answer that, but I'd assume we continue using the existing interface for md5 checksums (e.g the default case) unless we're advised otherwise.

from baremetal-operator.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues will close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle stale

from baremetal-operator.

/remove-lifecycle stale

from baremetal-operator.

Fixed in #401
/close.

from baremetal-operator.

/close

from baremetal-operator.

@maelk: Closing this issue.

from baremetal-operator.