<div dir="ltr"><div>I thought the 09 Feb 2021 review jam was highly valuable.<br></div><div><br></div><div>Without the discussions we had, I think the "Secure RBAC" patch set would be 
unapproachable for me. For example, having knowledge of the (new) 
oslo-policy features that the patches make use of seems to be a 
requirement for deeply understanding the changes. As a direct result of the review jam [0], I feel that I have enough understanding and comfortability to make valuable review feedback on these patches.<br><br></div><div>[0] and also having read/reviewed the secure-rbac spec previously, to be fair<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 5, 2021 at 7:10 AM Julia Kreger <<a href="mailto:juliaashleykreger@gmail.com">juliaashleykreger@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">In the Ironic team's recent mid-cycle call, we discussed the need to<br>
return to occasionally having review jams in order to help streamline<br>
the review process. In other words, get eyes on a change in parallel<br>
and be able to discuss the change. The goal is to help get people on<br>
the same page in terms of what and why. Be on hand to answer questions<br>
or back-fill context. This is to hopefully avoid the more iterative<br>
back and forth nature of code review, which can draw out a long chain<br>
of patches. As always, the goal is not perfection, but forward<br>
movement especially for complex changes.<br>
<br>
We've established two time windows that will hopefully not to be too<br>
hard for some contributors to make it to. It doesn't need to be<br>
everyone, but it would help for at least some people whom actively<br>
review or want to actively participate in reviewing, or whom are even<br>
interested in a feature to join us for our meeting.<br>
<br>
I've added an entry on to our wiki page to cover this, with the<br>
current agenda and anticipated review jam topic schedule. The tl;dr is<br>
we will use meetpad[1] and meet on Mondays at 2 PM UTC and Tuesdays at<br>
6 PM UTC. The hope is to to enable some overlap of reviewers. If<br>
people are interested in other times, please bring this up in the<br>
weekly meeting or on the mailing list.<br>
<br>
I'm not sending out calendar invites for this. Yet. :)<br>
<br>
See everyone next week!<br>
<br>
-Julia<br>
<br>
[0]: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__wiki.openstack.org_wiki_Meetings_Ironic-23Review-5FJams&d=DwIBaQ&c=sWW_bEwW_mLyN3Kx2v57Q8e-CRbmiT9yOhqES_g_wVY&r=OsbscIvhVDRWHpDZtO7nXdqGCfPHirpVEemMwL8l5tw&m=S4p8gD_wQlpR_rvzdqGkdq574-DkUsgBRet9-k3RpVg&s=gVApbMsmNPVlfYreqkQe4yKFxC66U6D8nFc_TwjW-FE&e=" rel="noreferrer" target="_blank">https://urldefense.proofpoint.com/v2/url?u=https-3A__wiki.openstack.org_wiki_Meetings_Ironic-23Review-5FJams&d=DwIBaQ&c=sWW_bEwW_mLyN3Kx2v57Q8e-CRbmiT9yOhqES_g_wVY&r=OsbscIvhVDRWHpDZtO7nXdqGCfPHirpVEemMwL8l5tw&m=S4p8gD_wQlpR_rvzdqGkdq574-DkUsgBRet9-k3RpVg&s=gVApbMsmNPVlfYreqkQe4yKFxC66U6D8nFc_TwjW-FE&e=</a> <br>
[1]: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__meetpad.opendev.org_ironic&d=DwIBaQ&c=sWW_bEwW_mLyN3Kx2v57Q8e-CRbmiT9yOhqES_g_wVY&r=OsbscIvhVDRWHpDZtO7nXdqGCfPHirpVEemMwL8l5tw&m=S4p8gD_wQlpR_rvzdqGkdq574-DkUsgBRet9-k3RpVg&s=iHBy7h99FQZ6Xb_fN2Hv3HZXIANl6BzR867jblUJvsk&e=" rel="noreferrer" target="_blank">https://urldefense.proofpoint.com/v2/url?u=https-3A__meetpad.opendev.org_ironic&d=DwIBaQ&c=sWW_bEwW_mLyN3Kx2v57Q8e-CRbmiT9yOhqES_g_wVY&r=OsbscIvhVDRWHpDZtO7nXdqGCfPHirpVEemMwL8l5tw&m=S4p8gD_wQlpR_rvzdqGkdq574-DkUsgBRet9-k3RpVg&s=iHBy7h99FQZ6Xb_fN2Hv3HZXIANl6BzR867jblUJvsk&e=</a> <br>
<br>
</blockquote></div>