log dir permissions are incorrect for use with swift

Bug #1474667 reported by Xav Paice
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
ceilometer (Ubuntu)
Won't Fix
Low
Unassigned
Trusty
Won't Fix
Low
Unassigned
Utopic
Won't Fix
Low
Unassigned
Vivid
Won't Fix
Low
Unassigned
Wily
Won't Fix
Low
Unassigned

Bug Description

[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

Tags: patch sts
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in ceilometer (Ubuntu):
status: New → Confirmed
Revision history for this message
Xav Paice (xavpaice) wrote :

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.

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :
Changed in ceilometer (Ubuntu Trusty):
status: New → In Progress
Changed in ceilometer (Ubuntu Utopic):
status: New → In Progress
Changed in ceilometer (Ubuntu Vivid):
status: New → In Progress
Changed in ceilometer (Ubuntu Trusty):
assignee: nobody → Jorge Niedbalski (niedbalski)
Changed in ceilometer (Ubuntu Utopic):
assignee: nobody → Jorge Niedbalski (niedbalski)
Changed in ceilometer (Ubuntu Vivid):
assignee: nobody → Jorge Niedbalski (niedbalski)
Changed in ceilometer (Ubuntu Wily):
assignee: nobody → Jorge Niedbalski (niedbalski)
tags: added: sts
description: updated
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

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

tags: added: patch
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

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.

Revision history for this message
Xav Paice (xavpaice) wrote :

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.

James Page (james-page)
Changed in ceilometer (Ubuntu Wily):
importance: Undecided → Low
Changed in ceilometer (Ubuntu Vivid):
importance: Undecided → Low
Changed in ceilometer (Ubuntu Utopic):
importance: Undecided → Low
Changed in ceilometer (Ubuntu Trusty):
importance: Undecided → Low
summary: - log dir permissions are incorrect
+ log dir permissions are incorrect for use with swift
James Page (james-page)
Changed in ceilometer (Ubuntu Wily):
status: Confirmed → Triaged
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

Hi,

has any progress been made on this?

Revision history for this message
Xav Paice (xavpaice) wrote :

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.

Revision history for this message
Chris J Arges (arges) wrote :

Unsubscribing sponsored until comment #8 is addressed.

Revision history for this message
Rolf Leggewie (r0lf) wrote :

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

Changed in ceilometer (Ubuntu Utopic):
status: In Progress → Won't Fix
James Page (james-page)
Changed in ceilometer (Ubuntu Vivid):
status: In Progress → Won't Fix
Changed in ceilometer (Ubuntu Wily):
status: Triaged → Won't Fix
Changed in ceilometer (Ubuntu Trusty):
status: In Progress → New
Changed in ceilometer (Ubuntu):
assignee: Jorge Niedbalski (niedbalski) → nobody
Changed in ceilometer (Ubuntu Trusty):
assignee: Jorge Niedbalski (niedbalski) → nobody
Changed in ceilometer (Ubuntu Utopic):
assignee: Jorge Niedbalski (niedbalski) → nobody
Changed in ceilometer (Ubuntu Vivid):
assignee: Jorge Niedbalski (niedbalski) → nobody
Changed in ceilometer (Ubuntu Wily):
assignee: Jorge Niedbalski (niedbalski) → nobody
Changed in ceilometer (Ubuntu Trusty):
status: New → Won't Fix
Changed in ceilometer (Ubuntu):
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.