From: Jeremiah Cox <jerecox@microsoft.com>
To: "Knop, Ryszard" <ryszard.knop@intel.com>,
stephano <stephano.cetola@linux.intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [edk2-announce] Research Request
Date: Tue, 27 Nov 2018 21:16:18 +0000 [thread overview]
Message-ID: <MWHPR21MB0176C5680A597A0026C711FCADD00@MWHPR21MB0176.namprd21.prod.outlook.com> (raw)
In-Reply-To: <01F0790E56F0534D8DCAD4AC5838792F618A554A@irsmsx111.ger.corp.intel.com>
That is a good data point, thank you Ryszard.
Do we have data on what it takes to deploy and operate Phabricator with Harbormaster or Jenkins? The up front development/deployment activity/costs and then also the ongoing patching/servicing/maintenance costs? Is Intel planning to provide this?
For Project Mu we are leveraging GitHub and Azure Dev Ops for gates & CI builds (free for OSS). We had this basically working in a day and is operating for free with all patching/maintenance provided by GitHub & Azure Dev Ops.
-----Original Message-----
From: Knop, Ryszard <ryszard.knop@intel.com>
Sent: Tuesday, November 27, 2018 1:34 AM
To: Jeremiah Cox <jerecox@microsoft.com>; stephano <stephano.cetola@linux.intel.com>; edk2-devel@lists.01.org
Subject: RE: [edk2] [edk2-announce] Research Request
To add on Phabricator not supporting Travis CI - since Travis works exclusively with GitHub and has zero interest in supporting anything else, there are other options, eg Harbormaster ("native" CI module in Phabricator) or Jenkins (as far as I'm aware, many teams at Intel already know Jenkins one way or another). For a public example, KDE hosts all their sources on a self-hosted Phabricator instance and does CI with Jenkins - see https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbuild.kde.org%2F&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284825543&sdata=4coAaUQSgmoxLC3SDsW1M0X0bu61hhWQJlP%2B1xyP%2FW0%3D&reserved=0 and https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fphabricator.kde.org%2F&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284825543&sdata=MJaEesRphYxAtZCSJ%2Fyz3ZwcT%2FmMBRGAYHL0GxD5KWw%3D&reserved=0 - so that's not a problem :)
-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Jeremiah Cox via edk2-devel
Sent: Monday, November 26, 2018 22:43
To: stephano <stephano.cetola@linux.intel.com>
Cc: edk2-devel@lists.01.org
Subject: Re: [edk2] [edk2-announce] Research Request
Feedback on GitHub as follows…
> 1. No Lock-In - What automated data export is available?
> We want to be able to leave and take all our data with us. "Data" here
> includes: review comments, pull requests / patches (including
> metadata), old (rejected) pull requests and metadata, issue tracker
> entries and comments (if issue tracker included). This archiving
> should be automated, not something we do by hand.
Untested, but might these all be easily satisfied by subscribing a mailing list to GitHub notifications?
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.github.com%2Farticles%2Fabout-notifications%2F%23watching-notifications&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284825543&sdata=g3nvEWQuUZzFBEeEpSq63lZLRoPwp06el3kiNmFjfQA%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fhelp.github.com%2Farticles%2Fabout-email-notifications%2F&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284825543&sdata=4k56B1IeZyK2f3d%2BX9D2q3FFpk0kHt4Jey1NlwYunzs%3D&reserved=0
Alternatively, the GitHub REST API appear to offer full export capability of all information & metadata:
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Fgit%2Fcommits%2F%23get-a-commit&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=JsmAP23PWWpvjgfH9XO3qR0OtW8unBujs3MCG7ABCfg%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Fpulls%2F%23list-pull-requests&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=5TgHzyJNYI2YxRASb4z5QCtui100Ftz25tVxQObytrs%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Fpulls%2Fcomments%2F%23list-comments-on-a-pull-request&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=iDB1PW%2B9G1Ywjj%2FFeLYtKmcBI3szuz9%2FX5rBtL0ZNxk%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Fissues%2Fevents%2F%23list-events-for-a-repository&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=1lqC1ZUraFJqs8AJm0Ffd2pJGdK0lA2RA0ADteB2msQ%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Fissues%2Fcomments%2F%23list-comments-on-an-issue&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=fzXJZ%2FQSLhhkAdkX5irJ6Kmu7ALwOBFXJ1J%2Fyp8hSFY%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Factivity%2Fevents%2F%23list-repository-events&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=SM1oYigZqo7hLhzpd9JC0ROmhy6mybhDsIUSVH7GBOU%3D&reserved=0
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.github.com%2Fv3%2Freactions%2F%23list-reactions-for-a-pull-request-review-comment&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=qFB2gEBJvJ5AoQeCypTrtMIhqzDxm2mdpjXjknMiacE%3D&reserved=0
* the above allows you to export all of the thumbs up/down, smileys, hearts ... that users have given to pull request & issue comments :)
> 2. Easy Administration - Are there any scripts or custom code required
> after initial setup? We would like to do as little customizing as possible.
Our interpretation of this bullet is to maximize developer productivity & minimize deployment & operations costs.
GitHub provides a ready-to-use, end-to-end solution. There are no servers for end-customers to patch & maintain.
GitHub is free for use by open source projects, and Microsoft is committed to continuing this tradition:
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.microsoft.com%2Fen-us%2FInvestor%2Fevents%2FFY-2018%2FMicrosoft-and-GitHub-Conference-Call&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=C1bH8GSrkJT%2FHyfVJ3z86TbjxGw%2FaFtYVX3sd%2FKzyOk%3D&reserved=0
GitHub’s enormous user base has motivated numerous developers to generate GitHub Apps that further enhance the GitHub experience.
> 3. Flexible Workflow - Can we use email patches / email review as well
> as pull requests / web UI review?**
> 3a. Can we can attach review comments to specific code *and* commit
> message locations?
> 3b. Are the comments faithfully translated to notification emails
> (including the locations in code the comment is addressing)?
> 3c. Are old topic branches (rejected or updated pull requests)
> available even after being rejected? (i.e. are they ever deleted?)
> 3d. Is plain text supported in code review comments?
> **To be clear, it is acceptable if the system handles only pull
> requests and a web UI. We do require, however, a *read-only* email
> notification system that thoroughly documents our process.
We propose that all review & issue tracking are through GitHub web, REST, or Graph APIs. Email becomes read-only for notification and archival only.
3A: Yes.
3B: From our testing this appears to be yes.
3C: GitHub can be configured to keep rejected and updated pull requests.
3D: Both plain text and markdown work
Some additional questions we feel are important:
* Does the workflow facilitate automated validation & PR-Gates?
GitHub: Yes
Phabricator: https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsecure.phabricator.com%2FT9456&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=8VcHl3KdEvHYJSuwjTSiCo74pzlw3VCBgbq%2FVwqeOUY%3D&reserved=0 : “Writing lots of integrations for third-party software is broadly something the upstream is not well equipped for.”
Travis-CI further declined support for Phabricator: https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftravis-ci%2Ftravis-ci%2Fissues%2F2143%23issuecomment-124150608&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=7LfJbdjJUFip0NHCCYLOfoM%2BCD9%2FI%2FzOFC7yJQyknzQ%3D&reserved=0 “we have no immediate plans to add this feature”
* Does workflow allow easy contribution process?
GitHub: Yes, well-known and well-documented
* Does it have comprehensive documentation?
GitHub: Yes
* Does it have a comprehensive programmatic API that enables extensibility, with numerous online examples?
GitHub: Yes
* Does workflow facilitate different server-enforced policies for different branches?
GitHub: Yes
Sincerely,
Jeremiah Cox
-----Original Message-----
From: stephano <stephano.cetola@linux.intel.com>
Sent: Tuesday, November 20, 2018 4:59 PM
To: Jeremiah Cox <jerecox@microsoft.com>
Cc: edk2-devel@lists.01.org; Sean Brogan <sean.brogan@microsoft.com>
Subject: Re: [edk2] [edk2-announce] Research Request
Thank you both for taking the time to add some insight to our discussions. Please see the list of questions here:
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.01.org%2Fpipermail%2Fedk2-devel%2F2018-November%2F032462.html&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=JtgEwui0tzWoxOLH7YSqclJo4IJUgD5pmIm2u%2B%2B0j%2BY%3D&reserved=0
These are a summary from our community meetings.
Enjoy the holiday!
Cheers,
Stephano
On 11/20/2018 3:47 PM, Jeremiah Cox wrote:
> Hi Stephano,
> Sean and I will put something together for GitHub by next Tuesday.
>
> Thank you,
> Jeremiah Cox (departing for Thanksgiving holiday... now...)
>
> -----Original Message-----
> From: edk2-devel <edk2-devel-bounces@lists.01.org> On Behalf Of
> stephano
> Sent: Wednesday, November 14, 2018 10:34 AM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [edk2-announce] Research Request
>
> We are currently researching several different options to help make contributing to TianoCore easier for the community. A big part of this effort will be enabling pull requests and allowing for a more customizable code review process.
>
> I am looking for members of the community willing to answer a few questions about these solutions to allow us to evaluate our options quickly. The options are:
>
> System/Tool Investigator
> Phabricator Rebecca Cran (thank you again :) )
> Github ???
> Gerrit ???
> Gitlab ???
>
> I have a list of questions that I can send out to each investigator.
> Assuming you are familiar with the software/system, these questions should be answerable with a couple hours of research, writing, and screenshots / examples.
>
> Thanks in advance for your help!
>
> -Stephano
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists
> .01.org%2Fmailman%2Flistinfo%2Fedk2-devel&data=02%7C01%7Cjerecox%4
> 0microsoft.com%7Caace5779691047a1809b08d64f4c7fb2%7C72f988bf86f141af91
> ab2d7cd011db47%7C1%7C0%7C636783587312509548&sdata=SOR9cdCLwmsl37RB
> S0fkk6a%2FIE1so1flDYG2%2BzjCBbQ%3D&reserved=0
>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.01.org%2Fmailman%2Flistinfo%2Fedk2-devel&data=02%7C01%7Cjerecox%40microsoft.com%7C2dc3cdfe68db4498136b08d6544b6cd6%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636789080284835552&sdata=t13IcLLPbGxmHxeQJAvs%2BrIoXbBk34KsJRUhIVPn010%3D&reserved=0
--------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
next prev parent reply other threads:[~2018-11-27 21:16 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-14 18:34 [edk2-announce] Research Request stephano
2018-11-20 23:47 ` Jeremiah Cox
2018-11-21 0:58 ` stephano
2018-11-26 21:43 ` Jeremiah Cox
2018-11-26 22:27 ` stephano
2018-11-27 9:33 ` Knop, Ryszard
2018-11-27 21:16 ` Jeremiah Cox [this message]
2018-11-27 22:23 ` Rebecca Cran
2018-11-28 18:19 ` Jeremiah Cox
2018-11-28 19:21 ` Rebecca Cran
2018-11-27 12:53 ` Laszlo Ersek
2018-11-27 21:55 ` Brian J. Johnson
2018-11-28 11:07 ` Laszlo Ersek
2018-11-28 18:31 ` Jeremiah Cox
2018-11-28 22:01 ` Laszlo Ersek
2018-11-29 1:07 ` Jeremiah Cox
2018-11-29 9:48 ` Laszlo Ersek
2018-11-29 21:20 ` Rebecca Cran
2018-12-03 9:29 ` Laszlo Ersek
2018-12-03 21:39 ` Rebecca Cran
2018-12-04 18:00 ` Laszlo Ersek
2018-12-05 12:55 ` Laszlo Ersek
2018-12-05 17:26 ` Rebecca Cran
2018-12-06 14:05 ` Laszlo Ersek
2018-12-06 14:07 ` Laszlo Ersek
2018-12-06 14:13 ` Laszlo Ersek
2018-12-06 15:25 ` Rebecca Cran
2018-12-07 6:10 ` Rebecca Cran
2018-12-07 12:00 ` my Phabricator findings [was: Research Request] Laszlo Ersek
2018-12-07 13:11 ` Rebecca Cran
2018-12-05 17:31 ` [edk2-announce] Research Request Rebecca Cran
2018-12-06 13:51 ` Laszlo Ersek
2018-12-03 17:22 ` Jeremiah Cox
2018-12-04 18:26 ` Laszlo Ersek
2018-12-05 19:09 ` Jeremiah Cox
2018-12-06 13:33 ` Laszlo Ersek
2018-11-28 5:54 ` Desimone, Nathaniel L
2018-11-28 6:22 ` Stephano Cetola
2018-12-04 18:20 ` Philippe Mathieu-Daudé
2018-12-05 16:03 ` stephano
2018-12-12 13:20 ` GitLab results from my POV [was: Research Request] Laszlo Ersek
2018-12-20 17:46 ` Rebecca Cran
2019-01-10 20:17 ` about 'sr.ht' " Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=MWHPR21MB0176C5680A597A0026C711FCADD00@MWHPR21MB0176.namprd21.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox