Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1430303

Summary: Allow Jiffin to merge patches to the glusterfs/release-3.8 branch
Product: [Community] GlusterFS Reporter: Niels de Vos <ndevos>
Component: project-infrastructureAssignee: bugs <bugs>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: mainlineCC: amye, bugs, gluster-infra, jthottan, nigelb, vbellur
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-03-09 15:25:09 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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