Able to list the L3 stats of neutron meter labels for either ingress or egress direction but NOT both.

[Koteswara Rao Kelam]

I created meter-label and added rule to it but i am not able to list the meters.

1. neutron meter-label-create --tenant-id xxx --description "label for tenant xxx" label_xxx
2. neutron meter-label-rule-create --direction ingress --tenant-id xxx label_xxx 10.64.201.0/24
3. To get the counters from ceilometer:
    ceilometer sample-list -m bandwidth -q project_id=xxx -l 10

following error observed in /var/log/neutron/metering_agent.log

2014-04-21 04:33:39.400 19954 ERROR neutron.openstack.common.notifier.api [req-0ad3e301-7b7e-4740-b210-893a7fc1b49a None] Failed to load notifier neutron.openstack.common.notifier.list_notifier. These notifications will not be sent.
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api Traceback (most recent call last):
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api File "/usr/lib/python2.7/dist-packages/neutron/openstack/common/notifier/api.py", line 168, in add_driver
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api driver = importutils.import_module(notification_driver)
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api File "/usr/lib/python2.7/dist-packages/neutron/openstack/common/importutils.py", line 57, in import_module
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api __import__(import_str)
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api ImportError: No module named list_notifier
2014-04-21 04:33:39.400 19954 TRACE neutron.openstack.common.notifier.api

Regards,
Kotewsar

[Koteswara Rao Kelam (koti-kelam)]

Always adding subnet prifix as destination (-d option in iptables rules) but not considering it adding as source (-s option)
Made the samw change and summitted for review.

iptables -L -n -x -v:
-----------------------------
.............
Chain neutron-meter-r-094aae08-3c6 (1 references)
    pkts bytes target prot opt in out source destination
       2 168 neutron-meter-l-094aae08-3c6 all -- * qg-1da591c6-da 0.0.0.0/0 192.212.0.0/24<<<<<<<<cdir added as destination

Chain neutron-meter-r-6b8ec0fb-6b6 (1 references)
    pkts bytes target prot opt in out source destination
       0 0 neutron-meter-l-6b8ec0fb-6b6 all -- * qg-1da591c6-da 0.0.0.0/0 10.10.10.0/24<<<<<<<<cdir added as destination

Chain neutron-meter-r-94a32949-aea (1 references)
    pkts bytes target prot opt in out source destination
       2 168 neutron-meter-l-94a32949-aea all -- qg-1da591c6-da * 0.0.0.0/0 10.10.10.0/24<<<<<<<<<cdir added as destination
..................

[Sylvain Afchain (sylvain-afchain)]

Hi,

Could you add more use cases, with the neutron meter command you use and the result expected ?

[Nachi Ueno (nati-ueno)]

This one looks like neutron side bug

[Kevin Benton (kevinbenton)]

what is the configuration value for notification_driver in neutron.conf?

[Koteswara Rao Kelam (koti-kelam)]

notification_driver = neutron.openstack.common.notifier.rpc_notifier

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/95526
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=72e8b5dc5a5e36722327efca4c399f0321e9bbbb
Submitter: Jenkins
Branch: master

commit 72e8b5dc5a5e36722327efca4c399f0321e9bbbb
Author: Koteswara Rao Kelam <email address hidden>
Date: Mon May 26 06:44:33 2014 -0700

    Add -s option for neutron metering rules

    While adding iptables rule, cidr is added as destination for both ingress
    and egress directions. Modified code to add -s for egress and -d for ingress.

    Closes-bug: 1310589

    Change-Id: Id9ca10855e6527d4bec689f8f9bcd6f681221954

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/107851

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/107851
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=3ad288d7a38c8aacad0134930624c1aae58c6429
Submitter: Jenkins
Branch: stable/icehouse

commit 3ad288d7a38c8aacad0134930624c1aae58c6429
Author: Koteswara Rao Kelam <email address hidden>
Date: Mon May 26 06:44:33 2014 -0700

    Add -s option for neutron metering rules

    While adding iptables rule, cidr is added as destination for both ingress
    and egress directions. Modified code to add -s for egress and -d for ingress.

    Closes-bug: 1310589

    Conflicts:
            neutron/tests/unit/services/metering/drivers/test_iptables_driver.py

    Change-Id: Id9ca10855e6527d4bec689f8f9bcd6f681221954
    (cherry-picked from commit 72e8b5dc5a5e36722327efca4c399f0321e9bbbb)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/114851

[OpenStack Infra (hudson-openstack)]

Change abandoned by Alan Pevec (<email address hidden>) on branch: stable/havana
Review: https://review.openstack.org/114851