log dir permissions are incorrect for use with swift

[Xav Paice]

[Impact]

In ceilometer-common.postinst, permissions for the dir /var/log/ceilometer are set to 750.

In http://docs.openstack.org/developer/ceilometer/install/manual.html it's stated to "Note Please make sure that ceilometer’s logging directory (if it’s configured) is read and write accessible for the user swift is started by."

That means the perms need to be 770, not 750.

[Test Case]

 * Install ceilometer package
 * Install swift package
 * This prevents parts of swift from logging, where it uses ceilometer's egg to log info to ceilometer about usage.

[Regression Potential]

 * None expected

[Other Info]

* None

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Xav Paice (xavpaice)]

I'm happy to submit a patch if someone could please point me at how? I looked at https://wiki.ubuntu.com/ServerTeam/OpenStack#Submitting_a_Patch but get "bzr: ERROR: development focus https://api.launchpad.net/1.0/ceilometer/liberty has no branch" in return. No doubt some newbie problem here, a point in the right direction would be great.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "Wily Patch" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

[Marc Deslauriers (mdeslaur)]

The "adm" group's purpose is to give read access to log files. I don't think it's appropriate to have members of that group be able to write into the log directory.

NACK on this approach, some other way of handling this is required.

[Xav Paice (xavpaice)]

I see your point, and agree in principal, but the effect of this is that we cannot use the Ubuntu Cloud Archive until Swift is changed to use the Ceilometer code in some other way. That's a pretty significant change, and a diversion from the approach described by the devs of Ceilometer in their own documentation.

[Serge Hallyn (serge-hallyn)]

Hi,

has any progress been made on this?

[Xav Paice (xavpaice)]

comment #8 was pretty clear that this isn't regarded as something that needs fixing. If that's still the case, this should be closed as wontfix.

I don't know another way round it though - and if changing the package is the wrong approach I would like to know the right approach so we can take the preferred action.

[Chris J Arges (arges)]

Unsubscribing sponsored until comment #8 is addressed.

[Rolf Leggewie (r0lf)]

utopic has seen the end of its life and is no longer receiving any updates. Marking the utopic task for this ticket as "Won't Fix".