I vote for parsing everything the same way.

Note that the unique identifier for these attributes is InstanceID.  See 
https://www.vmware.com/support/developer/cim-sdk/smash/u2/ga/apirefdoc/CIM_BIOSString.html
 for example.  I think it's fine to include GroupID and whatever attributes are 
needed, but InstanceID should be used as the key.  We shouldn't be making up 
other keys to use since we already have a perfectly good one that's being 
supplied.

Chris

-----Original Message-----
From: Miles Gould [mailto:mgo...@redhat.com]
Sent: Wednesday, September 14, 2016 7:46 AM
To: OpenStack Development Mailing List (not for usage questions)
Subject: Re: [openstack-dev] [ironic] Extending python-dracclient to fetch 
System/iDrac resources

On 13/09/16 20:30, Anish Bhatt wrote:
> Is parsing iDrac/System attributes differently from BIOS attributes
> the correct approach here (this will also make it match racadm
> output), or should I be changing all Attributes to be parsed the same way ?

"Parse everything the same way" sounds like the simpler and less brittle 
option; is there a good reason *not* to consider GroupID for BIOS attributes?

Miles

__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

Reply via email to