Merge lp:~zulcss/ceilometer/deps-fixes into lp:~openstack-ubuntu-testing/ceilometer/grizzly
Proposed by
Chuck Short
on 2012-12-05
| Status: | Needs review |
|---|---|
| Proposed branch: | lp:~zulcss/ceilometer/deps-fixes |
| Merge into: | lp:~openstack-ubuntu-testing/ceilometer/grizzly |
| Diff against target: |
114 lines (+44/-27) 4 files modified
debian/ceilometer-common.postinst (+1/-1) debian/changelog (+11/-0) debian/control (+29/-26) debian/watch (+3/-0) |
| To merge this branch: | bzr merge lp:~zulcss/ceilometer/deps-fixes |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Adam Gandelman | 2012-12-05 | Needs Fixing on 2012-12-05 | |
|
Review via email:
|
|||
Description of the Change
Dependency clean up and update.
To post a comment you must log in.
Unmerged revisions
- 5. By Chuck Short on 2012-12-05
-
debian/control: Add python-pymongo dep.
- 4. By Chuck Short on 2012-12-04
-
debian/control: Drop python-ming not needed
- 3. By Chuck Short on 2012-12-04
-
debian/control: Clean up and update pythhon-
dependencies - 2. By Chuck Short on 2012-11-22
-
* debian/
ceilometer- common. postinst: Add ceilometer to the user group
so that it can read its configuration files.
* debian/watch: Added watch file.


Don't think this will work. In ceilometer- common. postinst, the call to adduser with '--ingroup ceilometer,nova' will fail in cases where the nova group doesn't exist. Looking at the nova package, this group gets created by the nova-common package, but the only nova dependency ceilometer has is on python-nova.