<!DOCTYPE html>
<html>
<head>
<title></title>
</head>
<body><div> </div>
<div> </div>
<div> </div>
<div>On Tue, Jun 28, 2016, at 03:26 AM, Zhenyu Zheng wrote:<br></div>
<blockquote type="cite"><div dir="ltr"><div>Hi all,<br></div>
<div> </div>
<div>I'm working on add pagination and timestamp filter for os-instance-actions API:<br></div>
<div><a href="https://review.openstack.org/#/c/326326/" defang_rel="noreferrer" defang_data-ss1467116725="1">https://review.openstack.org/#/c/326326/</a><br></div>
<div>As Alex_xu pointed out that it will be better to filter by `updated_at` for timestamp filter which is reasonable, but when I tried to modify the patch I found out that:<br></div>
<div> </div>
<div>1. The current APIs only called _record_action_start (objects.InstanceAction.action_start) and never call action_finish, so <br></div>
<div>the field of `finish_time` is always empty in instance_actions table;<br></div>
</div>
</blockquote><div> </div>
<div>There was a spec proposed to address this, though I don't believe it was approved for Newton. So for now you have to assume this will continue to be empty.</div>
<div> </div>
<blockquote type="cite"><div dir="ltr"><div> </div>
<div>2. The updated_at field is also empty, should we sync the updated_at time to the created_at time when we create the action and also update it whenever the action status changed, e.g finished.<br></div>
</div>
</blockquote><div> </div>
<div>When a finish_time is recorded that should definitely also update updated_at. I would be in favor of having updated_at set when the instance action is created. I've never fully understood why Nova doesn't do that generally.</div>
<div> </div>
<blockquote type="cite"><div dir="ltr"><div> </div>
<div>Thanks,<br></div>
<div>Kevin Zheng<br></div>
</div>
<div><u>__________________________________________________________________________</u><br></div>
<div>OpenStack Development Mailing List (not for usage questions)<br></div>
<div>Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe<br></div>
<div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" defang_rel="noreferrer" defang_data-ss1467116725="1">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
</blockquote><div> </div>
</body>
</html>