https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2016-06-03T15:03:36ZCeph Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=720032016-06-03T15:03:36ZAlfredo Dezaadeza@redhat.com
<ul><li><strong>Assignee</strong> set to <i>Alfredo Deza</i></li></ul> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=720062016-06-03T16:05:14ZAlfredo Dezaadeza@redhat.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Fix Under Review</i></li></ul><p>Pull Request: <a class="external" href="https://github.com/ceph/ceph/pull/9485">https://github.com/ceph/ceph/pull/9485</a></p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818752016-11-23T11:02:58ZLoïc Dacharyloic@dachary.org
<ul></ul><p>@alfredo could you explain the purpose of this change in this context:</p>
<pre>
commit ca164160a3628582eecf54023898c8568cd2f5ca
Author: Alfredo Deza <adeza@redhat.com>
Date: Fri Jun 3 11:05:20 2016 -0400
ceph-disk: point to the authors file for contributors
Signed-off-by: Alfredo Deza <adeza@redhat.com>
diff --git a/src/ceph-disk/ceph_disk/main.py b/src/ceph-disk/ceph_disk/main.py
index 2328c96..27fc262 100755
--- a/src/ceph-disk/ceph_disk/main.py
+++ b/src/ceph-disk/ceph_disk/main.py
@@ -1,12 +1,10 @@
#!/usr/bin/env python
#
-# Copyright (C) 2015, 2016 Red Hat <contact@redhat.com>
+# Copyright (C) 2015-2016 Red Hat <contact@redhat.com> Authors
# Copyright (C) 2014 Inktank <info@inktank.com>
# Copyright (C) 2014 Cloudwatt <libre.licensing@cloudwatt.com>
# Copyright (C) 2014 Catalyst.net Ltd
#
-# Author: Loic Dachary <loic@dachary.org>
-#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU Library Public License as published by
# the Free Software Foundation; either version 2, or (at your option)
@@ -17,6 +15,7 @@
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU Library Public License for more details.
#
+# See the AUTHORS file for names of contributors
from __future__ import print_function
</pre> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818782016-11-23T13:26:05ZAlfredo Dezaadeza@redhat.com
<ul></ul><p>I think there was a discussion on what 'authors' should be at some point. I don't remember all the details (this ticket is six months old).</p>
<p>Since ceph-disk has been worked on by so many, do you feel an 'AUTHORS' list doesn't better reflect the contributors?</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818812016-11-23T14:27:52ZNathan Cutlerncutler@suse.cz
<ul></ul><p>I just did "egrep -r 'Author:' src/" in a ceph/ceph clone and saw that there are many lines similar to this one in the tree.</p>
<p>Alfredo, can you explain why this one needs to be dropped and not the others?</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818822016-11-23T14:33:04ZNathan Cutlerncutler@suse.cz
<ul></ul><p>Not to diminish anyone's contribution, but Loic did author over half of the lines:</p>
<pre>
$ wc --lines src/ceph-disk/ceph_disk/main.py
5146 src/ceph-disk/ceph_disk/main.py
$ git blame src/ceph-disk/ceph_disk/main.py | grep 'Loic Dachary' | wc --lines
2818
</pre> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818832016-11-23T14:36:25ZAlfredo Dezaadeza@redhat.com
<ul></ul><p>I didn't think a whole-project change was the right thing to do. I worked on ceph-disk for this ticket, so I updated this one file.</p>
<p>It wasn't my intention to make it controversial. It felt odd to have a single author listed there since many have contributed to it.</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=818992016-11-23T19:19:03ZNathan Cutlerncutler@suse.cz
<ul></ul><p>But removing other people's names without asking is always going to be controversial, right?</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=819012016-11-23T19:41:50ZAlfredo Dezaadeza@redhat.com
<ul></ul><p>I am not removing anyone's name: Loic's name is still in the Authors file.</p>
<p>This is not something I am doing without asking, that is exactly why there is a pull request for this and a ticket, it is out in the open, so others<br />can comment and give feedback (like you are in this ticket).</p>
<p>If you think my explanations aren't sufficient, could you propose (in the PR maybe?) what is it that you think would be more correct? Even if that means to avoid changing<br />the Author (which, again, I was not removing)</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=819322016-11-23T20:58:50ZLoïc Dacharyloic@dachary.org
<ul></ul><p>For the record I added <strong>Author: Loic Dachary</strong> in files where the contribution I made was covered by copyright (i.e. non trivial) for two reasons. Back when I was working for CloudWatt it clearly identified in which part my contribution was significant and was instrumental to convince my management to let me work on Ceph rather than divert me to other tasks. The other reason is that under copyright law (at least for all signatories of the Berne convention) the individual author has a moral right (which cannot be forfeited in France, even by contract) to display his name on his work. Although it could be mistaken as me claiming to be the sole author of the file for someone ignorant of revision control, it is not unreasonable to assume people reading the sources are well informed on that matter. This is not something I ever explained anywhere before because I don't think anyone thought of removing my name from a source file until now. Other authors added their names here and there, maybe for similar reasons, maybe for other reasons. Although it leads to a non uniform and somewhat chaotic display of copyright and authors, it is a status quo that did not raise any concern so far, even when Red Hat lawyers audited the code back when Inktank was acquired.</p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=828372016-12-09T13:09:00ZAlfredo Dezaadeza@redhat.com
<ul></ul><p>I have made the requested changes on the branch, and opened a new PR at <a class="external" href="https://github.com/ceph/ceph/pull/12414">https://github.com/ceph/ceph/pull/12414</a></p> Ceph - Bug #16125: ceph-disk should not spit out a traceback when a subprocess.call errorshttps://tracker.ceph.com/issues/16125?journal_id=828602016-12-09T13:39:52ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Status</strong> changed from <i>Fix Under Review</i> to <i>Resolved</i></li></ul>