<!DOCTYPE html>
<html>
<head>
<title></title>
</head>
<body><div> </div>
<div> </div>
<div> </div>
<div>On Tue, Jun 28, 2016, at 09:27 PM, Zhenyu Zheng wrote:<br></div>
<blockquote type="cite"><div dir="ltr">How about I sync updated_at and created_at in my patch, and leave the finish to the other BP, by this way, I can use updated_at for the timestamp filter I added and it don't need to change again once the finish BP is complete.<br></div>
</blockquote><div> </div>
<div>Sounds good to me.</div>
<div> </div>
<blockquote type="cite"><div><div> </div>
<div defang_data-gmailquote="yes"><div>On Tue, Jun 28, 2016 at 8:28 PM, Andrew Laski <span dir="ltr"><<a href="mailto:andrew@lascii.com" defang_rel="noreferrer" defang_data-ss1467200425="1">andrew@lascii.com</a>></span> wrote:<br></div>
<blockquote style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204, 204, 204);padding-left:1ex;" defang_data-gmailquote="yes"><div><u></u><br></div>
<div><div> </div>
<div> <br></div>
<div> <br></div>
<div> <br></div>
<div><span>On Tue, Jun 28, 2016, at 03:26 AM, Zhenyu Zheng wrote:</span><br></div>
<blockquote type="cite"><div dir="ltr"><div><span>Hi all,</span><br></div>
<div> <br></div>
<div><span>I'm working on add pagination and timestamp filter for os-instance-actions API:</span><br></div>
<div><span><a href="https://review.openstack.org/#/c/326326/" defang_rel="noreferrer" defang_data-ss1467200425="1">https://review.openstack.org/#/c/326326/</a></span><br></div>
<div><span>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:</span><br></div>
<div> <br></div>
<div><span>1. The current APIs only called _record_action_start (objects.InstanceAction.action_start) and never call action_finish, so </span><br></div>
<div><span>the field of `finish_time` is always empty in instance_actions table;</span><br></div>
</div>
</blockquote><div> <br></div>
<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.<br></div>
<div> </div>
<div> <br></div>
<blockquote type="cite"><div dir="ltr"><div> <br></div>
<div><span>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.</span><br></div>
</div>
</blockquote><div> <br></div>
<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.<br></div>
<div> <br></div>
<blockquote type="cite"><div dir="ltr"><div> <br></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: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" defang_rel="noreferrer" defang_data-ss1467200425="1">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br></div>
<div><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" defang_rel="noreferrer" defang_data-ss1467200425="1">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
</blockquote><div> <br></div>
</div>
<div> </div>
<div>__________________________________________________________________________<br></div>
<div> OpenStack Development Mailing List (not for usage questions)<br></div>
<div> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" defang_rel="noreferrer" defang_data-ss1467200425="1">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br></div>
<div> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" defang_rel="noreferrer" defang_data-ss1467200425="1">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
<div> </div>
</blockquote></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-ss1467200425="1">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br></div>
</blockquote></body>
</html>