That would be pretty easy to do. I did not pick InstanceId simply because
the code wasn't using it right now. Whatever the choice, the chosen key is
simply used as a key in the results dict so all of this would be
transparent to the users anyways.
-Anish

On Wed, Sep 14, 2016 at 11:33 AM, <christopher.dearb...@dell.com> wrote:

> 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
>
>


-- 
One socket to bind them all
__________________________________________________________________________
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