Merge lp://qastaging/~yolanda.robla/ceilometer/grizzly into lp://qastaging/~openstack-ubuntu-testing/ceilometer/grizzly
Proposed by
Yolanda Robla
| Status: | Merged |
|---|---|
| Merged at revision: | 22 |
| Proposed branch: | lp://qastaging/~yolanda.robla/ceilometer/grizzly |
| Merge into: | lp://qastaging/~openstack-ubuntu-testing/ceilometer/grizzly |
| Diff against target: |
97 lines (+25/-16) 5 files modified
debian/ceilometer-common.postinst (+5/-9) debian/changelog (+4/-5) debian/control (+2/-2) debian/patches/default-dbconnection-sqlite.patch (+13/-0) debian/patches/series (+1/-0) |
| To merge this branch: | bzr merge lp://qastaging/~yolanda.robla/ceilometer/grizzly |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| James Page | Approve | ||
|
Review via email:
|
|||
Description of the change
Default to sqlite
To post a comment you must log in.
Revision history for this message
| James Page (james-page) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Yolanda
1) modifying configuration files in postinst
Please don't do it this way; instead patch the database_connection into the configuration file in the source tree and ship it as the default configuration file. That way dpkg takes care of maintaining end-user changes to this file.
2) run dbsync as ceilometer user
- su -s /bin/sh -c '/usr/bin/
+ su -s /bin/sh -c '/usr/bin/