ceilometerclient needs a new release

Bug #1423291 reported by Pavlo Shchelokovskyy
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
High
Pavlo Shchelokovskyy
python-ceilometerclient
Fix Released
High
Eoghan Glynn

Bug Description

the latest as of now version 1.0.12 was released beginning of Oct 2014. Currently due to keystone AuthPlugin changes Heat can not create Ceilometer alarms when using 1.0.12 version of ceilometerclient

2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/opt/stack/heat/heat/engine/resources/ceilometer/alarm.py", line 264, in handle_create
2015-02-18 11:01:26.983 TRACE heat.engine.resource alarm = self.ceilometer().alarms.create(**props)
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/v2/alarms.py", line 156, in create
2015-02-18 11:01:26.983 TRACE heat.engine.resource return self._create(self._path(), new)
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/common/base.py", line 53, in _create
2015-02-18 11:01:26.983 TRACE heat.engine.resource body = self.api.post(url, json=body).json()
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 337, in post
2015-02-18 11:01:26.983 TRACE heat.engine.resource return self.client_request("POST", url, **kwargs)
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 328, in client_request
2015-02-18 11:01:26.983 TRACE heat.engine.resource self, method, url, **kwargs)
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 227, in client_request
2015-02-18 11:01:26.983 TRACE heat.engine.resource self.authenticate()
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/client.py", line 298, in authenticate
2015-02-18 11:01:26.983 TRACE heat.engine.resource self.auth_plugin.authenticate(self)
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/openstack/common/apiclient/auth.py", line 189, in authenticate
2015-02-18 11:01:26.983 TRACE heat.engine.resource self.sufficient_options()
2015-02-18 11:01:26.983 TRACE heat.engine.resource File "/usr/local/lib/python2.7/dist-packages/ceilometerclient/client.py", line 214, in sufficient_options
2015-02-18 11:01:26.983 TRACE heat.engine.resource raise exceptions.AuthPluginOptionsMissing(missing_opts)
2015-02-18 11:01:26.983 TRACE heat.engine.resource AuthPluginOptionsMissing: Authentication failed. Missing options: endpoint, username, password, tenant_name

at the same time master branch works just fine. Please make a release to PyPI and bump global requirements.

Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

added Heat to affected projects to keep track of it

Revision history for this message
Sergey Kraynev (skraynev) wrote :

Same log we can see on the gate job, when try to use ceilometer alrams:

https://jenkins05.openstack.org/job/check-heat-dsvm-functional-mysql/577/console
(https://review.openstack.org/#/c/153192/)

Note, that using ceilometer client from master fixes it (I have checked it locally).

Angus Salkeld (asalkeld)
Changed in heat:
status: New → Triaged
importance: Undecided → High
milestone: none → kilo-3
Dina Belova (dbelova)
Changed in python-ceilometerclient:
importance: Undecided → High
status: New → Triaged
Eoghan Glynn (eglynn)
Changed in python-ceilometerclient:
assignee: nobody → Eoghan Glynn (eglynn)
Revision history for this message
Eoghan Glynn (eglynn) wrote :
Changed in python-ceilometerclient:
status: Triaged → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Changed in heat:
assignee: nobody → Haiwei Xu (xu-haiwei)
status: Triaged → In Progress
Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

as we know that it is not working with previous version, would you kindly propose a bump to global requirements so that it propagates to Heat?

Revision history for this message
Haiwei Xu (xu-haiwei) wrote :

Thanks Pavlo, I will try it.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on heat (master)

Change abandoned by xu-haiwei (<email address hidden>) on branch: master
Review: https://review.openstack.org/159002
Reason: The bug has been fixed in Ceilometerclient side.

Angus Salkeld (asalkeld)
Changed in heat:
milestone: kilo-3 → kilo-rc1
Haiwei Xu (xu-haiwei)
Changed in heat:
assignee: Haiwei Xu (xu-haiwei) → nobody
Angus Salkeld (asalkeld)
Changed in heat:
milestone: kilo-rc1 → liberty-1
tags: added: kilo-rc-potential
Revision history for this message
Angus Salkeld (asalkeld) wrote :

Is there anything to be done here on the Heat side? Can we just wait for the requirements sync?

Changed in heat:
status: In Progress → Triaged
status: Triaged → Incomplete
milestone: liberty-1 → none
Revision history for this message
Sergey Kraynev (skraynev) wrote :

@Angus: I have uploaded related patch to global requirements. So wait sync. https://review.openstack.org/#/c/167527/

Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

Unfortunately Sean Dague has blocked the global requirements update due to requirements freeze. Can we somehow still push it through? We are dead sure that Heat in kilo won't work with <=1.0.12

Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

bot proposal for requirements update is merging, will update the status once successfull

Changed in heat:
milestone: none → kilo-rc1
assignee: nobody → Pavlo Shchelokovskyy (pshchelo)
status: Incomplete → Confirmed
status: Confirmed → In Progress
Revision history for this message
Pavlo Shchelokovskyy (pshchelo) wrote :

requirements bump is merged in Heat

https://review.openstack.org/#/c/170389/

minimal supported version is set to a working one. I am marking the Heat part as Fix Commited

Changed in heat:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
tags: removed: kilo-rc-potential
Thierry Carrez (ttx)
Changed in heat:
milestone: kilo-rc1 → 2015.1.0
Changed in python-ceilometerclient:
milestone: none → 1.3.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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