add functional testing for pollster service

Bug #1424821 reported by gordon chung
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Triaged
High
Unassigned
Kiwi Linux
New
Undecided
Unassigned

Bug Description

placeholder to add testing for polling agents.... right now we have no idea if anything is broken unless we look at logs.

gordon chung (chungg)
Changed in ceilometer:
milestone: kilo-3 → none
jiaxi (tjxiter)
Changed in ceilometer:
assignee: nobody → jiaxi (tjxiter)
jiaxi (tjxiter)
Changed in ceilometer:
assignee: jiaxi (tjxiter) → nobody
Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

I have an idea of using CL to get the information of polling agent,If we write a polling agent,we can use the cmd line to get the information,for example,ceilometer --polling_agent cpu interval 60(cpu is the name of agent which is defined in the setup.cfg).

Revision history for this message
gordon chung (chungg) wrote :

what is a 'CL'?

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

i mean comman line,we can use cliff to generate the command line and use the CL to test the polling agent.

Changed in ceilometer:
assignee: nobody → Qichen Hao (adsl123gg2008)
Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

I think we can add a command in ceilometer python client--poll,just like 'ceilometer poll --namesapce=ceilometer.poll.compute --module=cpu',the command will print the message generated by polling agent.

Revision history for this message
gordon chung (chungg) wrote :

to be honest, this would be quite complicated as these are two different services. it would go from client -> API -> polling agent and back... and there is currently no way to for API to communicate with polling agents. integration full stack testing might be the easiest and most realistic test

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

yes,it's quite complicated,but if we launch a new ceilometer compute|central service in ceilometer python client,and just loading the plugin using DriverManager in stevedore.I know this will increasing the load,but it's rare situations in developing,so we can destroy the service if the developers don't use the poll command,and we will reduce the burden as much as we can.

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

More easier,we can just load the pollster plugin that developers write and the discovery plugin.

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

hi,gordon,Why can't we implement the function in ceilometer API,in the api code ,we can just load the pollster plugin that developers write and the discovery plugin.I think it's easier than full stack testing.

no longer affects: python-ceilometerclient
Revision history for this message
gordon chung (chungg) wrote :

i'm not sure what you mean by API service here. do you mean the REST API service?

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

yes,REST API,when you request the ceilometer api service,the specific function will load the pollster plugin that developers write to get the samples produced by the pollster plugin.

Revision history for this message
gordon chung (chungg) wrote :

but the REST api does not communicate with polling agent?

Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

now the polling agents are managed by agentManager,and the aim of this bug is adding the function test for specifically polling agent(one or more),so we can add a function which loads the polling agent,and the function returns the result of the polling agent.Access the functon and get the result is achieved by ceilometer REST api.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

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

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in ceilometer:
status: In Progress → Fix Committed
Revision history for this message
gordon chung (chungg) wrote :

not merged.

your solution does not actually test the polling agent. it just tests that the pollsters it loads does something.

Changed in ceilometer:
status: Fix Committed → Triaged
Revision history for this message
Qichen Hao (adsl123gg2008) wrote :

so you want to konw what happened in the process of starting the polling agent service? including whether all of the pollsters are started correctly.

Changed in ceilometer:
assignee: Qichen Hao (adsl123gg2008) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ceilometer (master)

Change abandoned by gordon chung (<email address hidden>) on branch: master
Review: https://review.openstack.org/294900
Reason: REST api is deprecated

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by gordon chung (<email address hidden>) on branch: master
Review: https://review.openstack.org/294898
Reason: api is deprecated

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.