No way to restrict generating meter_links from Ceilometer CLI

Bug #1508708 reported by Rohit Jaiswal
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Triaged
Medium
Rohit Jaiswal

Bug Description

This affects stable/kilo.

[1] was added to kilo to enable control over generating meter_links from client. However, links is enabled by default for backward compatibility, but there is really no way to disable that from CLI since [2] doesnt pass a value for links.

[1] https://review.openstack.org/#/c/221353/
[2] https://github.com/openstack/python-ceilometerclient/blob/stable/kilo/ceilometerclient/v2/shell.py#L999

The resource-list shell should probably include an optional CLI argument to disable links, since currently its enabled by default.

This doesnt affect stable/liberty and master, since meter_links is disabled by default, the problem there is that it cannot be enabled.

Changed in python-ceilometerclient:
assignee: nobody → Rohit Jaiswal (rohit-jaiswal-3)
summary: - No way to restrict generating meter_links from client
+ No way to restrict generating meter_links from Ceilometer CLI
description: updated
ZhiQiang Fan (aji-zqfan)
Changed in python-ceilometerclient:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ceilometerclient (stable/kilo)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-ceilometerclient (stable/kilo)

Change abandoned by Rohit Jaiswal (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/238654
Reason: Abandoning in favor of cherry-picked patch from master: https://review.openstack.org/#/c/242297/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ceilometerclient (stable/kilo)

Reviewed: https://review.openstack.org/242297
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=19e3ad0149439f3228676db503b3116a312f359e
Submitter: Jenkins
Branch: stable/kilo

commit 19e3ad0149439f3228676db503b3116a312f359e
Author: Rohit Jaiswal <email address hidden>
Date: Fri Oct 23 14:46:23 2015 +0000

    Fix to disable meter-links on resource-list CLI

    This adds an optional param to resource-list
    CLI to disable meter-links. By default its
    False to generate meter links for backward
    compatibility.

    This is a cherry-pick from master patch that
    enables meter-links on resource-list CLI.

    Change-Id: I308b94c8a57833cf30b4815ecec2ff9a50e91af6
    Closes-Bug: #1508708
    (cherry picked from commit e4a62a20b1d072969e842c38b2885a46a20fc54c)

tags: added: in-stable-kilo
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/python-ceilometerclient 1.1.2

This issue was fixed in openstack/python-ceilometerclient 1.1.2 release.

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

This issue was fixed in the openstack/python-ceilometerclient 1.1.2 release.

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.