<div dir="ltr"><div><div><div><div><div><div><div><div>Hi,<br></div>there is a discussion starting in comment on <a href="https://review.openstack.org/#/c/321865/">https://review.openstack.org/#/c/321865/</a><br></div>I agree with Ruby Loo proposal about a base node payload.<br><br></div>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"<br><br></div>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<br></div>DB SELECT will be needed for base payload<br>"driver_info": it not represents node state, contains only driver settings and<br></div>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<br><br></div>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.<br><br></div>Yuriy Zveryanskyy<br><br></div>