<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="Title" content="">
<meta name="Keywords" content="">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@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:0cm;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-reply;
font-family:Calibri;
color:windowtext;}
span.msoIns
{mso-style-type:export-only;
mso-style-name:"";
text-decoration:underline;
color:teal;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body bgcolor="white" lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri">Hi Yuriy,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri">Thanks for bringing this up. I'm good with your list, with the exception of driver_info and instance_info. I'm on the fence with these two. If we assume that any secrets will be bleep'd
out (configdrives won't be there), is there other information there that might be useful? I'm not totally sure what notifications will be used for; it is somewhat hard to assume.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri">I suppose we could look at it this way, since you and Mario are fine without those two. If no one speaks up wanting them, then we'll do as you propose, and not expose those two fields.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri">--ruby<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:Calibri"><o:p> </o:p></span></p>
<blockquote style="border:none;border-left:solid #B5C4DF 4.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-right:0cm">
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-family:Calibri;color:black">From: </span>
</b><span style="font-family:Calibri;color:black">Yuriy Zveryanskyy <yzveryanskyy@mirantis.com><br>
<b>Reply-To: </b>"OpenStack Development Mailing List (not for usage questions)" <openstack-dev@lists.openstack.org><br>
<b>Date: </b>Tuesday, September 27, 2016 at 7:00 AM<br>
<b>To: </b>"openstack-dev@lists.openstack.org" <openstack-dev@lists.openstack.org><br>
<b>Subject: </b>[openstack-dev] [ironic] base node payload for notification<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal">Hi,<o:p></o:p></p>
</div>
<p class="MsoNormal">there is a discussion starting in comment on <a href="https://review.openstack.org/#/c/321865/">
https://review.openstack.org/#/c/321865/</a><o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">I agree with Ruby Loo proposal about a base node payload.<o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">Currently we have these node's fields exposed via API (in alphabetical order):<br>
<br>
"chassis_uuid", "clean_step", "console_enabled", "created_at", "driver",<br>
"driver_info", "driver_internal_info", "extra", "inspection_finished_at",<br>
"inspection_started_at", "instance_info", "instance_uuid", "last_error",<br>
"maintenance", "maintenance_reason", "name", "network_interface",<br>
"power_state", "properties", "provision_state", "provision_updated_at",<br>
"raid_config", "reservation", "resource_class", "target_power_state",<br>
"target_provision_state", "target_raid_config", "updated_at", "uuid"<o:p></o:p></p>
</div>
<p class="MsoNormal">In my opinion these field should be excluded from base node payload:<br>
<br>
"chassis_uuid": it not represents node state, not changed too often, additional<o:p></o:p></p>
</div>
<p class="MsoNormal">DB SELECT will be needed for base payload<br>
"driver_info": it not represents node state, contains only driver settings and<o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">secrets like IPMI passwords<br>
"driver_internal_info": it's driver internal info<br>
"instance_info": configdrive blob can be saved inside<br>
"raid_config": it's hardware related<br>
"reservation": it's not independent changed fields, only lock flag<br>
"target_raid_config": it's hardware related<o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">And resulting base payload fields list (for version 1.0):<br>
<br>
"clean_step", "console_enabled", "created_at", "driver", "extra",<br>
"inspection_finished_at", "inspection_started_at", "instance_uuid",<br>
"last_error", "maintenance", "maintenance_reason", "name",<br>
"network_interface", "power_state", "properties", "provision_state",<br>
"provision_updated_at", "resource_class", "target_power_state",<br>
"target_provision_state", "updated_at", "uuid"<br>
<br>
Any other suggestions are welcome.<o:p></o:p></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt">Yuriy Zveryanskyy<o:p></o:p></p>
</div>
</blockquote>
</div>
</body>
</html>