Merge lp:~zulcss/ceilometer/bug-catchup into lp:~ubuntu-server-dev/ceilometer/icehouse

Proposed by Chuck Short
Status: Merged
Approved by: Chuck Short
Approved revision: 83
Merged at revision: 82
Proposed branch: lp:~zulcss/ceilometer/bug-catchup
Merge into: lp:~ubuntu-server-dev/ceilometer/icehouse
Diff against target: 203 lines (+92/-7)
12 files modified
debian/ceilometer-agent-central.upstart (+1/-1)
debian/ceilometer-agent-compute.upstart (+1/-1)
debian/ceilometer-agent-notification.upstart (+1/-1)
debian/ceilometer-alarm-evaluator.install (+1/-0)
debian/ceilometer-alarm-evaluator.upstart (+17/-0)
debian/ceilometer-alarm-notifier.install (+1/-0)
debian/ceilometer-alarm-notifier.upstart (+17/-0)
debian/ceilometer-api.upstart (+1/-1)
debian/ceilometer-collector.upstart (+1/-1)
debian/ceilometer-common.install (+0/-2)
debian/changelog (+13/-0)
debian/control (+38/-0)
To merge this branch: bzr merge lp:~zulcss/ceilometer/bug-catchup
Reviewer Review Type Date Requested Status
James Page Needs Information
Review via email: mp+201041@code.launchpad.net

Description of the change

Ceilometer update

To post a comment you must log in.
Revision history for this message
James Page (james-page) wrote :

Typos in both upstart configs:

+stop on runlelvel [016]

Package description:

+ This package contains the arlarm notifier daemon.

Ideally short descriptions should start with a Capital letter.

review: Needs Fixing
Revision history for this message
James Page (james-page) wrote :

Attributation for logrotate files is incorrect, also typo:

Add logroate files,

review: Needs Fixing
Revision history for this message
James Page (james-page) wrote :

Does this actually fix bug 1261101 ?

review: Needs Information
lp:~zulcss/ceilometer/bug-catchup updated
83. By Chuck Short

debian/ceilometer-collector.logrotate: Add logrotate files,
thanks to Ahmed Rahal. (LP: #1224223)
* Fix typos in upstart files.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/ceilometer-agent-central.upstart'
--- debian/ceilometer-agent-central.upstart 2013-01-31 15:49:38 +0000
+++ debian/ceilometer-agent-central.upstart 2014-01-09 17:03:06 +0000
@@ -2,7 +2,7 @@
2author "Chuck Short <zulcss@ubuntu.com>"2author "Chuck Short <zulcss@ubuntu.com>"
33
4start on runlevel [2345]4start on runlevel [2345]
5stop on runlelvel [!2345]5stop on runlevel [!2345]
66
7chdir /var/run7chdir /var/run
88
99
=== modified file 'debian/ceilometer-agent-compute.upstart'
--- debian/ceilometer-agent-compute.upstart 2013-01-31 15:49:38 +0000
+++ debian/ceilometer-agent-compute.upstart 2014-01-09 17:03:06 +0000
@@ -2,7 +2,7 @@
2author "Chuck Short <zulcss@ubuntu.com>"2author "Chuck Short <zulcss@ubuntu.com>"
33
4start on runlevel [2345]4start on runlevel [2345]
5stop on runlelvel [!2345]5stop on runlevel [!2345]
66
7chdir /var/run7chdir /var/run
88
99
=== modified file 'debian/ceilometer-agent-notification.upstart'
--- debian/ceilometer-agent-notification.upstart 2013-12-04 11:17:00 +0000
+++ debian/ceilometer-agent-notification.upstart 2014-01-09 17:03:06 +0000
@@ -2,7 +2,7 @@
2author "Yolanda Robla <yolanda.robla@canonical.com>"2author "Yolanda Robla <yolanda.robla@canonical.com>"
33
4start on runlevel [2345]4start on runlevel [2345]
5stop on runlelvel [!2345]5stop on runlevel [!2345]
66
7chdir /var/run7chdir /var/run
88
99
=== added file 'debian/ceilometer-alarm-evaluator.install'
--- debian/ceilometer-alarm-evaluator.install 1970-01-01 00:00:00 +0000
+++ debian/ceilometer-alarm-evaluator.install 2014-01-09 17:03:06 +0000
@@ -0,0 +1,1 @@
1usr/bin/ceilometer-alarm-evaluator
02
=== added file 'debian/ceilometer-alarm-evaluator.upstart'
--- debian/ceilometer-alarm-evaluator.upstart 1970-01-01 00:00:00 +0000
+++ debian/ceilometer-alarm-evaluator.upstart 2014-01-09 17:03:06 +0000
@@ -0,0 +1,17 @@
1description "ceilometer-alarm-evaluator"
2author "Chuck Short <zulcss@ubuntu.com>"
3
4start on runlevel [2345]
5stop on runlevel [016]
6
7chdir /var/run
8
9pre-start script
10 mkdir -p /var/run/ceilometer
11 chown ceilometer:ceilometer /var/run/ceilometer
12
13 mkdir -p /var/lock/ceilometer
14 chown ceilometer:ceilometer /var/lock/ceilometer
15end script
16
17exec start-stop-daemon --start --chuid ceilometer --exec /usr/bin/ceilometer-alarm-evaluator -- --log-dir=/var/log/ceilometer
018
=== added file 'debian/ceilometer-alarm-notifier.install'
--- debian/ceilometer-alarm-notifier.install 1970-01-01 00:00:00 +0000
+++ debian/ceilometer-alarm-notifier.install 2014-01-09 17:03:06 +0000
@@ -0,0 +1,1 @@
1usr/bin/ceilometer-alarm-notifier
02
=== added file 'debian/ceilometer-alarm-notifier.upstart'
--- debian/ceilometer-alarm-notifier.upstart 1970-01-01 00:00:00 +0000
+++ debian/ceilometer-alarm-notifier.upstart 2014-01-09 17:03:06 +0000
@@ -0,0 +1,17 @@
1description "ceilometer-alarm-notifier"
2author "Chuck Short <zulcss@ubuntu.com>"
3
4start on runlevel [2345]
5stop on runlevel [016]
6
7chdir /var/run
8
9pre-start script
10 mkdir -p /var/run/ceilometer
11 chown ceilometer:ceilometer /var/run/ceilometer
12
13 mkdir -p /var/lock/ceilometer
14 chown ceilometer:ceilometer /var/lock/ceilometer
15end script
16
17exec start-stop-daemon --start --chuid ceilometer --exec /usr/bin/ceilometer-alarm-notifier -- --log-dir=/var/log/ceilometer
018
=== modified file 'debian/ceilometer-api.upstart'
--- debian/ceilometer-api.upstart 2013-01-31 15:49:38 +0000
+++ debian/ceilometer-api.upstart 2014-01-09 17:03:06 +0000
@@ -2,7 +2,7 @@
2author "Chuck Short <zulcss@ubuntu.com>"2author "Chuck Short <zulcss@ubuntu.com>"
33
4start on runlevel [2345]4start on runlevel [2345]
5stop on runlelvel [016]5stop on runlevel [016]
66
7chdir /var/run7chdir /var/run
88
99
=== modified file 'debian/ceilometer-collector.upstart'
--- debian/ceilometer-collector.upstart 2013-01-31 15:49:38 +0000
+++ debian/ceilometer-collector.upstart 2014-01-09 17:03:06 +0000
@@ -2,7 +2,7 @@
2author "Chuck Short <zulcss@ubuntu.com>"2author "Chuck Short <zulcss@ubuntu.com>"
33
4start on runlevel [2345]4start on runlevel [2345]
5stop on runlelvel [!2345]5stop on runlevel [!2345]
66
7chdir /var/run7chdir /var/run
88
99
=== modified file 'debian/ceilometer-common.install'
--- debian/ceilometer-common.install 2013-09-30 13:26:41 +0000
+++ debian/ceilometer-common.install 2014-01-09 17:03:06 +0000
@@ -3,7 +3,5 @@
3etc/ceilometer/policy.json etc/ceilometer3etc/ceilometer/policy.json etc/ceilometer
4etc/ceilometer/sources.json etc/ceilometer4etc/ceilometer/sources.json etc/ceilometer
5tools/show_data.py usr/share/doc/ceilometer5tools/show_data.py usr/share/doc/ceilometer
6usr/bin/ceilometer-alarm-evaluator
7usr/bin/ceilometer-alarm-notifier
8usr/bin/ceilometer-dbsync6usr/bin/ceilometer-dbsync
9usr/bin/ceilometer-expirer7usr/bin/ceilometer-expirer
108
=== modified file 'debian/changelog'
--- debian/changelog 2013-12-18 10:52:40 +0000
+++ debian/changelog 2014-01-09 17:03:06 +0000
@@ -1,7 +1,20 @@
1ceilometer (2014.1~b1+master-0ubuntu1) UNRELEASED; urgency=medium1ceilometer (2014.1~b1+master-0ubuntu1) UNRELEASED; urgency=medium
22
3 [ James Page ]
3 * d/control: Add python-jsonpath-rw to BD's.4 * d/control: Add python-jsonpath-rw to BD's.
4 * d/p/fix-setup-requirements.patch: Bump WebOb to support < 1.4.5 * d/p/fix-setup-requirements.patch: Bump WebOb to support < 1.4.
6 (LP: #1261101)
7
8 [ Chuck Short ]
9 * debian/control, debian/ceilometer-common.install: Split out
10 ceilometer-alarm-evaluator and ceilometer-alarm-notifier into their
11 own packages. (LP: #1250002)
12 * debian/ceilometer-agent-central.logrotate,
13 debian/ceilometer-agent-compute.logrotate,
14 debian/ceilometer-api.logrotate,
15 debian/ceilometer-collector.logrotate: Add logrotate files,
16 thanks to Ahmed Rahal. (LP: #1224223)
17 * Fix typos in upstart files.
518
6 -- James Page <james.page@ubuntu.com> Tue, 17 Dec 2013 15:43:07 +000019 -- James Page <james.page@ubuntu.com> Tue, 17 Dec 2013 15:43:07 +0000
720
821
=== modified file 'debian/control'
--- debian/control 2013-12-18 10:52:40 +0000
+++ debian/control 2014-01-09 17:03:06 +0000
@@ -209,3 +209,41 @@
209 (A ceilometer is an instrument that measures cloud coverage.)209 (A ceilometer is an instrument that measures cloud coverage.)
210 .210 .
211 This package contains the notificaiton agent.211 This package contains the notificaiton agent.
212
213Package: ceilometer-alarm-evaluator
214Architecture: all
215Depends:
216 ceilometer-common (= ${binary:Version}),
217 ${misc:Depends},
218 ${python:Depends}
219Description: Ceilometer alarm evaluator
220 Ceilometer aims to deliver a unique point of contact for billing systems to
221 aquire all counters they need to establish customer billing, accross all
222 current and future OpenStack components. The delivery of counters must
223 be tracable and auditable, the counters must be easily extensible to support
224 new projects, and agents doing data collections should be
225 independent of the overall system.
226 .
227 (A ceilometer is an instrument that measures cloud coverage.)
228 .
229 This package contains the alarm evaluator daemon.
230
231Package: ceilometer-alarm-notifier
232Architecture: all
233Depends:
234 ceilometer-common (= ${binary:Version}),
235 ${misc:Depends},
236 ${python:Depends}
237Description: Ceilometer notification agent
238 Ceilometer aims to deliver a unique point of contact for billing systems to
239 aquire all counters they need to establish customer billing, accross all
240 current and future OpenStack components. The delivery of counters must
241 be tracable and auditable, the counters must be easily extensible to support
242 new projects, and agents doing data collections should be
243 independent of the overall system.
244 .
245 (A ceilometer is an instrument that measures cloud coverage.)
246 .
247 This package contains the arlarm notifier daemon.
248
249

Subscribers

People subscribed via source and target branches