Ceilometer

Ceilometer-api can't start after changing local language

Bug #1446983 reported by Lan Qi song on 2015-04-22

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Ceilometer	Fix Released	Undecided	Lan Qi song	Ceilometer 5.0.0 "liberty"
	Kilo	Fix Released	Medium	Unassigned	Ceilometer 2015.1.0 "kilo"

Bug Description

After change local language to ja, ceilometer-api failed to start with the following error:

2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/retrying.py", line
49, in wrapped_f
2015-04-21 10:35:44.917 25729 TRACE ceilometer return Retrying(*dargs, **dkw).call(f, *args, **kw)
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/retrying.py", line
212, in call
2015-04-21 10:35:44.917 25729 TRACE ceilometer raise attempt.get()
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/retrying.py", line
247, in get
2015-04-21 10:35:44.917 25729 TRACE ceilometer six.reraise(self.value[0], self.value[1], self.value[2]
)
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/retrying.py", line
200, in call
2015-04-21 10:35:44.917 25729 TRACE ceilometer attempt = Attempt(fn(*args, **kwargs), attempt_number,
False)
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/ceilometer/storage
/__init__.py", line 116, in _inner
2015-04-21 10:35:44.917 25729 TRACE ceilometer return get_connection(url, namespace)
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/ceilometer/storage
/__init__.py", line 128, in get_connection
2015-04-21 10:35:44.917 25729 TRACE ceilometer {'name': engine_name,'namespace': namespace}))
2015-04-21 10:35:44.917 25729 TRACE ceilometer File "/usr/lib/python2.7/site-packages/oslo_i18n/_message
.py", line 112, in __mod__
2015-04-21 10:35:44.917 25729 TRACE ceilometer unicode_mod = super(Message, self).__mod__(params)
2015-04-21 10:35:44.917 25729 TRACE ceilometer ValueError: unsupported format character '?' (0xff52) at in
dex 22

Instead use "%" in log message, prefer to use "," :
http://docs.openstack.org/developer/oslo.i18n/guidelines.html#adding-variables-to-log-messages

The code:
https://github.com/openstack/ceilometer/blob/master/ceilometer/storage/__init__.py#L126-L127

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-04-22: Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/176191

Changed in ceilometer:
assignee:	nobody → Lan Qi song (lqslan)
status:	New → In Progress

gordon chung (chungg) on 2015-04-22

tags:

added: kilo-backport-potential

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-04-22: Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/176191
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=f2ab05ac3dc95981ce183277a52e8c395a4cdbeb
Submitter: Jenkins
Branch: master

commit f2ab05ac3dc95981ce183277a52e8c395a4cdbeb
Author: Lan Qi song <email address hidden>
Date: Wed Apr 22 15:05:35 2015 +0800

Fix valueerror when ceilometer-api start

If change local language to other language(like ja), ceilometer-api
will failed to start.

This patch will remove i18 support from ceilometer/storage/__init__.py

Change-Id: I8dc7ff0921d69e64a41e588207c97548513c99ed
Closes-Bug: #1446983

Changed in ceilometer:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-04-23: Fix proposed to ceilometer (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/176572

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-04-23: Fix merged to ceilometer (stable/kilo)

Reviewed: https://review.openstack.org/176572
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=2004c4cb563a8adb38b2498711ff8a24c2897e49
Submitter: Jenkins
Branch: stable/kilo

commit 2004c4cb563a8adb38b2498711ff8a24c2897e49
Author: Lan Qi song <email address hidden>
Date: Wed Apr 22 15:05:35 2015 +0800

Fix valueerror when ceilometer-api start

If change local language to other language(like ja), ceilometer-api
will failed to start.

This patch will remove i18 support from ceilometer/storage/__init__.py

    Change-Id: I8dc7ff0921d69e64a41e588207c97548513c99ed
    Closes-Bug: #1446983
    (cherry picked from commit f2ab05ac3dc95981ce183277a52e8c395a4cdbeb)

Thierry Carrez (ttx) on 2015-04-24

tags:

removed: kilo-backport-potential

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-04-30: Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/179290

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-06-10: Fix merged to ceilometer (master)

Download full text (3.5 KiB)

Reviewed: https://review.openstack.org/179290
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=b192f69f166317bbf048886f3a78d68fc89e64d1
Submitter: Jenkins
Branch: master

commit b0447ed8e7bee371bf7095c86e47d717abe89edc
Author: Chris Dent <email address hidden>
Date: Thu Apr 23 14:01:35 2015 +0000

Have eventlet monkeypatch the time module

    Without this, mongod retry logic in the various services started as
    commands fails to behave as expected and does not reconnect as soon as
    the mongod service has returned to availability.

    In addition to the mongod sleep there are two other time.sleep calls
    that may be reached by this change. Review and discussion with others
    indicates that their behavior will continue to be correct with the
    monkeypatch in place.

Cherry-pick from https://review.openstack.org/#/c/176751/

Change-Id: I4eca290acc3b06658951f070935ebb39936e13d3
Closes-Bug: #1447599

commit 2a0ecfa6e8246352376f463b3237017b529ca464
Author: OpenStack Proposal Bot <email address hidden>
Date: Thu Apr 23 02:12:27 2015 +0000

Updated from global requirements

Change-Id: Ia56c489de41738e318d00f04698a68a149bf3ac5

commit 2004c4cb563a8adb38b2498711ff8a24c2897e49
Author: Lan Qi song <email address hidden>
Date: Wed Apr 22 15:05:35 2015 +0800

Fix valueerror when ceilometer-api start

If change local language to other language(like ja), ceilometer-api
will failed to start.

This patch will remove i18 support from ceilometer/storage/__init__.py

    Change-Id: I8dc7ff0921d69e64a41e588207c97548513c99ed
    Closes-Bug: #1446983
    (cherry picked from commit f2ab05ac3dc95981ce183277a52e8c395a4cdbeb)

commit f5b994829bbd19794eba062ea79599f3b78a9286
Author: ZhiQiang Fan <email address hidden>
Date: Fri Apr 17 17:06:13 2015 +0800

use message id to generate hbase unique key

    We currently can disable computing signature for metering message,
    but HBase still uses signature as a dimension to generate unique row
    key, this will lead to different samples treated as same one.

Message id should be used in stead of message signature, because
id will always exist and also unique.

    Change-Id: I47cdce59c9934573076cf609ce1a0c37aea75c44
    Closes-Bug: #1445227
    (cherry picked from commit 15a92ec1ef2ab3e6d1d187ad909a0d5800ee4a2e)

commit 446cd1440beda53e4216dc5da362fd74ded4f1df
Author: Mehdi Abaakouk <email address hidden>
Date: Wed Apr 15 14:31:24 2015 +0200

gnocchi: fix typo in the aggregation endpoint

Closes-bug: #1446201

Change-Id: I92a727b518f77ad7149fc43a564adea5e8e97240
(cherry picked from commit d1130e810d40ecd553a325fa5c32626be944625c)

commit dc6f4bfecbde54f663e96f485a948944223ef231
Author: Andreas Jaeger <email address hidden>
Date: Mon Apr 20 11:13:57 2015 +0200

Release Import of Translations from Transifex

    Manual import of Translations from Transifex. This change also removes
    all po files that are less than 66 per cent translated since such
    partially translated files will not help users.

This change needs to be don...

Reviewed:  https://review.openstack.org/179290
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=b192f69f166317bbf048886f3a78d68fc89e64d1
Submitter: Jenkins
Branch:    master

commit b0447ed8e7bee371bf7095c86e47d717abe89edc
Author: Chris Dent <chdent@redhat.com>
Date:   Thu Apr 23 14:01:35 2015 +0000

Have eventlet monkeypatch the time module
    
    Without this, mongod retry logic in the various services started as
    commands fails to behave as expected and does not reconnect as soon as
    the mongod service has returned to availability.
    
    In addition to the mongod sleep there are two other time.sleep calls
    that may be reached by this change. Review and discussion with others
    indicates that their behavior will continue to be correct with the
    monkeypatch in place.
    
    Cherry-pick from https://review.openstack.org/#/c/176751/
    
    Change-Id: I4eca290acc3b06658951f070935ebb39936e13d3
    Closes-Bug: #1447599

commit 2a0ecfa6e8246352376f463b3237017b529ca464
Author: OpenStack Proposal Bot <openstack-infra@lists.openstack.org>
Date:   Thu Apr 23 02:12:27 2015 +0000

Updated from global requirements
    
    Change-Id: Ia56c489de41738e318d00f04698a68a149bf3ac5

commit 2004c4cb563a8adb38b2498711ff8a24c2897e49
Author: Lan Qi song <lqslan@cn.ibm.com>
Date:   Wed Apr 22 15:05:35 2015 +0800

Fix valueerror when ceilometer-api start
    
    If change local language to other language(like ja), ceilometer-api
    will failed to start.
    
    This patch will remove i18 support from ceilometer/storage/__init__.py
    
    Change-Id: I8dc7ff0921d69e64a41e588207c97548513c99ed
    Closes-Bug: #1446983
    (cherry picked from commit f2ab05ac3dc95981ce183277a52e8c395a4cdbeb)

commit f5b994829bbd19794eba062ea79599f3b78a9286
Author: ZhiQiang Fan <zhiqiang.fan@huawei.com>
Date:   Fri Apr 17 17:06:13 2015 +0800

use message id to generate hbase unique key
    
    We currently can disable computing signature for metering message,
    but HBase still uses signature as a dimension to generate unique row
    key, this will lead to different samples treated as same one.
    
    Message id should be used in stead of message signature, because
    id will always exist and also unique.
    
    Change-Id: I47cdce59c9934573076cf609ce1a0c37aea75c44
    Closes-Bug: #1445227
    (cherry picked from commit 15a92ec1ef2ab3e6d1d187ad909a0d5800ee4a2e)

commit 446cd1440beda53e4216dc5da362fd74ded4f1df
Author: Mehdi Abaakouk <mehdi.abaakouk@enovance.com>
Date:   Wed Apr 15 14:31:24 2015 +0200

gnocchi: fix typo in the aggregation endpoint
    
    Closes-bug: #1446201
    
    Change-Id: I92a727b518f77ad7149fc43a564adea5e8e97240
    (cherry picked from commit d1130e810d40ecd553a325fa5c32626be944625c)

commit dc6f4bfecbde54f663e96f485a948944223ef231
Author: Andreas Jaeger <aj@suse.de>
Date:   Mon Apr 20 11:13:57 2015 +0200

Release Import of Translations from Transifex
    
    Manual import of Translations from Transifex. This change also removes
    all po files that are less than 66 per cent translated since such
    partially translated files will not help users.
    
    This change needs to be done manually since the automatic import does
    not handle the proposed branches and we need to sync with latest
    translations.
    
    Change-Id: Ieab43148a8d77d5896f02dd96d54097cc3d2e37b

commit 4af4f5d5db88b68d88ccf53c4e2dbf5fa7203865
Author: Thierry Carrez <thierry@openstack.org>
Date:   Fri Apr 17 10:32:59 2015 +0200

update .gitreview for stable/kilo
    
    Change-Id: I376d1df0fd9cd6b11de27bb876ab96c6c4cfdb0d

Thierry Carrez (ttx) on 2015-06-23

Changed in ceilometer:
milestone:	none → liberty-1
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2015-10-15

Changed in ceilometer:
milestone:	liberty-1 → 5.0.0

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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