<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Wingdings;
        panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:.5in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle20
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
/* List Definitions */
@list l0
        {mso-list-id:615599229;
        mso-list-type:hybrid;
        mso-list-template-ids:1276391366 1104020748 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l0:level1
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        mso-ascii-font-family:Calibri;
        mso-fareast-font-family:Calibri;
        mso-hansi-font-family:Calibri;
        mso-bidi-font-family:"Times New Roman";}
@list l0:level2
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:"Courier New";}
@list l0:level3
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:Wingdings;}
@list l0:level4
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:Symbol;}
@list l0:level5
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:"Courier New";}
@list l0:level6
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:Wingdings;}
@list l0:level7
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:Symbol;}
@list l0:level8
        {mso-level-number-format:bullet;
        mso-level-text:o;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:"Courier New";}
@list l0:level9
        {mso-level-number-format:bullet;
        mso-level-text:;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;
        font-family:Wingdings;}
@list l1
        {mso-list-id:1274287290;
        mso-list-template-ids:-1350547484;}
@list l2
        {mso-list-id:1806696524;
        mso-list-type:hybrid;
        mso-list-template-ids:1104475884 67698703 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l2:level1
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level2
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level3
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l2:level4
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level5
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level6
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l2:level7
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level8
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l2:level9
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
ol
        {margin-bottom:0in;}
ul
        {margin-bottom:0in;}
--></style>
</head>
<body lang="EN-US" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal">Matt,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">I agree with your thoughts on making available both approaches. A couple things to consider:<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l2 level1 lfo4">Regarding disk failures: would these disks even be available to be selected for any sort of RAID if they’re bad? If not, we’d quickly hit an issue with a mismatched number of disks
 vs. what’s declared (at least today).<o:p></o:p></li><li class="MsoListParagraph" style="margin-left:0in;mso-list:l2 level1 lfo4">Can we allow other disk/raid-related config to be tuned? I’m thinking Stripe size, Read policy, Write policy?<o:p></o:p></li></ol>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Just some food for thought.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Regards,<o:p></o:p></p>
<p class="MsoNormal">Darren<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">"MCEUEN, MATT" <MM9745@att.com><br>
<b>Date: </b>Tuesday, June 1, 2021 at 10:04 AM<br>
<b>To: </b>Zainub Wahid <zainub.wahid@xflowresearch.com>, "airship-discuss@lists.airshipit.org" <airship-discuss@lists.airshipit.org>, "SPHICAS, PHIL G" <ps3910@att.com>, "OBERFIELD, GREG" <go4243@att.com>, "DEJAEGER, DARREN R" <dd118r@att.com>, "STEIN, JARED
 W" <js5175@att.com><br>
<b>Cc: </b>"Williams, J T" <J.T.Williams@dell.com>, Airship <airship@xflowresearch.com>, "richard_pioso@dell.com" <richard_pioso@dell.com><br>
<b>Subject: </b>RE: [Airship-discuss] RAID Config Implementation<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">Refreshing this in post-holiday inboxes:  Phil, Craig, Greg, Darren, (& anyone else with experience with RAID config today) – please take a look at my thoughts below and let me know if that lines up with what you would find most useful
 as well. Thanks!<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">  <o:p></o:p></p>
<p class="MsoNormal">Also, Merging Mahnoor’s comments with the email thread:<o:p></o:p></p>
<p class="MsoNormal">(quote)<o:p></o:p></p>
<p class="MsoNormal">Thank you for your email, Matt.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">About the Airship use cases:<o:p></o:p></p>
<p class="MsoNormal">1. The strongly declarative one would be easy to support via Metal3, to the best of my knowledge. This would be perfect as long as the Airship user is okay with finding out the disk and RAID controller names through other means (using iDRAC,
 for example). It would also be an easier and valuable implementation.<o:p></o:p></p>
<p class="MsoNormal">2. Metal3 will perform the disk selection itself (or delegate to Ironic). <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Providing some level of guarantee for the disk failures would contribute to a reliable user experience. For the declarative case, we can perhaps ensure that the specified disks are functional. For the specification with hints, I suggest
 making the disk capacity a metric that must always be satisfied (e.g. x GB always provided). <o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Input on these requirements is much appreciated, so we can carry the discussion forward with the Metal3 community and get to implementation. <o:p></o:p></p>
<p class="MsoNormal">Thank you so much!<o:p></o:p></p>
<p class="MsoNormal">(end quote)<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> MCEUEN, MATT <br>
<b>Sent:</b> Wednesday, May 26, 2021 4:17 PM<br>
<b>To:</b> Zainub Wahid <zainub.wahid@xflowresearch.com>; airship-discuss@lists.airshipit.org; SPHICAS, PHIL G <ps3910@att.com>; OBERFIELD, GREG <go4243@att.com>; DEJAEGER, DARREN R <dd118r@att.com>; STEIN, JARED W <js5175@att.com><br>
<b>Cc:</b> Williams, J T <J.T.Williams@dell.com>; Airship <airship@xflowresearch.com>; richard_pioso@dell.com<br>
<b>Subject:</b> RE: [Airship-discuss] RAID Config Implementation<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks for sharing & summarizing this, Zainub.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Phil, Craig, Greg, Darren, (& anyone else with experience with RAID config today) – here are my thoughts; please refine/agree/disagree with them so we can give the Metal3 community good feedback on the interface we need for declarative
 RAID config.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">First: long term, we want to be able to declaratively configure all the BIOS settings that need to be configured.  Steps like this RAID config are a valuable step in that direction.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Second: I think there are use cases for both of these setups, and I’d advocate for M3 facilitating both of them in the long run*:<o:p></o:p></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo3">Strongly declarative: I want a RAID-N array on ControllerX with disks [DiskY, DiskZ]<o:p></o:p></li><li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo3">Declarative with hints: I want a RAID-N array with two SSDs (and I don’t care what disk or what controller – you figure it out, M3!)<o:p></o:p></li></ol>
<p class="MsoNormal">* Rook supports something similar – “use all disks:yes/no”, or alternately “use these specific disks:…”
<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Third: I think that #1 is a good fit for our most common use cases, where we put a lot of work into a reference definition, and then stamp it out many times on
<i>identical</i> hardware.  This is also the simplest implementation and would probably get something usable into Airship’s hands fastest.<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Does that sound right to you all?<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">The other thing to consider is thresholding when disks are bad.  I.e. a config item that says something like, “tolerate disk failures as long as I get Xgb” or “as long as I get 3 disks” or something.  Do you think this is critical enough
 to be needed on Day 1?<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<p class="MsoNormal">Thanks!<o:p></o:p></p>
<p class="MsoNormal">Matt<o:p></o:p></p>
<p class="MsoNormal"> <o:p></o:p></p>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Zainub Wahid <<a href="mailto:zainub.wahid@xflowresearch.com">zainub.wahid@xflowresearch.com</a>>
<br>
<b>Sent:</b> Tuesday, May 25, 2021 11:50 AM<br>
<b>To:</b> <a href="mailto:airship-discuss@lists.airshipit.org">airship-discuss@lists.airshipit.org</a><br>
<b>Cc:</b> Williams, J T <<a href="mailto:J.T.Williams@dell.com">J.T.Williams@dell.com</a>>; Airship <<a href="mailto:airship@xflowresearch.com">airship@xflowresearch.com</a>>;
<a href="mailto:richard_pioso@dell.com">richard_pioso@dell.com</a><br>
<b>Subject:</b> [Airship-discuss] RAID Config Implementation<o:p></o:p></p>
</div>
<p class="MsoNormal"> <o:p></o:p></p>
<div>
<p class="MsoNormal">Hello everyone,<o:p></o:p></p>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">As discussed in today's meeting, this is related to the demo represented by Noor, last year, related to performing RAID configurations in Airship via Metal3 (and Ironic).<br>
Add RAID to example hardwareprofile (I50c82ce5) · Gerrit Code Review (<a href="https://urldefense.com/v3/__http:/opendev.org__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYyvEP2e5U$">opendev.org</a>)<br>
<br>
Through Airship, the user should be able to perform RAID configurations on their hardware. The user can specify the RAID controller names and their associated disks, along with the required RAID level, the desired number of disks and the amount of storage required.
<br>
Here is the proposal for specifying the RAID configuration for a bare metal host via Metal3:<br>
<a href="https://urldefense.com/v3/__https:/github.com/metal3-io/metal3-docs/pull/148/files__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYyGgo40Mo$">https://github.com/metal3-io/metal3-docs/pull/148/files</a><br>
<br>
In terms of the CRD, the current implementation of the RAID has the following sections in it, for hardware RAID:<br>
spec:<br>
  raid:<br>
    hardwareRAIDVolumes:<br>
      - sizeGibiBytes: 1000<br>
        level: 1<br>
        name: volume-name<br>
        rotational: true<br>
        numberOfPhysicalDisks: 2<br>
We propose to add fields into the CRD, in the following manner:<br>
spec:<br>
  raid:<br>
    hardwareRAIDVolumes:<br>
      - sizeGibiBytes: 1000<br>
        level: 1<br>
        name: volume-1<br>
        rotational: true<br>
        numberOfPhysicalDisks: 2<br>
        physicalDisks:<br>
          - diskName: Disk-1<br>
          - diskName: Disk-2<br>
        controller: primary-controller<br>
      - sizeGibiBytes: 2000<br>
        level: 0<br>
        name: volume-2<br>
        rotational: false<br>
        numberOfPhysicalDisks: 2<br>
        physicalDisks:<br>
          - diskName: Disk-3<br>
          - diskName: Disk-4<br>
        controller: secondary-controller-2<br>
<br>
However, it remains unclear how the user would have the in-depth knowledge of the RAID controllers available to them, along with the vendor-specific disk names.
<br>
Example disk names (Dell): <br>
['Disk.Bay.0:Enclosure.Internal.0-1:RAID.Intergrated.1-1', 'Disk.Bay.2:Enclosure.Internal.0-1:RAID.Integrated.1-1', 'Disk.Bay.3:Enclosure.Internal.0-1:RAID.Integrated.1-1',...]<br>
Example disk names (HP):<br>
['2I:1:1', '2I:1:2', '2I:1:3',...]<br>
<br>
Example RAID Controllers (Dell):<br>
[‘Boss-F1 RAID Controller’, 'Dell PowerEdge RAID Controller H740P', ' Dell PowerEdge RAID Controller H830',...]<br>
Example RAID  controller (HP):<br>
['HP Smart Array P400 Controller', 'HPE Smart Array E208i-a Controller', 'HPE Smart Array P816i-a Controller',...]<br>
<br>
One possible solution is to let the user specify the required RAID level, number of disks and storage capacity needed. Metal3 could then validate this request using the Ironic drivers (e.g. Redfish driver, iDRAC driver etc.) in order to ensure the required
 hardware is available in the cluster. If it is available, the RAID configuration can be applied. Ifnot, the user can be informed about the error.<br>
<br>
Another solution is that Airship can perform inventory of the storage subsystems available in the cluster, and so the user can construct a RAID configuration accordingly. Conversely, the logic to select the physical disks and associated controllers may be implemented
 in Airship, whilst the user just specifies the high level requirements.<br>
<br>
One issue is that Redfish requires the explicit RAID controller and disk names in order to do the RAID configuration, but it would be more convenient if the user did not have to know the low level details of their system.<br>
Here is the Redfish Action URI: {Base URI of target resource}/Actions/Volume.ChangeRAIDLayout<br>
(Provided in 6.109.4.2 of Redfish Resource and Schema Guide (<a href="https://urldefense.com/v3/__http:/dmtf.org__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYys3w9hc0$">dmtf.org</a>))<br>
<br>
The disk names can be generated using the Redfish logic constructing a human friendly identifier or label for hardware devices.
<a href="https://urldefense.com/v3/__https:/github.com/DMTF/Redfish-Tacklebox/blob/e429f70a79cfe288756618498ce485ab4be37131/redfish_utilities/inventory.py*L192__;Iw!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYyuJLYIII$">
https://github.com/DMTF/Redfish-Tacklebox/blob/e429f70a79cfe288756618498ce485ab4be37131/redfish_utilities/inventory.py#L192</a><br>
<br>
Disk selection logic needs to be implemented in the most appropriate place. Airship may require that the user be unaware of the storage subsystem details when requesting the RAID config. Ironic requires a vendor independent implementation for this purpose.<br>
<br>
Here is relevant work done in metal3 to support RAID configuration in baremetal servers, using the Fujitsu iRMC driver:<br>
<a href="https://urldefense.com/v3/__https:/github.com/metal3-io/baremetal-operator/issues/206__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYyAS9JYzU$">https://github.com/metal3-io/baremetal-operator/issues/206</a><br>
<a href="https://urldefense.com/v3/__https:/github.com/metal3-io/baremetal-operator/pull/292__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYym4_G3ew$">https://github.com/metal3-io/baremetal-operator/pull/292</a><br clear="all">
<o:p></o:p></p>
<div>
<div>
<div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Warm Regards,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
<p class="MsoNormal">Zainub Wahid<o:p></o:p></p>
<div>
<p class="MsoNormal">Network Design Engineer II,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:#222222">xFlow Research Inc.</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial",sans-serif;color:#222222">+923044506900</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><a href="mailto:zainub.wahid@xflowresearch.com" target="_blank"><span style="font-size:10.0pt;font-family:"Arial",sans-serif">zainub.wahid@xflowresearch.com</span></a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><a href="https://urldefense.com/v3/__http:/www.xFlowResearch.com__;!!BhdT!0FCxfaRXqTKY8FTua2QbHU5D5562OD_9g_BLbFq-DtlLGrsxsjRowGYyG48Ww0I$" target="_blank">www.xFlowResearch.com</a><o:p></o:p></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>