cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eric Evans (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-3198) debian packaging installation problem when installing for the first time
Date Wed, 14 Sep 2011 19:04:09 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-3198?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13104787#comment-13104787
] 

Eric Evans commented on CASSANDRA-3198:
---------------------------------------

{quote}
Hum, the goal of this ticket is that anyone that have started using the deb package after
0.6.4-2 have /var/{lib,log}/cassandra owned by root.root, which was not intended. So we can
decide that we want to leave all those people with those "wrong" permissions, but it's not
safer to remove the comparison now that it was when the patch was initially committed.
{quote}

I think that statement was always bugged.  What it _should_ do is to ensure that /var/{lib,log}/cassandra
are owned by user cassandra on new installs only, not upgrades (allowing the user to override
that ownership).

At this point, I'm not sure what the comparison to 0.6.4-2 was supposed to accomplish, I assume
it was a (misguided )attempt at applying this policy to versions greater than or equal to
0.6.4-2 (we didn't always create the cassandra user and chown in postinst).  Either way, the
current packaging will never be used to upgrade from an 0.6.x release directly, so there is
no danger in removing it.

So, looking at that (untested )snippet again:

{noformat}
if [ -z "$2" ]; then
    chown -R cassandra: /var/lib/cassandra
    chown -R cassandra: /var/log/cassandra
fi
{noformat}

would only apply ownership changes if argument #2 is empty, that is to say, there is no previously
configured version (i.e. not an upgrade).

> debian packaging installation problem when installing for the first time
> ------------------------------------------------------------------------
>
>                 Key: CASSANDRA-3198
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3198
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Packaging
>    Affects Versions: 0.8.5
>            Reporter: Jérémy Sevellec
>            Assignee: Jérémy Sevellec
>             Fix For: 0.8.6
>
>         Attachments: debian-postinst-fixperms.patch, trunk-3198-v1.patch
>
>
> when installing cassandra through the debian packaging for the first time, there is permission
problem when starting Cassandra.
> Normally, the postinst script change owner of /var/log/cassandra and /var/lib/cassandra
from root to cassandra user.
> there is a problem with the test which verify if threre is a need to change the owner
of these directory or not.
> On a new install, the $2 parameter is not set and the the test is false and the owner
is not changed.
> (simply, i think replace "&&" with "||" might work)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message