You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by Sean Busbey <bu...@apache.org> on 2018/09/02 16:00:29 UTC
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> This came up at the recent devs meeting: could we move to github flow
> committing to Apache HBase? Do folks want this? If so, what would it take?
> What would it look like?
>
> The new gitbox repos at apache allow contribution back into apache via
> github tooling: PRs can be merged into apache repos with a click of a
> button, github-based comments can show as comments in apache JIRA. The new
> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
> experiments there with fear of damage to the core.
>
> The justification is that if our project supported PRs and contribution via
> github, we could glean more contributors.
>
> Below I repeat two follow-on comments taken from the "Rough notes from dev
> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
> kickstart:
>
> From our Josh Elser:
>
> > This [supporting PRs] is something the PMC should take to heart. If we
> are excluding
> > contributions because of how we choose accept them, we're limiting our
> own
> > growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> accept
> > PR's or is it just because "we do patches because we do patches"?
> >
>
> By our Sean:
>
> "I don't want to bog down this thread, but there are a ton of
> unanswered questions for allowing github PRs.
>
> "The biggest one for me is that JIRA is currently our best hope for an
> authoritative place for authorship information. If we're taking PRs
> from folks who have GitHub accounts but find ASF JIRA accounts too
> burdensome, what are we putting for the author in JIRA? Am I going to
> have to look in JIRA before a certain date and in Git after? Or in Git
> only if JIRA is set to some "HBase Contributor from GitHub" account?"
>
> Thanks,
> St.Ack
>
Hey folks,
This authorship bit might be coming up now. There's a PR for an existing
JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
JIRA account.
I've asked them about it. If they don't have an account and don't want to
make one, I suppose I'll make a placeholder JIRA account and send the
details to the PMC, unless someone objects.
I'm not happy about the bifrucation of authorship information but I don't
see how forcing JIRA account creation could possibly be sustainable.
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Josh Elser <el...@apache.org>.
Reading between the lines: so, this is just HBase convention to have a
Jira assignee mapping to the change's author, right? We're not concerned
about Jira assignee and its relation to ASF intellectual property
provenance.
If I'm getting this right, I think requiring a Jira account is
unnecessary and us coming up with a convention is fine _if_ we want to
be tracking the number of commits from authors that don't have Jira
accounts. But, I don't see why we would care about such a metric.
On 9/4/18 9:04 AM, Andrew Purtell wrote:
> Author/contributor information will be in the commit log. It doesnt matter if assigned on JIRA or not. We can update documentation for committers to make it clear proper attribution via commit metadata (either git author field or name in parenthesis at tail of summary line) is expected.
>
> All we currently require from JIRA is that an issue for a commit exists, the identifier matches what was specified in the commit, the fix versions correctly cover the code lines where the change was committed, and the state reflects commit state (resolved when committed). This is so when RMs use the JIRA changelog for the release the information is accurate.
>
>
>> On Sep 4, 2018, at 7:00 AM, Sean Busbey <bu...@apache.org> wrote:
>>
>> As a project we haven't always been diligent on JIRA fields. If
>> "Unassigned" means "you need to look at the git author" then there's
>> no way to differentiate that from "someone forgot to assign the JIRA".
>>> On Mon, Sep 3, 2018 at 9:43 PM Josh Elser <el...@apache.org> wrote:
>>>
>>>
>>>
>>>> On 9/2/18 12:00 PM, Sean Busbey wrote:
>>>>> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
>>>>>
>>>>> This came up at the recent devs meeting: could we move to github flow
>>>>> committing to Apache HBase? Do folks want this? If so, what would it take?
>>>>> What would it look like?
>>>>>
>>>>> The new gitbox repos at apache allow contribution back into apache via
>>>>> github tooling: PRs can be merged into apache repos with a click of a
>>>>> button, github-based comments can show as comments in apache JIRA. The new
>>>>> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
>>>>> experiments there with fear of damage to the core.
>>>>>
>>>>> The justification is that if our project supported PRs and contribution via
>>>>> github, we could glean more contributors.
>>>>>
>>>>> Below I repeat two follow-on comments taken from the "Rough notes from dev
>>>>> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
>>>>> kickstart:
>>>>>
>>>>> From our Josh Elser:
>>>>>
>>>>>> This [supporting PRs] is something the PMC should take to heart. If we
>>>>> are excluding
>>>>>> contributions because of how we choose accept them, we're limiting our
>>>>> own
>>>>>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
>>>>> accept
>>>>>> PR's or is it just because "we do patches because we do patches"?
>>>>>>
>>>>>
>>>>> By our Sean:
>>>>>
>>>>> "I don't want to bog down this thread, but there are a ton of
>>>>> unanswered questions for allowing github PRs.
>>>>>
>>>>> "The biggest one for me is that JIRA is currently our best hope for an
>>>>> authoritative place for authorship information. If we're taking PRs
>>>>> from folks who have GitHub accounts but find ASF JIRA accounts too
>>>>> burdensome, what are we putting for the author in JIRA? Am I going to
>>>>> have to look in JIRA before a certain date and in Git after? Or in Git
>>>>> only if JIRA is set to some "HBase Contributor from GitHub" account?"
>>>>>
>>>>> Thanks,
>>>>> St.Ack
>>>>>
>>>>
>>>>
>>>> Hey folks,
>>>>
>>>> This authorship bit might be coming up now. There's a PR for an existing
>>>> JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
>>>> JIRA account.
>>>>
>>>> I've asked them about it. If they don't have an account and don't want to
>>>> make one, I suppose I'll make a placeholder JIRA account and send the
>>>> details to the PMC, unless someone objects.
>>>>
>>>> I'm not happy about the bifrucation of authorship information but I don't
>>>> see how forcing JIRA account creation could possibly be sustainable.
>>>
>>> Maybe I'm missing something, but what does an "empty" Jira account just
>>> to assign a Jira issue to actually get us over "Unassigned"? To Andrew's
>>> point about the committer ensuring IP provenance for changes hitting the
>>> repo to satisfy ASF requirements, I think that's orthogonal to having a
>>> Jira name tied to it, right?
>>>
>>> Maybe I'm missing something though?
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Andrew Purtell <an...@gmail.com>.
Author/contributor information will be in the commit log. It doesnt matter if assigned on JIRA or not. We can update documentation for committers to make it clear proper attribution via commit metadata (either git author field or name in parenthesis at tail of summary line) is expected.
All we currently require from JIRA is that an issue for a commit exists, the identifier matches what was specified in the commit, the fix versions correctly cover the code lines where the change was committed, and the state reflects commit state (resolved when committed). This is so when RMs use the JIRA changelog for the release the information is accurate.
> On Sep 4, 2018, at 7:00 AM, Sean Busbey <bu...@apache.org> wrote:
>
> As a project we haven't always been diligent on JIRA fields. If
> "Unassigned" means "you need to look at the git author" then there's
> no way to differentiate that from "someone forgot to assign the JIRA".
>> On Mon, Sep 3, 2018 at 9:43 PM Josh Elser <el...@apache.org> wrote:
>>
>>
>>
>>> On 9/2/18 12:00 PM, Sean Busbey wrote:
>>>> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
>>>>
>>>> This came up at the recent devs meeting: could we move to github flow
>>>> committing to Apache HBase? Do folks want this? If so, what would it take?
>>>> What would it look like?
>>>>
>>>> The new gitbox repos at apache allow contribution back into apache via
>>>> github tooling: PRs can be merged into apache repos with a click of a
>>>> button, github-based comments can show as comments in apache JIRA. The new
>>>> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
>>>> experiments there with fear of damage to the core.
>>>>
>>>> The justification is that if our project supported PRs and contribution via
>>>> github, we could glean more contributors.
>>>>
>>>> Below I repeat two follow-on comments taken from the "Rough notes from dev
>>>> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
>>>> kickstart:
>>>>
>>>> From our Josh Elser:
>>>>
>>>>> This [supporting PRs] is something the PMC should take to heart. If we
>>>> are excluding
>>>>> contributions because of how we choose accept them, we're limiting our
>>>> own
>>>>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
>>>> accept
>>>>> PR's or is it just because "we do patches because we do patches"?
>>>>>
>>>>
>>>> By our Sean:
>>>>
>>>> "I don't want to bog down this thread, but there are a ton of
>>>> unanswered questions for allowing github PRs.
>>>>
>>>> "The biggest one for me is that JIRA is currently our best hope for an
>>>> authoritative place for authorship information. If we're taking PRs
>>>> from folks who have GitHub accounts but find ASF JIRA accounts too
>>>> burdensome, what are we putting for the author in JIRA? Am I going to
>>>> have to look in JIRA before a certain date and in Git after? Or in Git
>>>> only if JIRA is set to some "HBase Contributor from GitHub" account?"
>>>>
>>>> Thanks,
>>>> St.Ack
>>>>
>>>
>>>
>>> Hey folks,
>>>
>>> This authorship bit might be coming up now. There's a PR for an existing
>>> JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
>>> JIRA account.
>>>
>>> I've asked them about it. If they don't have an account and don't want to
>>> make one, I suppose I'll make a placeholder JIRA account and send the
>>> details to the PMC, unless someone objects.
>>>
>>> I'm not happy about the bifrucation of authorship information but I don't
>>> see how forcing JIRA account creation could possibly be sustainable.
>>
>> Maybe I'm missing something, but what does an "empty" Jira account just
>> to assign a Jira issue to actually get us over "Unassigned"? To Andrew's
>> point about the committer ensuring IP provenance for changes hitting the
>> repo to satisfy ASF requirements, I think that's orthogonal to having a
>> Jira name tied to it, right?
>>
>> Maybe I'm missing something though?
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Sean Busbey <bu...@apache.org>.
As a project we haven't always been diligent on JIRA fields. If
"Unassigned" means "you need to look at the git author" then there's
no way to differentiate that from "someone forgot to assign the JIRA".
On Mon, Sep 3, 2018 at 9:43 PM Josh Elser <el...@apache.org> wrote:
>
>
>
> On 9/2/18 12:00 PM, Sean Busbey wrote:
> > On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> >
> >> This came up at the recent devs meeting: could we move to github flow
> >> committing to Apache HBase? Do folks want this? If so, what would it take?
> >> What would it look like?
> >>
> >> The new gitbox repos at apache allow contribution back into apache via
> >> github tooling: PRs can be merged into apache repos with a click of a
> >> button, github-based comments can show as comments in apache JIRA. The new
> >> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
> >> experiments there with fear of damage to the core.
> >>
> >> The justification is that if our project supported PRs and contribution via
> >> github, we could glean more contributors.
> >>
> >> Below I repeat two follow-on comments taken from the "Rough notes from dev
> >> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
> >> kickstart:
> >>
> >> From our Josh Elser:
> >>
> >>> This [supporting PRs] is something the PMC should take to heart. If we
> >> are excluding
> >>> contributions because of how we choose accept them, we're limiting our
> >> own
> >>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> >> accept
> >>> PR's or is it just because "we do patches because we do patches"?
> >>>
> >>
> >> By our Sean:
> >>
> >> "I don't want to bog down this thread, but there are a ton of
> >> unanswered questions for allowing github PRs.
> >>
> >> "The biggest one for me is that JIRA is currently our best hope for an
> >> authoritative place for authorship information. If we're taking PRs
> >> from folks who have GitHub accounts but find ASF JIRA accounts too
> >> burdensome, what are we putting for the author in JIRA? Am I going to
> >> have to look in JIRA before a certain date and in Git after? Or in Git
> >> only if JIRA is set to some "HBase Contributor from GitHub" account?"
> >>
> >> Thanks,
> >> St.Ack
> >>
> >
> >
> > Hey folks,
> >
> > This authorship bit might be coming up now. There's a PR for an existing
> > JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
> > JIRA account.
> >
> > I've asked them about it. If they don't have an account and don't want to
> > make one, I suppose I'll make a placeholder JIRA account and send the
> > details to the PMC, unless someone objects.
> >
> > I'm not happy about the bifrucation of authorship information but I don't
> > see how forcing JIRA account creation could possibly be sustainable.
>
> Maybe I'm missing something, but what does an "empty" Jira account just
> to assign a Jira issue to actually get us over "Unassigned"? To Andrew's
> point about the committer ensuring IP provenance for changes hitting the
> repo to satisfy ASF requirements, I think that's orthogonal to having a
> Jira name tied to it, right?
>
> Maybe I'm missing something though?
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Yu Li <ca...@gmail.com>.
Just checked Flink document more carefully, it requires to file a JIRA but
no mention of creating account or assigning to oneself. However, I think
it's OK to require our contributor to create a JIRA account with the same
email address as github account - it requires no big effort and gives
people credit (telling who resolves the issue) - or at least we could make
it a strong recommendation.
Maybe I'm too old to guess the young generation's thought, but I believe
process won't be the obstacle of contributing if one really loves open
source, not to mention it's a one-time effort (smile)
Best Regards,
Yu
On Tue, 4 Sep 2018 at 10:43, Josh Elser <el...@apache.org> wrote:
>
>
> On 9/2/18 12:00 PM, Sean Busbey wrote:
> > On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> >
> >> This came up at the recent devs meeting: could we move to github flow
> >> committing to Apache HBase? Do folks want this? If so, what would it
> take?
> >> What would it look like?
> >>
> >> The new gitbox repos at apache allow contribution back into apache via
> >> github tooling: PRs can be merged into apache repos with a click of a
> >> button, github-based comments can show as comments in apache JIRA. The
> new
> >> hbase-operator-tools and hbase-connector repos are gitbox based. We can
> run
> >> experiments there with fear of damage to the core.
> >>
> >> The justification is that if our project supported PRs and contribution
> via
> >> github, we could glean more contributors.
> >>
> >> Below I repeat two follow-on comments taken from the "Rough notes from
> dev
> >> meetup, day after hbaseconasia 2018, saturday morning" thread by way of
> a
> >> kickstart:
> >>
> >> From our Josh Elser:
> >>
> >>> This [supporting PRs] is something the PMC should take to heart. If we
> >> are excluding
> >>> contributions because of how we choose accept them, we're limiting our
> >> own
> >>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> >> accept
> >>> PR's or is it just because "we do patches because we do patches"?
> >>>
> >>
> >> By our Sean:
> >>
> >> "I don't want to bog down this thread, but there are a ton of
> >> unanswered questions for allowing github PRs.
> >>
> >> "The biggest one for me is that JIRA is currently our best hope for an
> >> authoritative place for authorship information. If we're taking PRs
> >> from folks who have GitHub accounts but find ASF JIRA accounts too
> >> burdensome, what are we putting for the author in JIRA? Am I going to
> >> have to look in JIRA before a certain date and in Git after? Or in Git
> >> only if JIRA is set to some "HBase Contributor from GitHub" account?"
> >>
> >> Thanks,
> >> St.Ack
> >>
> >
> >
> > Hey folks,
> >
> > This authorship bit might be coming up now. There's a PR for an existing
> > JIRA issue that I'd like to accept, but AFAICT the PR author doesn't
> have a
> > JIRA account.
> >
> > I've asked them about it. If they don't have an account and don't want to
> > make one, I suppose I'll make a placeholder JIRA account and send the
> > details to the PMC, unless someone objects.
> >
> > I'm not happy about the bifrucation of authorship information but I don't
> > see how forcing JIRA account creation could possibly be sustainable.
>
> Maybe I'm missing something, but what does an "empty" Jira account just
> to assign a Jira issue to actually get us over "Unassigned"? To Andrew's
> point about the committer ensuring IP provenance for changes hitting the
> repo to satisfy ASF requirements, I think that's orthogonal to having a
> Jira name tied to it, right?
>
> Maybe I'm missing something though?
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Josh Elser <el...@apache.org>.
On 9/2/18 12:00 PM, Sean Busbey wrote:
> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
>
>> This came up at the recent devs meeting: could we move to github flow
>> committing to Apache HBase? Do folks want this? If so, what would it take?
>> What would it look like?
>>
>> The new gitbox repos at apache allow contribution back into apache via
>> github tooling: PRs can be merged into apache repos with a click of a
>> button, github-based comments can show as comments in apache JIRA. The new
>> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
>> experiments there with fear of damage to the core.
>>
>> The justification is that if our project supported PRs and contribution via
>> github, we could glean more contributors.
>>
>> Below I repeat two follow-on comments taken from the "Rough notes from dev
>> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
>> kickstart:
>>
>> From our Josh Elser:
>>
>>> This [supporting PRs] is something the PMC should take to heart. If we
>> are excluding
>>> contributions because of how we choose accept them, we're limiting our
>> own
>>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
>> accept
>>> PR's or is it just because "we do patches because we do patches"?
>>>
>>
>> By our Sean:
>>
>> "I don't want to bog down this thread, but there are a ton of
>> unanswered questions for allowing github PRs.
>>
>> "The biggest one for me is that JIRA is currently our best hope for an
>> authoritative place for authorship information. If we're taking PRs
>> from folks who have GitHub accounts but find ASF JIRA accounts too
>> burdensome, what are we putting for the author in JIRA? Am I going to
>> have to look in JIRA before a certain date and in Git after? Or in Git
>> only if JIRA is set to some "HBase Contributor from GitHub" account?"
>>
>> Thanks,
>> St.Ack
>>
>
>
> Hey folks,
>
> This authorship bit might be coming up now. There's a PR for an existing
> JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
> JIRA account.
>
> I've asked them about it. If they don't have an account and don't want to
> make one, I suppose I'll make a placeholder JIRA account and send the
> details to the PMC, unless someone objects.
>
> I'm not happy about the bifrucation of authorship information but I don't
> see how forcing JIRA account creation could possibly be sustainable.
Maybe I'm missing something, but what does an "empty" Jira account just
to assign a Jira issue to actually get us over "Unassigned"? To Andrew's
point about the committer ensuring IP provenance for changes hitting the
repo to satisfy ASF requirements, I think that's orthogonal to having a
Jira name tied to it, right?
Maybe I'm missing something though?
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Sean Busbey <bu...@apache.org>.
The issue is specific to us being an ASF project. ASF project
repositories can either be set up with "Git WIP" or "Dual Master"[1].
Repositories that use "Git WIP" have their repo mirrored read-only to
GitHub. Repositories that use "Dual Master" have the canonical repo on
GitHub.
Our main project repo is a "Git WIP" one. That means we don't have
write access on GitHub, including merging or closing PRs. Thus the
only way we can close PRs on our main repo is to use the github
integrated commit message handling[2]. Which relies on pushing to the
master branch.
Since our main repo is used to create release artifacts we can only
have committers pushing to it, which precludes having a bot do so.
[1]:
"Dual Master" is also sometimes called "gitbox" because of where the
asf infra side of it lives.
http://git.apache.org/
[2]: https://help.github.com/articles/closing-issues-using-keywords/
On Tue, Sep 4, 2018 at 6:16 PM Misty Linville <mi...@apache.org> wrote:
>
> The code I posted above is an abstraction that is configured by YAML stuff
> like the following (where the test exists for closing PRs):
> https://github.com/kubernetes/test-infra/blob/a8cee5a60a2d9476341cf843867221a8bd18a3e8/config/jobs/kubernetes/test-infra/test-infra-periodics.yaml#L47
>
> I am not up on Go, but others pointed me there.
>
> On Tue, Sep 4, 2018 at 4:12 PM, Misty Linville <mi...@apache.org> wrote:
>
> > Here's the code to the bot: https://github.com/kubernetes/test-infra/tree/
> > master/robots/commenter
> >
> > On Tue, Sep 4, 2018 at 10:21 AM, Sean Busbey <bu...@apache.org> wrote:
> >
> >> Do you have a link to an instance of the bot you're talking about?
> >> On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
> >> >
> >> > The below snip isn't true. Somehow CNCF has tooling that does it (it is
> >> a
> >> > bot called fejtabot). I can try to find out more about it if this is a
> >> > mechanism we are interested in pursuing.
> >> >
> >> > On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
> >> > >
> >> > >
> >> > > We can't automate closing PRs because the only way to close a PR is by
> >> > > pushing a commit to our master branch and ASF policy doesn't allow for
> >> > > non-commiters pushing into the repository if the repository is used to
> >> > > make release artifacts.
> >> > >
> >> > >
> >>
> >
> >
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Misty Linville <mi...@apache.org>.
The code I posted above is an abstraction that is configured by YAML stuff
like the following (where the test exists for closing PRs):
https://github.com/kubernetes/test-infra/blob/a8cee5a60a2d9476341cf843867221a8bd18a3e8/config/jobs/kubernetes/test-infra/test-infra-periodics.yaml#L47
I am not up on Go, but others pointed me there.
On Tue, Sep 4, 2018 at 4:12 PM, Misty Linville <mi...@apache.org> wrote:
> Here's the code to the bot: https://github.com/kubernetes/test-infra/tree/
> master/robots/commenter
>
> On Tue, Sep 4, 2018 at 10:21 AM, Sean Busbey <bu...@apache.org> wrote:
>
>> Do you have a link to an instance of the bot you're talking about?
>> On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
>> >
>> > The below snip isn't true. Somehow CNCF has tooling that does it (it is
>> a
>> > bot called fejtabot). I can try to find out more about it if this is a
>> > mechanism we are interested in pursuing.
>> >
>> > On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
>> > >
>> > >
>> > > We can't automate closing PRs because the only way to close a PR is by
>> > > pushing a commit to our master branch and ASF policy doesn't allow for
>> > > non-commiters pushing into the repository if the repository is used to
>> > > make release artifacts.
>> > >
>> > >
>>
>
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Misty Linville <mi...@apache.org>.
Here's the code to the bot:
https://github.com/kubernetes/test-infra/tree/master/robots/commenter
On Tue, Sep 4, 2018 at 10:21 AM, Sean Busbey <bu...@apache.org> wrote:
> Do you have a link to an instance of the bot you're talking about?
> On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
> >
> > The below snip isn't true. Somehow CNCF has tooling that does it (it is a
> > bot called fejtabot). I can try to find out more about it if this is a
> > mechanism we are interested in pursuing.
> >
> > On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
> > >
> > >
> > > We can't automate closing PRs because the only way to close a PR is by
> > > pushing a commit to our master branch and ASF policy doesn't allow for
> > > non-commiters pushing into the repository if the repository is used to
> > > make release artifacts.
> > >
> > >
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Andrew Purtell <ap...@apache.org>.
I use JIRA, good point.
On Tue, Sep 4, 2018 at 10:27 AM Mike Drob <md...@apache.org> wrote:
> Do folks use JIRA issues to account for work done when evaluating potential
> committers? Or is that generally gathered from git log? I don't think I
> have a preference either way but with multiple sources of "truth" we need
> to be aware of the possible inconsistencies.
>
> Mike
>
> On Tue, Sep 4, 2018 at 12:21 PM, Sean Busbey <bu...@apache.org> wrote:
>
> > Do you have a link to an instance of the bot you're talking about?
> > On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
> > >
> > > The below snip isn't true. Somehow CNCF has tooling that does it (it
> is a
> > > bot called fejtabot). I can try to find out more about it if this is a
> > > mechanism we are interested in pursuing.
> > >
> > > On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
> > > >
> > > >
> > > > We can't automate closing PRs because the only way to close a PR is
> by
> > > > pushing a commit to our master branch and ASF policy doesn't allow
> for
> > > > non-commiters pushing into the repository if the repository is used
> to
> > > > make release artifacts.
> > > >
> > > >
> >
>
--
Best regards,
Andrew
Words like orphans lost among the crosstalk, meaning torn from truth's
decrepit hands
- A23, Crosstalk
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Mike Drob <md...@apache.org>.
Do folks use JIRA issues to account for work done when evaluating potential
committers? Or is that generally gathered from git log? I don't think I
have a preference either way but with multiple sources of "truth" we need
to be aware of the possible inconsistencies.
Mike
On Tue, Sep 4, 2018 at 12:21 PM, Sean Busbey <bu...@apache.org> wrote:
> Do you have a link to an instance of the bot you're talking about?
> On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
> >
> > The below snip isn't true. Somehow CNCF has tooling that does it (it is a
> > bot called fejtabot). I can try to find out more about it if this is a
> > mechanism we are interested in pursuing.
> >
> > On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
> > >
> > >
> > > We can't automate closing PRs because the only way to close a PR is by
> > > pushing a commit to our master branch and ASF policy doesn't allow for
> > > non-commiters pushing into the repository if the repository is used to
> > > make release artifacts.
> > >
> > >
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Sean Busbey <bu...@apache.org>.
Do you have a link to an instance of the bot you're talking about?
On Tue, Sep 4, 2018 at 11:18 AM Misty Linville <mi...@apache.org> wrote:
>
> The below snip isn't true. Somehow CNCF has tooling that does it (it is a
> bot called fejtabot). I can try to find out more about it if this is a
> mechanism we are interested in pursuing.
>
> On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
> >
> >
> > We can't automate closing PRs because the only way to close a PR is by
> > pushing a commit to our master branch and ASF policy doesn't allow for
> > non-commiters pushing into the repository if the repository is used to
> > make release artifacts.
> >
> >
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Misty Linville <mi...@apache.org>.
The below snip isn't true. Somehow CNCF has tooling that does it (it is a
bot called fejtabot). I can try to find out more about it if this is a
mechanism we are interested in pursuing.
On Tue, Sep 4, 2018 at 7:08 AM, Sean Busbey <bu...@apache.org> wrote:
>
>
> We can't automate closing PRs because the only way to close a PR is by
> pushing a commit to our master branch and ASF policy doesn't allow for
> non-commiters pushing into the repository if the repository is used to
> make release artifacts.
>
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Sean Busbey <bu...@apache.org>.
We could require ASF JIRA accounts, but it will definitely keep some
people from contributing. We have no way to measure what the fall off
is because we won't be able to tell those folks apart from those who
don't follow up on the PR for other reasons.
We can't automate closing PRs because the only way to close a PR is by
pushing a commit to our master branch and ASF policy doesn't allow for
non-commiters pushing into the repository if the repository is used to
make release artifacts.
This needn't be a pressing issue. We've had 90 PRs _ever_, so right
now we're only talking about ~2 per month.
On Mon, Sep 3, 2018 at 1:26 PM Misty Linville <mi...@apache.org> wrote:
>
> I like this suggestion, Yu, along with enthusiastic closing of GitHub PRs
> that don't follow the established procedures. Could we create an automated
> test for such compliance, like scanning the PR comments for specific text?
> CNCF projects use automation to parse a specific syntax in comments and set
> GitHub labels based on it. It could be as simple as /jira HBASE-foo, and
> automation could check that the issue actually exists and is open. The PR
> author could get a grace period of say a week before the PR is
> automatically closed, with warnings at regular intervals. Even if the PR is
> closed they can easily reopen it when they have created the issue. If they
> don't, we can't take their contribution, no matter how good it is.
>
> This approach front-loads the work of creating the automation, but reduces
> overall load of reviewers verifying compliance of the PR. It also enforces
> the policy with more consistency than relying on humans to do it.
>
> I suspect that ASF will eventually need to offer account holders to
> associate their account with a GitHub account, and that would alleviate
> some of this. There could even be a chain of trust by committing a
> cryptographically unique file generated by ASF processes into the user's
> GitHub account maybe (as a gist?) as part of that connection, or by using
> GitHub's authorization system.
>
> On Sun, Sep 2, 2018, 11:32 PM Yu Li <ca...@gmail.com> wrote:
>
> > Maybe we could follow some other projects' process? For example I could
> > find below lines of Flink's Code of Conduct
> > <https://flink.apache.org/contribute-code.html>, JFYI:
> >
> > *Before you start coding…*
> >
> > *…please make sure there is a Jira issue that corresponds to your
> > contribution. This is a general rule that the Flink community follows for
> > all code contributions, including bug fixes, improvements, or new features,
> > with an exception for trivial hot fixes. If you would like to fix a bug
> > that you found or if you would like to add a new feature or improvement to
> > Flink, please follow the File a bug report
> > <https://flink.apache.org/how-to-contribute.html#file-a-bug-report> or
> > Propose
> > an improvement or a new feature
> > <
> > https://flink.apache.org/how-to-contribute.html#propose-an-improvement-or-a-new-feature
> > >
> > guidelines
> > to open an issue in Flink’s Jira
> > <http://issues.apache.org/jira/browse/FLINK> before starting with the
> > implementation.*
> >
> > Best Regards,
> > Yu
> >
> >
> > On Mon, 3 Sep 2018 at 00:52, Andrew Purtell <an...@gmail.com>
> > wrote:
> >
> > > It's on us as committers to ensure Apache policies on code contribution
> > > are followed. I think that is the bottom line. Our identities are known
> > to
> > > Apache infrastructure and recorded in commit history. By committing we
> > are
> > > asserting we believe the provenance of the contribution is known and
> > > conforms to Apache policy. If you have concerns then don't commit.
> > > Otherwise, it is the same as it always was. Both of Apache's JIRA
> > instance
> > > and Github offer self service account signups without any sort of
> > identity
> > > verification, only verification that the email account provided is valid
> > > and under the control of the contributor.
> > >
> > > I think opening placeholder JIRAs as committers because we need it for
> > > tracking is fine. With my PMC hat on I'd like to wonder aloud what is the
> > > problem with signing up to Apache's JIRA instance. Any reason to be
> > > concerned? I take it you don't think so. I also take it you believe you
> > > know the identity of the contributor. Assuming you answer in the
> > > affirmative I don't see that we have any issues.
> > >
> > >
> > > > On Sep 2, 2018, at 9:00 AM, Sean Busbey <bu...@apache.org> wrote:
> > > >
> > > >> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> > > >>
> > > >> This came up at the recent devs meeting: could we move to github flow
> > > >> committing to Apache HBase? Do folks want this? If so, what would it
> > > take?
> > > >> What would it look like?
> > > >>
> > > >> The new gitbox repos at apache allow contribution back into apache via
> > > >> github tooling: PRs can be merged into apache repos with a click of a
> > > >> button, github-based comments can show as comments in apache JIRA. The
> > > new
> > > >> hbase-operator-tools and hbase-connector repos are gitbox based. We
> > can
> > > run
> > > >> experiments there with fear of damage to the core.
> > > >>
> > > >> The justification is that if our project supported PRs and
> > contribution
> > > via
> > > >> github, we could glean more contributors.
> > > >>
> > > >> Below I repeat two follow-on comments taken from the "Rough notes from
> > > dev
> > > >> meetup, day after hbaseconasia 2018, saturday morning" thread by way
> > of
> > > a
> > > >> kickstart:
> > > >>
> > > >> From our Josh Elser:
> > > >>
> > > >>> This [supporting PRs] is something the PMC should take to heart. If
> > we
> > > >> are excluding
> > > >>> contributions because of how we choose accept them, we're limiting
> > our
> > > >> own
> > > >>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> > > >> accept
> > > >>> PR's or is it just because "we do patches because we do patches"?
> > > >>>
> > > >>
> > > >> By our Sean:
> > > >>
> > > >> "I don't want to bog down this thread, but there are a ton of
> > > >> unanswered questions for allowing github PRs.
> > > >>
> > > >> "The biggest one for me is that JIRA is currently our best hope for an
> > > >> authoritative place for authorship information. If we're taking PRs
> > > >> from folks who have GitHub accounts but find ASF JIRA accounts too
> > > >> burdensome, what are we putting for the author in JIRA? Am I going to
> > > >> have to look in JIRA before a certain date and in Git after? Or in Git
> > > >> only if JIRA is set to some "HBase Contributor from GitHub" account?"
> > > >>
> > > >> Thanks,
> > > >> St.Ack
> > > >>
> > > >
> > > >
> > > > Hey folks,
> > > >
> > > > This authorship bit might be coming up now. There's a PR for an
> > existing
> > > > JIRA issue that I'd like to accept, but AFAICT the PR author doesn't
> > > have a
> > > > JIRA account.
> > > >
> > > > I've asked them about it. If they don't have an account and don't want
> > to
> > > > make one, I suppose I'll make a placeholder JIRA account and send the
> > > > details to the PMC, unless someone objects.
> > > >
> > > > I'm not happy about the bifrucation of authorship information but I
> > don't
> > > > see how forcing JIRA account creation could possibly be sustainable.
> > > >
> > > >>
> > >
> >
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Misty Linville <mi...@apache.org>.
I like this suggestion, Yu, along with enthusiastic closing of GitHub PRs
that don't follow the established procedures. Could we create an automated
test for such compliance, like scanning the PR comments for specific text?
CNCF projects use automation to parse a specific syntax in comments and set
GitHub labels based on it. It could be as simple as /jira HBASE-foo, and
automation could check that the issue actually exists and is open. The PR
author could get a grace period of say a week before the PR is
automatically closed, with warnings at regular intervals. Even if the PR is
closed they can easily reopen it when they have created the issue. If they
don't, we can't take their contribution, no matter how good it is.
This approach front-loads the work of creating the automation, but reduces
overall load of reviewers verifying compliance of the PR. It also enforces
the policy with more consistency than relying on humans to do it.
I suspect that ASF will eventually need to offer account holders to
associate their account with a GitHub account, and that would alleviate
some of this. There could even be a chain of trust by committing a
cryptographically unique file generated by ASF processes into the user's
GitHub account maybe (as a gist?) as part of that connection, or by using
GitHub's authorization system.
On Sun, Sep 2, 2018, 11:32 PM Yu Li <ca...@gmail.com> wrote:
> Maybe we could follow some other projects' process? For example I could
> find below lines of Flink's Code of Conduct
> <https://flink.apache.org/contribute-code.html>, JFYI:
>
> *Before you start coding…*
>
> *…please make sure there is a Jira issue that corresponds to your
> contribution. This is a general rule that the Flink community follows for
> all code contributions, including bug fixes, improvements, or new features,
> with an exception for trivial hot fixes. If you would like to fix a bug
> that you found or if you would like to add a new feature or improvement to
> Flink, please follow the File a bug report
> <https://flink.apache.org/how-to-contribute.html#file-a-bug-report> or
> Propose
> an improvement or a new feature
> <
> https://flink.apache.org/how-to-contribute.html#propose-an-improvement-or-a-new-feature
> >
> guidelines
> to open an issue in Flink’s Jira
> <http://issues.apache.org/jira/browse/FLINK> before starting with the
> implementation.*
>
> Best Regards,
> Yu
>
>
> On Mon, 3 Sep 2018 at 00:52, Andrew Purtell <an...@gmail.com>
> wrote:
>
> > It's on us as committers to ensure Apache policies on code contribution
> > are followed. I think that is the bottom line. Our identities are known
> to
> > Apache infrastructure and recorded in commit history. By committing we
> are
> > asserting we believe the provenance of the contribution is known and
> > conforms to Apache policy. If you have concerns then don't commit.
> > Otherwise, it is the same as it always was. Both of Apache's JIRA
> instance
> > and Github offer self service account signups without any sort of
> identity
> > verification, only verification that the email account provided is valid
> > and under the control of the contributor.
> >
> > I think opening placeholder JIRAs as committers because we need it for
> > tracking is fine. With my PMC hat on I'd like to wonder aloud what is the
> > problem with signing up to Apache's JIRA instance. Any reason to be
> > concerned? I take it you don't think so. I also take it you believe you
> > know the identity of the contributor. Assuming you answer in the
> > affirmative I don't see that we have any issues.
> >
> >
> > > On Sep 2, 2018, at 9:00 AM, Sean Busbey <bu...@apache.org> wrote:
> > >
> > >> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> > >>
> > >> This came up at the recent devs meeting: could we move to github flow
> > >> committing to Apache HBase? Do folks want this? If so, what would it
> > take?
> > >> What would it look like?
> > >>
> > >> The new gitbox repos at apache allow contribution back into apache via
> > >> github tooling: PRs can be merged into apache repos with a click of a
> > >> button, github-based comments can show as comments in apache JIRA. The
> > new
> > >> hbase-operator-tools and hbase-connector repos are gitbox based. We
> can
> > run
> > >> experiments there with fear of damage to the core.
> > >>
> > >> The justification is that if our project supported PRs and
> contribution
> > via
> > >> github, we could glean more contributors.
> > >>
> > >> Below I repeat two follow-on comments taken from the "Rough notes from
> > dev
> > >> meetup, day after hbaseconasia 2018, saturday morning" thread by way
> of
> > a
> > >> kickstart:
> > >>
> > >> From our Josh Elser:
> > >>
> > >>> This [supporting PRs] is something the PMC should take to heart. If
> we
> > >> are excluding
> > >>> contributions because of how we choose accept them, we're limiting
> our
> > >> own
> > >>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> > >> accept
> > >>> PR's or is it just because "we do patches because we do patches"?
> > >>>
> > >>
> > >> By our Sean:
> > >>
> > >> "I don't want to bog down this thread, but there are a ton of
> > >> unanswered questions for allowing github PRs.
> > >>
> > >> "The biggest one for me is that JIRA is currently our best hope for an
> > >> authoritative place for authorship information. If we're taking PRs
> > >> from folks who have GitHub accounts but find ASF JIRA accounts too
> > >> burdensome, what are we putting for the author in JIRA? Am I going to
> > >> have to look in JIRA before a certain date and in Git after? Or in Git
> > >> only if JIRA is set to some "HBase Contributor from GitHub" account?"
> > >>
> > >> Thanks,
> > >> St.Ack
> > >>
> > >
> > >
> > > Hey folks,
> > >
> > > This authorship bit might be coming up now. There's a PR for an
> existing
> > > JIRA issue that I'd like to accept, but AFAICT the PR author doesn't
> > have a
> > > JIRA account.
> > >
> > > I've asked them about it. If they don't have an account and don't want
> to
> > > make one, I suppose I'll make a placeholder JIRA account and send the
> > > details to the PMC, unless someone objects.
> > >
> > > I'm not happy about the bifrucation of authorship information but I
> don't
> > > see how forcing JIRA account creation could possibly be sustainable.
> > >
> > >>
> >
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from
dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Yu Li <ca...@gmail.com>.
Maybe we could follow some other projects' process? For example I could
find below lines of Flink's Code of Conduct
<https://flink.apache.org/contribute-code.html>, JFYI:
*Before you start coding…*
*…please make sure there is a Jira issue that corresponds to your
contribution. This is a general rule that the Flink community follows for
all code contributions, including bug fixes, improvements, or new features,
with an exception for trivial hot fixes. If you would like to fix a bug
that you found or if you would like to add a new feature or improvement to
Flink, please follow the File a bug report
<https://flink.apache.org/how-to-contribute.html#file-a-bug-report> or Propose
an improvement or a new feature
<https://flink.apache.org/how-to-contribute.html#propose-an-improvement-or-a-new-feature>
guidelines
to open an issue in Flink’s Jira
<http://issues.apache.org/jira/browse/FLINK> before starting with the
implementation.*
Best Regards,
Yu
On Mon, 3 Sep 2018 at 00:52, Andrew Purtell <an...@gmail.com>
wrote:
> It's on us as committers to ensure Apache policies on code contribution
> are followed. I think that is the bottom line. Our identities are known to
> Apache infrastructure and recorded in commit history. By committing we are
> asserting we believe the provenance of the contribution is known and
> conforms to Apache policy. If you have concerns then don't commit.
> Otherwise, it is the same as it always was. Both of Apache's JIRA instance
> and Github offer self service account signups without any sort of identity
> verification, only verification that the email account provided is valid
> and under the control of the contributor.
>
> I think opening placeholder JIRAs as committers because we need it for
> tracking is fine. With my PMC hat on I'd like to wonder aloud what is the
> problem with signing up to Apache's JIRA instance. Any reason to be
> concerned? I take it you don't think so. I also take it you believe you
> know the identity of the contributor. Assuming you answer in the
> affirmative I don't see that we have any issues.
>
>
> > On Sep 2, 2018, at 9:00 AM, Sean Busbey <bu...@apache.org> wrote:
> >
> >> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
> >>
> >> This came up at the recent devs meeting: could we move to github flow
> >> committing to Apache HBase? Do folks want this? If so, what would it
> take?
> >> What would it look like?
> >>
> >> The new gitbox repos at apache allow contribution back into apache via
> >> github tooling: PRs can be merged into apache repos with a click of a
> >> button, github-based comments can show as comments in apache JIRA. The
> new
> >> hbase-operator-tools and hbase-connector repos are gitbox based. We can
> run
> >> experiments there with fear of damage to the core.
> >>
> >> The justification is that if our project supported PRs and contribution
> via
> >> github, we could glean more contributors.
> >>
> >> Below I repeat two follow-on comments taken from the "Rough notes from
> dev
> >> meetup, day after hbaseconasia 2018, saturday morning" thread by way of
> a
> >> kickstart:
> >>
> >> From our Josh Elser:
> >>
> >>> This [supporting PRs] is something the PMC should take to heart. If we
> >> are excluding
> >>> contributions because of how we choose accept them, we're limiting our
> >> own
> >>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
> >> accept
> >>> PR's or is it just because "we do patches because we do patches"?
> >>>
> >>
> >> By our Sean:
> >>
> >> "I don't want to bog down this thread, but there are a ton of
> >> unanswered questions for allowing github PRs.
> >>
> >> "The biggest one for me is that JIRA is currently our best hope for an
> >> authoritative place for authorship information. If we're taking PRs
> >> from folks who have GitHub accounts but find ASF JIRA accounts too
> >> burdensome, what are we putting for the author in JIRA? Am I going to
> >> have to look in JIRA before a certain date and in Git after? Or in Git
> >> only if JIRA is set to some "HBase Contributor from GitHub" account?"
> >>
> >> Thanks,
> >> St.Ack
> >>
> >
> >
> > Hey folks,
> >
> > This authorship bit might be coming up now. There's a PR for an existing
> > JIRA issue that I'd like to accept, but AFAICT the PR author doesn't
> have a
> > JIRA account.
> >
> > I've asked them about it. If they don't have an account and don't want to
> > make one, I suppose I'll make a placeholder JIRA account and send the
> > details to the PMC, unless someone objects.
> >
> > I'm not happy about the bifrucation of authorship information but I don't
> > see how forcing JIRA account creation could possibly be sustainable.
> >
> >>
>
Re: [DISCUSS] Move to github flow (WAS subtopic of "Rough notes from dev meetup, day after hbaseconasia 2018, saturday morning")
Posted by Andrew Purtell <an...@gmail.com>.
It's on us as committers to ensure Apache policies on code contribution are followed. I think that is the bottom line. Our identities are known to Apache infrastructure and recorded in commit history. By committing we are asserting we believe the provenance of the contribution is known and conforms to Apache policy. If you have concerns then don't commit. Otherwise, it is the same as it always was. Both of Apache's JIRA instance and Github offer self service account signups without any sort of identity verification, only verification that the email account provided is valid and under the control of the contributor.
I think opening placeholder JIRAs as committers because we need it for tracking is fine. With my PMC hat on I'd like to wonder aloud what is the problem with signing up to Apache's JIRA instance. Any reason to be concerned? I take it you don't think so. I also take it you believe you know the identity of the contributor. Assuming you answer in the affirmative I don't see that we have any issues.
> On Sep 2, 2018, at 9:00 AM, Sean Busbey <bu...@apache.org> wrote:
>
>> On Mon, Aug 20, 2018, 22:09 Stack <st...@duboce.net> wrote:
>>
>> This came up at the recent devs meeting: could we move to github flow
>> committing to Apache HBase? Do folks want this? If so, what would it take?
>> What would it look like?
>>
>> The new gitbox repos at apache allow contribution back into apache via
>> github tooling: PRs can be merged into apache repos with a click of a
>> button, github-based comments can show as comments in apache JIRA. The new
>> hbase-operator-tools and hbase-connector repos are gitbox based. We can run
>> experiments there with fear of damage to the core.
>>
>> The justification is that if our project supported PRs and contribution via
>> github, we could glean more contributors.
>>
>> Below I repeat two follow-on comments taken from the "Rough notes from dev
>> meetup, day after hbaseconasia 2018, saturday morning" thread by way of a
>> kickstart:
>>
>> From our Josh Elser:
>>
>>> This [supporting PRs] is something the PMC should take to heart. If we
>> are excluding
>>> contributions because of how we choose accept them, we're limiting our
>> own
>>> growth. Do we have technical reasons (e.g. PreCommit) which we cannot
>> accept
>>> PR's or is it just because "we do patches because we do patches"?
>>>
>>
>> By our Sean:
>>
>> "I don't want to bog down this thread, but there are a ton of
>> unanswered questions for allowing github PRs.
>>
>> "The biggest one for me is that JIRA is currently our best hope for an
>> authoritative place for authorship information. If we're taking PRs
>> from folks who have GitHub accounts but find ASF JIRA accounts too
>> burdensome, what are we putting for the author in JIRA? Am I going to
>> have to look in JIRA before a certain date and in Git after? Or in Git
>> only if JIRA is set to some "HBase Contributor from GitHub" account?"
>>
>> Thanks,
>> St.Ack
>>
>
>
> Hey folks,
>
> This authorship bit might be coming up now. There's a PR for an existing
> JIRA issue that I'd like to accept, but AFAICT the PR author doesn't have a
> JIRA account.
>
> I've asked them about it. If they don't have an account and don't want to
> make one, I suppose I'll make a placeholder JIRA account and send the
> details to the PMC, unless someone objects.
>
> I'm not happy about the bifrucation of authorship information but I don't
> see how forcing JIRA account creation could possibly be sustainable.
>
>>