Ticket #3135 (new task)

Opened 5 years ago

Last modified 4 years ago

pkgs git repo doesn't allow deleting one off branches

Reported by: rstrode Owned by: cvsadmin-members,sysadmin-cvs-members
Priority: minor Milestone: Fedora 17
Component: SCM (Source Code Management) Version:
Severity: Normal Keywords:
Cc: ausil Blocked By:
Blocking: Sensitive:


<halfline> jkeating: hey any idea why i get (hook declined) when trying to delete rstrode/wip/just-for-airlied branch of gnome-shell (in fedora) <halfline> (or should i ask #fedora-admin) <jkeating> halfline: probably because the Fedora ACLs weren't setup to allow branch deletion. Easy enough to fix, it just hasn't come up yet <halfline> jkeating: is the acl thing something you can fix straight away? or should i file a trac ticket or something along those lines ? <jkeating> I'd suggest a trac ticket. I haven't touched that system in that way in a while, and it may require some code modification since the ACLs are generated via cron <jkeating> well the trac ticket to allow branch deletion. Also a trac ticket to request that particular branch be whacked in the short term <halfline> k, sounds good <halfline> i don't need a short term fix <halfline> it's not like the branch is hurting anything

Change History

comment:1 in reply to: ↑ description ; follow-up: ↓ 2 Changed 5 years ago by rstrode

ugh sorry about the lack of newlines, I guess trac strips them.

comment:2 in reply to: ↑ 1 Changed 5 years ago by rstrode

jkeating points out that right now we don't enforce the requirement that official builds come from official branches, so allowing deleting branches could complicate the audit trail back to source.

A couple thoughts:

1) we should probably require official builds to come from official branches or 2) koji should probably tag the git repo whenever it does a successful build. that would prevent "important" hashes from getting GCd or 3) we should disallow custom branches, since they don't seem to be that popular of a featurea anyway.

comment:3 Changed 4 years ago by kevin

  • Cc ausil added

FESCo and/or rel-eng might want to weigh in here...

Adding Dennis for comment.

Note: See TracTickets for help on using tickets.