<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 5, 2014 at 5:01 AM, Thierry Carrez <span dir="ltr"><<a href="mailto:thierry@openstack.org" target="_blank">thierry@openstack.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="">Dolph Mathews wrote:<br>
> A backport for the first bug is now in review:<br>
><br>
> <a href="https://bugs.launchpad.net/ossa/+bug/1348820" target="_blank">https://bugs.launchpad.net/ossa/+bug/1348820</a><br>
> master <a href="https://review.openstack.org/#/c/109747/" target="_blank">https://review.openstack.org/#/c/109747/</a> (merged)<br>
> stable/icehouse <a href="https://review.openstack.org/#/c/111772/" target="_blank">https://review.openstack.org/#/c/111772/</a><br>
<br>
</div>Ihar has a comment that you merged two patches in a single review.. any<br>
chance you could decouple the two and propose them separately ?<br>
<div class=""><br>
> But at the moment, we're actually still working to land complete fixes<br>
> for the second two in master.<br>
><br>
> <a href="https://bugs.launchpad.net/ossa/+bug/1349597" target="_blank">https://bugs.launchpad.net/ossa/+bug/1349597</a><br>
> master <a href="https://review.openstack.org/#/c/109820/" target="_blank">https://review.openstack.org/#/c/109820/</a> (gating at this moment;<br>
> this also should have been Closes-Bug)<br>
><br>
> <a href="https://bugs.launchpad.net/ossa/+bug/1347961" target="_blank">https://bugs.launchpad.net/ossa/+bug/1347961</a><br>
> master <a href="https://review.openstack.org/#/c/111106/" target="_blank">https://review.openstack.org/#/c/111106/</a> (gating at this moment)<br>
<br>
</div>Those two just landed. Could you propose backports for them now ? I'll<br>
get the approvals lined up.<br>
<br></blockquote><div><br></div><div>The fixes made to Juno are proposed as backports to stable/icehouse:<br><br><a href="https://review.openstack.org/#/c/111845/">https://review.openstack.org/#/c/111845/</a> - Don't override tox envdir for pep8<br>
<a href="https://review.openstack.org/#/c/112082/">https://review.openstack.org/#/c/112082/</a> - Add tests related to V2 token issued_at time changing<br><a href="https://review.openstack.org/#/c/111772/">https://review.openstack.org/#/c/111772/</a> - Fix for V2 token issued_at time changing<br>
<a href="https://review.openstack.org/#/c/112083/">https://review.openstack.org/#/c/112083/</a> - Correct revocation event test for domain_id<br><a href="https://review.openstack.org/#/c/112084/">https://review.openstack.org/#/c/112084/</a> - Fix revoking domain-scoped tokens<br>
<a href="https://review.openstack.org/#/c/112085/">https://review.openstack.org/#/c/112085/</a> - Add a test for revoking a scoped token from an unscoped<br><a href="https://review.openstack.org/#/c/112086/">https://review.openstack.org/#/c/112086/</a> - Fix revoking a scoped token from an unscoped token<br>
<a href="https://review.openstack.org/#/c/112102/">https://review.openstack.org/#/c/112102/</a> - Make test_revoke expiry times distinct<br><a href="https://review.openstack.org/#/c/112087/">https://review.openstack.org/#/c/112087/</a> - Fix revocation event handling with MySQL<br>
<br></div><div>There were some minor-looking conflicts. There's also an issue where the tox envdir was shared and this was causing reviews to fail (111845), and there had been an earlier fix for the revocation tests that I picked up to get the unit tests to pass (112102).<br>
</div><div><br></div><div>- Brant<br></div><div><br></div></div></div></div>