[go: nahoru, domu]

Skip to content
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

Merged
merged 1 commit into from
Aug 10, 2015

Conversation

codenrhoden
Copy link
Contributor

This seems like a much more accurate list to me.

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>
Copy link

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.

Copy link
Contributor Author

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?

Copy link
Contributor Author

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
Copy link
Member
dmick commented Jul 24, 2015
  1. not clear to me that this is appropriate for upstream in general. That information looks old, but 2) Debian has fairly-opaque conventions about who and what and what responsibilities there are for those addresses.

@codenrhoden
Copy link
Contributor Author

@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.

@dmick
Copy link
Member
dmick commented Jul 24, 2015

Could be right. the flavor I meant to express was "I don't know".

@codenrhoden
Copy link
Contributor Author

Took me a while to look up James's nick. @javacruft any opinions on the proper convention here?

@javacruft
Copy link
Contributor

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

@javacruft
Copy link
Contributor

@trhoden completely agree with your comment that the Uploaders field should be populated with the uploaders to ceph.com (not to Debian or Ubuntu).

@codenrhoden
Copy link
Contributor Author

@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?

@ghost
Copy link
ghost commented Aug 10, 2015

@trhoden I agree with what you propose.

@ktdreyer
Copy link
Member

Ok let's do it

@ktdreyer ktdreyer merged commit 6f54c61 into ceph:master Aug 10, 2015
ktdreyer added a commit that referenced this pull request Aug 10, 2015
debian: Update maintainers and uploaders

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants