Glance meters aren't work

Bug #1335891 reported by Anastasia Palkina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Ivan Berezovskiy
5.0.x
Fix Committed
High
Ivan Berezovskiy

Bug Description

"build_id": "2014-06-30_00-31-14",
"mirantis": "yes",
"build_number": "278",
"ostf_sha": "7171951202a22d8dfe9b3b76c9824001e2485630",
"nailgun_sha": "bcb3f0795acbd21f4882103c735a64051492cf7f",
"production": "docker",
"api": "1.0",
"fuelmain_sha": "74ee2ba18e29f81917227767bbf77dfc4c9ffc1d",
"astute_sha": "d1d54d57b3ce1b2f975cbc16c354f1ae7b3391ee",
"release": "5.1",
"fuellib_sha": "c6adc9f28e722f3845da49e45f1e1729d6f6b1a0"

1. Create new environment (Ubuntu, simple mode)
2. Choose Ceilometer
3. Add controller+mongo, compute
4. Start deployment. It was successful
5. Glance meters are't work even after uploading image

root@node-9:~# ceilometer meter-list
+------------+-------+-------+--------------------------------------+---------+----------------------------------+
| Name | Type | Unit | Resource ID | User ID | Project ID |
+------------+-------+-------+--------------------------------------+---------+----------------------------------+
| image | gauge | image | 4887d920-570c-4b17-9d0c-a6d8b9dcebf8 | None | 5567b99d45804deebd1814bc16439970 |
| image.size | gauge | B | 4887d920-570c-4b17-9d0c-a6d8b9dcebf8 | None | 5567b99d45804deebd1814bc16439970 |
+------------+-------+-------+--------------------------------------+---------+----------------------------------+

root@node-9:~# ceilometer meter-list | grep image
| image | gauge | B | fbe53d1d-3351-47f8-beb3-8d28e90bdac5 | None | 5488ab13d19b4675bc66955bda08ee18 |
| image | gauge | image | 50cc7bf6-7f56-439d-b234-f0e9eb52ed6e | None | 5488ab13d19b4675bc66955bda08ee18 |
| image | gauge | image | fbe53d1d-3351-47f8-beb3-8d28e90bdac5 | None | 5488ab13d19b4675bc66955bda08ee18 |
| image.size | gauge | B | 50cc7bf6-7f56-439d-b234-f0e9eb52ed6e | None | 5488ab13d19b4675bc66955bda08ee18 |
| image.size | gauge | B | fbe53d1d-3351-47f8-beb3-8d28e90bdac5 | None | 5488ab13d19b4675bc66955bda08ee18 |

Revision history for this message
Anastasia Palkina (apalkina) wrote :
Changed in fuel:
importance: Undecided → High
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Also image and image.size come with a delay of 5 minutes after booting image.

Changed in fuel:
assignee: MOS Ceilometer (mos-ceilometer) → Ivan Berezovskiy (iberezovskiy)
Dina Belova (dbelova)
Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/104183

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.0)

Reviewed: https://review.openstack.org/104183
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=cd6e3bb316999aa1035aac37d207001dc42305ba
Submitter: Jenkins
Branch: stable/5.0

commit cd6e3bb316999aa1035aac37d207001dc42305ba
Author: iberezovskiy <email address hidden>
Date: Wed Jul 2 17:04:44 2014 +0400

    Fix glance meters for ceilometer to stable/5.0

    Change-Id: Ia1891593f57de6533da0479074423fd0b73e7f52
    Closes-bug: #1335891

Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :

Glance control_exchange in glance-api.conf wasn't set and glance used default value for this - 'openstack':

DEBUG glance.common.config [-] control_exchange = openstack log_opt_values /usr/lib/python2.6/site-packages/oslo/config/cfg.py:1945

 So, exchange 'glance' and queue 'glance' didn't exist in rabbit. But ceilometer as default glance control exchange uses 'glance'. So meters weren't get by ceilometer. To fix this we need to add control_exchange=glance to glance-api.conf

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

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

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

Change abandoned by Ivan Berezovskiy (<email address hidden>) on branch: master
Review: https://review.openstack.org/103827
Reason: https://review.openstack.org/#/c/107288/

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

Summary of current state of fixes for this bug:

https://review.openstack.org/104183 sets Glance rabbit_notification_exchange and Ceilometer glance_control_exchange to "openstack" and was merged into stable/5.0 on July 4. Glance control_exchange is not set in stable/5.0 which leaves it at oslo.messaging default of "openstack". The rabbit_notification_exchange in Glance (stable/icehouse) is only present in etc/glance-api.conf and isn't actually used for anything since Glance moved to oslo.messaging, so the first part of that change is meaningless but harmless.

https://review.openstack.org/107288 targeted for fuel-library master sets Glance control_exchange to "glance", leaves Glance rabbit_notification_exchange at its current state of explicitly setting it to "glance" (once again meaningless but harmless and matches recently merged upstream), and leaves Ceilometer glance_control_exchange not set, which leaves it at Ceilometer's default of "glance".

I like the master version of this fix much better so I'm going to merge it even though it deviates from what was done in stable/5.0. Please create a backport of https://review.openstack.org/107288 with the same Change-Id that brings stable/5.0 in sync with the current state of this fix for master.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/107288
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=f52b79e1d2a5094b9fca0fbab29c5e98a8d1eaf0
Submitter: Jenkins
Branch: master

commit f52b79e1d2a5094b9fca0fbab29c5e98a8d1eaf0
Author: iberezovskiy <email address hidden>
Date: Wed Jul 16 13:53:35 2014 +0400

    Set glance control_exchange to 'glance'

    Glance control_exchange in glance-api.conf wasn't set
    and glance used default value for this - 'openstack'.
    So, exchange 'glance' and queue 'glance' didn't exist in rabbit.
    But ceilometer as default glance control exchange uses 'glance'.
    This patch fixes this problem.

    Change-Id: Iac363ba3f95a300c416dbfd94e37d21f5f9cef40
    Closes-bug: #1335891

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

P.S. Please also contribute the control_exchange patch to upstream puppetlabs-glance.

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

Note for QA: please don't close this until this change (comment #10) is proposed to upstream.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-library (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/108462

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

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

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

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

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

Change abandoned by Aleksandr Didenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/108588
Reason: Wrong review

Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/108462
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=850f9d5e6fc53ffc99b179054099ad3dcd8c0c7e
Submitter: Jenkins
Branch: master

commit 850f9d5e6fc53ffc99b179054099ad3dcd8c0c7e
Author: Bogdan Dobrelya <email address hidden>
Date: Mon Jul 21 20:22:42 2014 +0300

    Revert "Set glance control_exchange to 'glance'"

    This reverts commit
    f52b79e1d2a5094b9fca0fbab29c5e98a8d1eaf0.
    We should not change upstream code in manifests.

    Change-Id: Idd1566dd507878b2f01c2d0ff9956dedc77a4461
    Related-bug: #1335891

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/108463
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=b483189c848351ab277d20f208e6c97a986a98ea
Submitter: Jenkins
Branch: master

commit b483189c848351ab277d20f208e6c97a986a98ea
Author: Bogdan Dobrelya <email address hidden>
Date: Mon Jul 21 20:27:25 2014 +0300

    Set glance control_exchange to 'glance'

    Glance control_exchange in glance-api.conf wasn't set
    and glance used default value for this - 'openstack'.
    So, exchange 'glance' and queue 'glance' didn't exist in rabbit.
    But ceilometer as default glance control exchange uses 'glance'.
    This patch fixes the fix for this problem and change-id as well

    Closes-bug: #1335891

    Change-Id: Idd1566dd507878b2f01c2d0ff9956dedc77a4461
    Signed-off-by: Bogdan Dobrelya <email address hidden>

Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #366
"build_id": "2014-07-28_02-01-14",
"ostf_sha": "8c328521b1444f22c50463b9432193e20ed33813",
"build_number": "366",
"auth_required": true,
"api": "1.0",
"nailgun_sha": "83cc9ed44ebc8dd97248483b6d414ebbc4cff3c0",
"production": "docker",
"fuelmain_sha": "9adfbf5a52cedbdd16ec1a74f6c44c5b3419b87c",
"astute_sha": "aa5aed61035a8dc4035ab1619a8bb540a7430a95",
"feature_groups": ["mirantis"],
"release": "5.1",
"fuellib_sha": "d1c7f67b3cf51978d3178c8666ea398f2477dcb5"

Changed in fuel:
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.