ooi

resource_tpl mixins missing default values

Bug #1687943 reported by Boris Parak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ooi
Fix Released
High
Alvaro Lopez

Bug Description

Mixins depending on resource_tpl should specify default attribute values at least for important attributes:

occi.compute.cores
occi.compute.memory

This information is crucial to the user.

Alvaro Lopez (aloga)
Changed in ooi:
importance: Undecided → High
milestone: none → 1.2.0
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ooi (master)

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

Changed in ooi:
assignee: nobody → Enol Fernández (enolfc)
status: Confirmed → In Progress
Revision history for this message
Boris Parak (araxxxx) wrote :

According to [1], there is also a numeric attribute

occi.compute.ephemeral_storage.size

which should be used to indicate the size of ephemeral storage attached to an instance. Although, I'm not sure if it should replace

org.openstack.flavor.disk

or

org.openstack.flavor.ephemeral

[1] https://www.ogf.org/documents/GFD.222.pdf

Revision history for this message
Enol Fernández (enolfc) wrote :

Actually, ooi does not implement the "OCCI Compute Resource Templates Profile" defined at GWD-R-P.222. Is it mandatory?

Revision history for this message
Boris Parak (araxxxx) wrote :

It is not necessary to implement the profile (the actual sizes mentioned in the profile). However, if you are publishing flavor information as attributes, it would be useful to match them to as many existing OCCI attributes as possible. For the sake of usability. If we can agree at least on cores, memory, and disk size, we will be able to report useful information to users via CLI (without forcing them to interrogate external services). So, it is more about attribute names than actual sizes.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ooi (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/490010

Changed in ooi:
assignee: Enol Fernández (enolfc) → Alvaro Lopez (aloga)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ooi (master)

Reviewed: https://review.openstack.org/486558
Committed: https://git.openstack.org/cgit/openstack/ooi/commit/?id=8500d14450abe0faae8d65e64aa20f29bc9c594b
Submitter: Jenkins
Branch: master

commit 8500d14450abe0faae8d65e64aa20f29bc9c594b
Author: Enol Fernandez <email address hidden>
Date: Mon Jul 24 11:25:59 2017 +0100

    Add default values to resource templates

    Change-Id: I55e5b66cdc1d861651658d8e3bad204618ad2e35
    Closes-Bug: 1687943

Changed in ooi:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ooi (master)

Reviewed: https://review.openstack.org/490010
Committed: https://git.openstack.org/cgit/openstack/ooi/commit/?id=54d97ffcda7409c06829780578b0b7f7f4417939
Submitter: Jenkins
Branch: master

commit 54d97ffcda7409c06829780578b0b7f7f4417939
Author: Enol Fernandez <email address hidden>
Date: Wed Aug 2 13:44:58 2017 +0100

    Add occi.compute.ephemeral_storage.size attribute

    Implemented as an alias of org.openstack.flavor.ephemeral so both can be
    used.

    Change-Id: I8764af5778d73b3a9e112cfff97be7efdda6af33
    Related-Bug: 1687943

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.