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