<div dir="ltr">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.<div>-Anish</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 14, 2016 at 11:33 AM, <span dir="ltr"><<a href="mailto:Christopher.Dearborn@dell.com" target="_blank">Christopher.Dearborn@dell.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="#0563C1" vlink="#954F72">
<div>
<p class="MsoNormal">I vote for parsing everything the same way.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Note that the unique identifier for these attributes is InstanceID. See
<a href="https://www.vmware.com/support/developer/cim-sdk/smash/u2/ga/apirefdoc/CIM_BIOSString.html" target="_blank">
https://www.vmware.com/<wbr>support/developer/cim-sdk/<wbr>smash/u2/ga/apirefdoc/CIM_<wbr>BIOSString.html</a> 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.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Chris<u></u><u></u></p><div><div class="h5">
<p>-----Original Message-----<br>
From: Miles Gould [mailto:<a href="mailto:mgould@redhat.com" target="_blank">mgould@redhat.com</a>] <br>
Sent: Wednesday, September 14, 2016 7:46 AM<br>
To: OpenStack Development Mailing List (not for usage questions) <br>
Subject: Re: [openstack-dev] [ironic] Extending python-dracclient to fetch System/iDrac resources<br>
<br>
On 13/09/16 20:30, Anish Bhatt wrote:<br>
> Is parsing iDrac/System attributes differently from BIOS attributes <br>
> the correct approach here (this will also make it match racadm <br>
> output), or should I be changing all Attributes to be parsed the same way ?<br>
<br>
"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?<br>
<br>
Miles<br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><u></u><u></u></p>
</div></div></div>
</div>
<br>______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">One socket to bind them all</div></div>
</div>