[RFE] No notification on floating ip status change

Bug #1593793 reported by Steve McLellan
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Searchlight
New
High
Unassigned
neutron
Won't Fix
Wishlist
Boden R

Bug Description

Tested on neutron/master in devstack.

When I associate or disassociate a floating IP I get a notification indicating the new fixed IP and port association, but the status still indicates the current value (so for associate, the notification contains "fixed_ip_address":"a.b.c.d", "status": "DOWN") because there's a short latency time before the FIP is marked as ACTIVE.

I see the status change after a very short period after in the q-svc log:

    New status for floating IP 8470e622-0f3b-419f-ac21-deed658d3260: ACTIVE

There's no notification emitted representing this state change, however. The same is true on disassociate; the port and fixed IP are cleared but the status is indicated ACTIVE and there's no notification marking it DOWN.

E.g.
$ neutron floatingip-create 0e97853d-3ac0-4c08-a8b3-f6bfa443dd24

{"event_type": "floatingip.create.end", "payload": {"floatingip": {"router_id": null, "status": "DOWN", "description": "", "tenant_id": "c1b853bf78bc43c69daa8d42cb9fb07f", "floating_network_id": "0e97853d-3ac0-4c08-a8b3-f6bfa443dd24", "fixed_ip_address": null, "floating_ip_address": "172.25.0.11", "port_id": null, "id": "6442165e-4caa-437c-9377-196f4db638f9"}}, "publisher_id": "network.devstack", "ctxt": {...}, "metadata": {"timestamp": "2016-06-17 16:44:20.600943", "message_id": "547b0f34-1bc0-4fbc-8e58-a0c2d2dc1751"}}

$ neutron floatingip-associate 6442165e-4caa-437c-9377-196f4db638f9 8068c884-1ab4-42d9-97d0-f65848fca2b0

{"event_type": "floatingip.update.end", "payload": {"floatingip": {"router_id": "390ab366-0eee-46e9-a07f-f49cf0b31652", "status": "DOWN", "description": "", "tenant_id": "c1b853bf78bc43c69daa8d42cb9fb07f", "floating_network_id": "0e97853d-3ac0-4c08-a8b3-f6bfa443dd24", "fixed_ip_address": "172.40.0.4", "floating_ip_address": "172.25.0.11", "port_id": "8068c884-1ab4-42d9-97d0-f65848fca2b0", "id": "6442165e-4caa-437c-9377-196f4db638f9"}}, "publisher_id": "network.devstack", "ctxt": {...}, "metadata": {"timestamp": "2016-06-17 16:44:50.149631", "message_id": "f6f7ec1f-6363-49b0-b10f-84fd81e0a506"}}

Steve McLellan (sjmc7)
description: updated
Changed in neutron:
status: New → Triaged
importance: Undecided → Wishlist
Boden R (boden)
Changed in neutron:
assignee: nobody → Boden R (boden)
summary: - No notification on floating ip status change
+ [RFE] No notification on floating ip status change
tags: added: rfe
Revision history for this message
Boden R (boden) wrote :

I assume this one was discussed in the neutron meeting and decided it should be an RFE??

Adding notifications for FIP seemed pretty straight forward to me, and hence I was assuming it could move straight to tech discussions via patch reviews.

Revision history for this message
Akihiro Motoki (amotoki) wrote :

At now, notification events are only sent when API operations are made.

This request is to send notification events for resource changes from other reasons. It sounds reasonable, but it is better to generalize the change to all other events (for example, port status).

Technically, I think it can be achieved by the event registry.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Btw I encountered something similar with DHCP notifications. I ended up solving this with [1], however in this case when the agent comes back and update the floating IP status, only the DB field is updated without putting a message on the notification queue. As Akihiro suggested this is achievable via a registry callback notification and the nuances of this can be dealt with code review.

[1] https://review.openstack.org/#/c/329006/

tags: removed: rfe
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

@Boden: go for it.

Boden R (boden)
Changed in neutron:
status: Triaged → In Progress
summary: - [RFE] No notification on floating ip status change
+ No notification on floating ip status change
Boden R (boden)
summary: - No notification on floating ip status change
+ [RFE] No notification on floating ip status change
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

It seems the patch that actually fixes the bug is not being posted yet.

Changed in neutron:
milestone: none → ocata-1
Steve McLellan (sjmc7)
Changed in searchlight:
importance: Undecided → High
Revision history for this message
Boden R (boden) wrote :

Armando RE comment #6:

Why is [1] not a fix for this (albeit stalled waiting on a dependent patch)?

[1] https://review.openstack.org/#/c/345654/

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

I didn't say that, I said it is not complete.

Changed in neutron:
milestone: ocata-1 → ocata-2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/345654
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in neutron:
milestone: ocata-2 → ocata-3
Changed in neutron:
milestone: ocata-3 → ocata-rc1
Changed in neutron:
milestone: ocata-rc1 → ocata-rc2
Changed in neutron:
milestone: ocata-rc2 → pike-1
tags: added: rfe
tags: removed: rfe
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Kevin Benton (<email address hidden>) on branch: master
Review: https://review.openstack.org/345654
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in neutron:
milestone: pike-1 → pike-2
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Bug closed due to lack of activity, please feel free to reopen if needed.

Changed in neutron:
status: In Progress → Won't Fix
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.