Merge lp://staging/~zulcss/ceilometer/deps-refresh into lp://staging/~openstack-ubuntu-testing/ceilometer/grizzly
Proposed by
Chuck Short
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://staging/~zulcss/ceilometer/deps-refresh |
| Merge into: | lp://staging/~openstack-ubuntu-testing/ceilometer/grizzly |
| Diff against target: |
82 lines (+15/-24) 3 files modified
debian/changelog (+2/-0) debian/control (+13/-23) debian/pydist-overrides (+0/-1) |
| To merge this branch: | bzr merge lp://staging/~zulcss/ceilometer/deps-refresh |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Adam Gandelman (community) | Needs Fixing | ||
|
Review via email:
|
|||
Description of the change
deps refresh
To post a comment you must log in.
Unmerged revisions
- 21. By Chuck Short
-
Streamline a bit more
- 20. By Chuck Short
-
Fix typo
- 19. By Chuck Short
-
* debian/control: Refresh build deps.
* debian/pydist- overrides: Dropped no longer needed.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks like a typo?
57 - python-lxml,
58 + python-lml,
python-flask is dropped but a quick grep thru the ceilometer source shows it is used all over, still, and it is listed in tools/pip-requires ming is still referenced as an import in a couple of tests.
The python-{swift, glance, nova} dependencies are dropped but quickly grepping thru code shows imports directly from the server code. tools/pip-requires says otherwise. I don't know enough about ceilometer to know whether or not they are actually needed.