OS::Glance::Image missing several properties

Bug #1606150 reported by Johannes Grassler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
Undecided
Dmitriy

Bug Description

The OS::Glance::Image resource is missing the following properties (all supported by python-glanceclient as of version 2.0.0 (that's the one I used for comparison):

  --architecture <ARCHITECTURE>
                        Operating system architecture as specified in
                        http://docs.openstack.org/trunk/openstack-
                        compute/admin/content/adding-images.html
  --visibility <VISIBILITY>
                        Scope of image accessibility Valid values: public,
                        private
  --kernel-id <KERNEL_ID>
                        ID of image stored in Glance that should be used as
                        the kernel when booting an AMI-style image.
  --os-distro <OS_DISTRO>
                        Common name of operating system distribution as
                        specified in http://docs.openstack.org/trunk
                        /openstack-compute/admin/content/adding-images.html
  --owner <OWNER> Owner of the image
  --ramdisk-id <RAMDISK_ID>
                        ID of image stored in Glance that should be used as
                        the ramdisk when booting an AMI-style image.

Apart from these there are two special cases:

  --property <key=value>
                        Arbitrary property to associate with image. May be
                        used multiple times.

It's probably sensible to have a map attribute named `properties` or something along these lines.

  --instance-uuid <INSTANCE_UUID>
                        Metadata which can be used to record which instance
                        this image is associated with. (Informational only,
                        does not create an instance snapshot.)

I don't think this needs to be supported in Heat, since it presents a bit of a chicken-egg problem: once you have an instance UUID to populate this property with, there already is an instance running this image.

Revision history for this message
Johannes Grassler (jgr-launchpad) wrote :

Note: I used Heat's master branch to determine the properties missing from heat.

Revision history for this message
Dmitriy (duvarenkov) wrote :

Support of '--property' is added in this https://review.openstack.org/#/c/318973/ patch (still on review).
I can work on adding other properties to OS::Glance::Image resource.

Changed in heat:
assignee: nobody → Dmitriy (duvarenkov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

Fix proposed to branch: master
Review: https://review.openstack.org/352487

Changed in heat:
status: New → In Progress
Revision history for this message
Dmitriy (duvarenkov) wrote :

--visibility is already implemented by is_public property by the way.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/352487
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=652a1a8b33246beb5fa18692f614274c2a03b841
Submitter: Jenkins
Branch: master

commit 652a1a8b33246beb5fa18692f614274c2a03b841
Author: Dmitriy Uvarenkov <email address hidden>
Date: Thu Aug 18 15:33:23 2016 +0300

    Add v2 glance image props

    Added properties, supported by glanceclient v2:
    architecture
    kernel_id
    os_distro
    ramdisk_id
    and also owner property, supported by glanceclient v1.

    Closes-Bug: #1606150
    Change-Id: I15b9295f631615c1d98d7872fb973d5e8ca226da

Changed in heat:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/heat 7.0.0.0b3

This issue was fixed in the openstack/heat 7.0.0.0b3 development milestone.

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.