<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On 25/08/2017 5:41 AM, "Albert Astals Cid" <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">El dijous, 24 d’agost de 2017, a les 21:07:49 CEST, Ben Cooksley va escriure:<br>
<div class="quoted-text">> Hi all,<br>
><br>
> The following is Sysadmin's suggested plan for the retirement of<br>
> Reviewboard now that Phabricator is fully up and running for hosting<br>
> of code reviews.<br>
><br>
> Phase 1: Commences September 2: All repositories are closed for<br>
> accepting new reviews on Reviewboard. A notice is added to the top of<br>
> the main page indicating that reviews should now be done on<br>
> Phabricator.<br>
><br>
> Phase 2: Commences September 16: Login to Reviewboard is disabled, and<br>
> final backups are taken. A static copy of Reviewboard is generated and<br>
> published online, and the software itself is taken down.<br>
<br>
</div>Does this mean i can still see:<br>
 * Diffs<br>
 * emails of the people that made those diffs<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Diffs should definitely be - as long as they're not broken due to the issue I noted. </div><div dir="auto"><br></div><div dir="auto">As you wouldn't be able to access them currently there is no change in that regard.</div><div dir="auto"><br></div><div dir="auto">In terms of email addresses this will only be the case if Reviewboard displays them publicly. Their username will still be shown though which we can lookup on Identity.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Ben</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
After phase 2?<br>
<br>
Cheers,<br>
  Albert<br>
<div class="elided-text"><br>
><br>
> The vast majority of projects should now be migrated to Phabricator,<br>
> with only historical reviews needing to be cleaned up.<br>
><br>
> Note that due to how Reviewboard stores diffs and reproduces them for<br>
> use, some reviews may have decayed and may no longer be readable. This<br>
> is due to short-hashes which are used by Git/Reviewboard in diffs now<br>
> having collisions with other commits which previously did not exist.<br>
> Unfortunately there is nothing we can do about this.<br>
><br>
> Any comments on the above?<br>
><br>
> Regards,<br>
> Ben<br>
<br>
<br>
</div></blockquote></div><br></div></div></div>