<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none"><!--P{margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p>Hey,</p>
<p><br>
</p>
<p>I've updated the project [1]. There are a few PRs that Need Reviews and a few others that need to be updated by their owners (Need Changes column). Wink, wink.</p>
<p><br>
</p>
<p>Regards,<br>
</p>
<p>Andreea<br>
</p>
<p><br>
</p>
<p>[1] <a href="https://github.com/orgs/rust-vmm/projects/2">https://github.com/orgs/rust-vmm/projects/2</a><br>
</p>
<div style="word-wrap:break-word; line-break:after-white-space">
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> Jiang Liu <liuj97@gmail.com><br>
<b>Sent:</b> Tuesday, July 16, 2019 5:56 AM<br>
<b>To:</b> Florescu, Andreea<br>
<b>Cc:</b> rust-vmm ML<br>
<b>Subject:</b> Re: [Rust-VMM] Sync Call - Follow ups</font>
<div> </div>
</div>
<div>
<div class="" style="word-wrap:break-word; line-break:after-white-space">
<div class="" style="word-wrap:break-word; line-break:after-white-space">Thanks for great work, I like it:)<br class="">
<div class=""><br class="">
<blockquote type="cite" class="">
<div class="">On Jul 16, 2019, at 12:09 AM, Florescu, Andreea <<a href="mailto:fandree@amazon.com" class="">fandree@amazon.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div class="" style="margin-top:0px; margin-bottom:0px; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
I really want the title spammer of the year.</div>
<div class="" style="margin-top:0px; margin-bottom:0px; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
Here is the project I was mentioning in my previous email:<span class="Apple-converted-space"> </span><a href="https://github.com/orgs/rust-vmm/projects/2" class="">https://github.com/orgs/rust-vmm/projects/2</a><br class="">
</div>
<div dir="ltr" class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<hr tabindex="-1" class="" style="display:inline-block; width:911.390625px">
<div id="divRplyFwdMsg" dir="ltr" class=""><font class="" style="font-size:11pt" face="Calibri, sans-serif"><b class="">From:</b><span class="Apple-converted-space"> </span>Florescu, Andreea<br class="">
<b class="">Sent:</b><span class="Apple-converted-space"> </span>Monday, July 15, 2019 5:57 PM<br class="">
<b class="">To:</b><span class="Apple-converted-space"> </span>rust-vmm ML<br class="">
<b class="">Subject:</b><span class="Apple-converted-space"> </span>Sync Call - Follow ups</font>
<div class=""> </div>
</div>
<div class="">
<div class="" style="margin-top:0px; margin-bottom:0px">Hello everyone,</div>
<div class="" style="margin-top:0px; margin-bottom:0px"><br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px">Today we had an out of band sync call and we discussed quite a few things. The meeting notes are here:<a href="https://etherpad.openstack.org/p/rust_vmm_2019_biweekly_calls" class="">https://etherpad.openstack.org/p/rust_vmm_2019_biweekly_calls</a></div>
<div class="" style="margin-top:0px; margin-bottom:0px"><br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px">One of the highlights of the meeting was regarding the review process taking a long time. We discussed about how we can improve it including having smaller PRs, assign people as reviewers and others (full
list in meeting notes). As a follow-up, I also created a project [1] in GitHub at the organization level to track open Pull Requests and issues and set priorities for them. The project is currently empty, I will start updating it although I not sure we have
the right template for it right now. In the long run, it would be nice for submitters of issues/PRs to update the project as well so people can have a high overview of what is under development/what needs reviewers.<br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px"><br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px">We also discussed about having the initial PR on a crate with the following format:</div>
<div class="" style="margin-top:0px; margin-bottom:0px">- README.md file that contains the design/architecture of the crate, interactions with other crates and other useful information. Once we create a new crate and open a PR, the design discussion will move
from the community issue that introduces the crate to the PR. We can have multiple iterations on this and the conversation should be easier to follow.</div>
<div class="" style="margin-top:0px; margin-bottom:0px">- Any public traits or simple definition of public structures so people can have a grasp of how the implementation of the crate will continue.</div>
<div class="" style="margin-top:0px; margin-bottom:0px">- We can also link in the PR any PoC if that is necessary.</div>
<div class="" style="margin-top:0px; margin-bottom:0px">This should improve the review time as you don't need to look over thousands of lines of code at once.</div>
<div class="" style="margin-top:0px; margin-bottom:0px"><br class="">
</div>
<div class="" style="margin-top:0px; margin-bottom:0px">Regards,</div>
<div class="" style="margin-top:0px; margin-bottom:0px">Andreea<br class="">
</div>
</div>
</div>
<p class="" style="margin-top:0px; margin-bottom:0px; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
</p>
<div class="" style="margin-top:0px; margin-bottom:0px; font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<br class="">
Amazon Development Center (Romania) S.R.L. registered office: 27A Sf. Lazar Street, UBC5, floor 2, Iasi, Iasi County, 700045, Romania. Registered in Romania. Registration number J22/2621/2005.</div>
<span class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none; float:none; display:inline!important">_______________________________________________</span><br class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<span class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none; float:none; display:inline!important">Rust-vmm
mailing list</span><br class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<span class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none; float:none; display:inline!important"><a href="mailto:Rust-vmm@lists.opendev.org" class="">Rust-vmm@lists.opendev.org</a></span><br class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
<span class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none; float:none; display:inline!important"><a href="http://lists.opendev.org/cgi-bin/mailman/listinfo/rust-vmm" class="">http://lists.opendev.org/cgi-bin/mailman/listinfo/rust-vmm</a></span><br class="" style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:16px; font-style:normal; font-weight:normal; letter-spacing:normal; text-align:start; text-indent:0px; text-transform:none; white-space:normal; word-spacing:0px; background-color:rgb(255,255,255); text-decoration:none">
</div>
</blockquote>
</div>
<br class="">
</div>
</div>
</div>
</div>
<p></p>
<p><br>
Amazon Development Center (Romania) S.R.L. registered office: 27A Sf. Lazar Street, UBC5, floor 2, Iasi, Iasi County, 700045, Romania. Registered in Romania. Registration number J22/2621/2005.</p>
</body>
</html>