-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
debian: Update maintainers and uploaders #5344
Conversation
Signed-off-by: Travis Rhoden <trhoden@redhat.com>
@@ -4,8 +4,9 @@ Priority: optional | |||
Homepage: http://ceph.com/ | |||
Vcs-Git: git://github.com/ceph/ceph.git | |||
Vcs-Browser: https://github.com/ceph/ceph | |||
Maintainer: Laszlo Boszormenyi (GCS) <gcs@debian.hu> | |||
Uploaders: Sage Weil <sage@newdream.net> | |||
Maintainer: Ceph Maintainers <ceph-maintainers@lists.ceph.com> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to add the current Debian maintainer in the Uploaders.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dachary But does Laszlo have access to Jenkins and our VPN to upload packages to ceph.com?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And to be clear, I'm not arguing at all. I'm not sure what the convention is here. Just because someone could do it years ago, does that mean they stay here forever?
|
@dmick Yeah, this is definitely more of a proposal than something that has to be done. I look at it and think "well that's not right". I found the convention/policy pretty straightforward: https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Maintainer The maintainer should definitely be the list, IMO. Since it's a list, we need real people in Uploaders. Who has access to upload packages to ceph.com (and ceph.com only). Ubuntu/debian distro-hosted packages are separately controlled by Laszlo and James Page and others. Really, that means Alfredo and Ken to me. So I view this as how has the ability to upload packages to upstream ceph.com, completely independent of what may happen downstream. |
Could be right. the flavor I meant to express was "I don't know". |
Took me a while to look up James's nick. @javacruft any opinions on the proper convention here? |
Typically the Maintainer: field is fine as a team; policy dictates that the Uploaders field should be populated with individuals. For example - https://anonscm.debian.org/cgit/pkg-ceph/ceph.git/tree/debian/control#n4 |
@trhoden completely agree with your comment that the Uploaders field should be populated with the uploaders to ceph.com (not to Debian or Ubuntu). |
@ktdreyer I think this is good as-is, based on other comments. The maintainers change is a no brainer. The uploaders field should be "who can do this right now". The point is to be able to see who can do this with a given package version. If a maintainer isn't around anymore and uploading packages, they shouldn't be in this list. @dachary still have feelings against this? |
@trhoden I agree with what you propose. |
Ok let's do it |
debian: Update maintainers and uploaders Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
This seems like a much more accurate list to me.