<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 30, 2020 at 1:38 PM Chris Lattner via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">I want to bubble out of this discussion, because most of the conversation has been about merit of various tools, how much the cloud license costs, etc.<br>
<br>
<br>
In my opinion, none of this actually matters.  There are much larger strategic questions that we should be talking about instead:<br>
<br>
1) Why is LLVM special?  We are a tiny community compared to the larger GitHub community - anything that makes us “weird” (even if weird is better in some ways) increases impedance mismatch, reduces collaboration with other communities etc.  Many users of LLVM are already using GitHub for code reviews and PRs etc. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
2) Why should compiler nerds :-) be working on this sort of infrastructure?  We have many talented people who are capable of doing many impressive things, why spent that energy on this?<br>
<br>
3) Even if someone is willing to keep this going, what ongoing liability is this for the project as a whole?  Phabricator was done for the weekend, did someone’s pager go off?  What is the SLA/SLO for the new system?<br>
<br>
As I mentioned upthread, I like Phabricator and its workflow from a personal perspective, but from a project perspective, I can’t see any reason to defend bespoke infra like this.<br></blockquote><div><br></div><div>From a project point of view: the productivity of the developers on the project seems like a major reason to support this infra. </div><div>I'd rather work on writing compiler infra than dev infra, but I still spent a few hours this week-end on setting up a cloud VM to migrate our Phab to it ; because until we have a good story for Phab features we rely on today it does not seem reasonable to me to *rush* the migration to GitHub.</div><div>That said I'm interested to hear how Swift developers (or Rust, or...) are adjusting their workflow to not depend on Herald for example? What about the review dashboard that you have as a landing page on Phab (and that is customizable)? </div><div>Having an answer to all these would help planning a timely migration to GitHub PRs.</div><div><br></div><div>As of the current Phab instance: it has been running for 6 years and we did pretty well with the current setup. It does not seem like an urgent problem to address SLA/SLO just right now. We actually know why the system has some issues like this Saturday: the root partition has 10GB and the apache2 logs can get quite large (I zipped a log file on the server this weekend when folks reported trouble on Saturday, and I'm looking into more maintenance on this topic next weekend).</div><div><br></div><div>-- </div><div>Mehdi</div><div><br></div><div><br></div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
-Chris<br>
<br>
<br>
<br>
<br>
> On Jun 29, 2020, at 4:11 AM, Nicolai Hähnle via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> <br>
> On Fri, Jun 26, 2020 at 9:54 AM James Henderson via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>> 2) Typically in our Phabricator, the description and summary of a patch is considered to be roughly what goes in the final commit message (same with Github). However, updating said commit message isn't possible to my knowledge without force-pushing in Github (which as discussed elsewhere in the thread causes other problems).<br>
> <br>
> Rebasing _should_ be how commit messages are changed. Ideally, we'd<br>
> treat commit messages as part of the artefact under review, since good<br>
> commit messages matter. Phabricator isn't ideal here either: by<br>
> default, updating a patch doesn't update its commit message.<br>
> <br>
> On the GitHub side, the real problem here is how easily it loses the<br>
> plot when you rebase.<br>
> <br>
> <br>
>> 3) Marking comments as "Done" in Phabricator does not auto-hide them, whereas in Github marking a comment as "Resolved" does. Spoken from experience, this leads to situations in Github where a developer thinks they've resolved a reviewers comments, marks them as Resolved, but actually they weren't. The reviewer in turn then has to browse the comments in the summary page, to see if they have any unaddressed comments that were supposedly resolved, which given the limited context available there is a non-trivial task sometimes.<br>
> <br>
> Ideally, the "Resolved" state should be per-user. When multiple people<br>
> review a patch, I might not want to duplicate a comment that another<br>
> reviewer made, but I would like to confirm for myself whether an issue<br>
> was resolved or not.<br>
> <br>
> <br>
>> P.S. Thanks Mehdi/Fangrui for stepping up! Whilst I'm not opposed to working with Github towards getting the feature parity with Phabricator sorted, I don't want to switch until the two are on a par with each other, which in my opinion is not yet, so being forced to do so prematurely due to lack of maintainers would have made me very sad (P.P.S thank you Manuel for Phabricator in the first place!).<br>
> <br>
> Seconded on all counts!<br>
> <br>
> Cheers,<br>
> Nicolai<br>
> <br>
> -- <br>
> Lerne, wie die Welt wirklich ist,<br>
> aber vergiss niemals, wie sie sein sollte.<br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>