Bug 1430303 - Allow Jiffin to merge patches to the glusterfs/release-3.8 branch
Summary: Allow Jiffin to merge patches to the glusterfs/release-3.8 branch
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: project-infrastructure
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-03-08 10:35 UTC by Niels de Vos
Modified: 2017-03-10 05:49 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-03-09 15:25:09 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Niels de Vos 2017-03-08 10:35:10 UTC
Jiffin (jthottan) is my assistant-maintainer for the 3.8 release, and it would help if he can get merge permissions for the release-3.8 branch in the glusterfs repository on Gerrit.

Comment 1 Niels de Vos 2017-03-08 10:54:13 UTC
Vijay, Amye, could you approve this please?

11:45 <@nigelb> ndevos: 3.8 doesn't have any special permissions. He should be
    able to do them if he can merge onto master.
11:46 <@nigelb> ndevos: If he can't merge onto master, please get ack from
    Vijay + Amye onto the bug.
11:47 < ndevos> nigelb: hmm, no he does not have that, it should only be
    applied for the release-3.8 branch... is that not possible?
11:48 <@nigelb> It's granting commit access to gluster to someone who doesn't
    have it. And we don't have a process for it.
11:48 <@nigelb> So I'm not working around the lack of a process.
11:49 < ndevos> but setting up the "special permissions" for release-3.8 should
    not be a problem?
11:49  * ndevos just hates to be the SPOF for 3.8 releases
11:49 <@nigelb> Er, why don't you ask someone else who does have commit access
    for help?
11:50 < ndevos> well, jiffin is listed as the co-maintainer for 3.8
11:50 <@nigelb> So it shouldn't be difficult to get sign off from Vijay and
    Amye.
11:50 <@nigelb> So why don't you do that?
11:50 < ndevos> some of the other component maintainers merge patches in 3.8,
    but they do not really take release dates into account
11:51 < ndevos> if that is what is needed, I'll get it, sure, and as maintainer
    of 3.8 I would be surprised to see any objection from them
11:52 <@nigelb> Exactly. So just get it done so it's official.
11:52 <@nigelb> Then we can repeat this in the future as well.

Comment 2 Vijay Bellur 2017-03-08 19:45:44 UTC
(In reply to Niels de Vos from comment #1)
> Vijay, Amye, could you approve this please?
> 


Approved.

Comment 3 Amye Scavarda 2017-03-09 15:11:39 UTC
Approved. 
Also, we need a process for this, but this works for being able to have maintainers choose their own peers to assist them.

Comment 4 Nigel Babu 2017-03-09 15:25:09 UTC
Granted non-exclusive permissions to Niels and Jiffin for 3.8. If this needs to be made exclusive to the two of you (like 3.10), let me know and I can change.

Comment 5 Jiffin 2017-03-10 05:49:59 UTC
Thanks everyone for resolving this issue as quick as possible


Note You need to log in before you can comment on or make changes to this bug.