keyerror while updating dhcp port

Bug #1324194 reported by Armando Migliaccio
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Armando Migliaccio

Bug Description

When an update for a dhcp port fails, a keyerror on missing network_id while attempting to log a trace masks the underlying exception. An example is here:

http://logs.openstack.org/91/94891/1/check/check-tempest-dsvm-neutron/c87b4fc/logs/screen-q-svc.txt.gz?level=TRACE

This is because on update, the dhcp agent only sends either this:

https://github.com/openstack/neutron/blob/master/neutron/agent/linux/dhcp.py#L770

or this:

https://github.com/openstack/neutron/blob/master/neutron/agent/linux/dhcp.py#L787

This is somewhat a corner case, but it would be good to address the issue to see what actually went wrong.

Tags: l3-ipam-dhcp
tags: added: l3-ipam-dhcp
Changed in neutron:
assignee: nobody → Armando Migliaccio (armando-migliaccio)
description: updated
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/96226

Changed in neutron:
status: New → In Progress
Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Low
Changed in neutron:
milestone: none → juno-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/96226
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=82683feff64b5b0dff36f983526421e608f13d94
Submitter: Jenkins
Branch: master

commit 82683feff64b5b0dff36f983526421e608f13d94
Author: armando-migliaccio <email address hidden>
Date: Wed May 28 10:37:15 2014 -0700

    Fix KeyError exception while updating dhcp port

    This is caused by a missing network_id in the port body.
    This patch adds it so that a warning message can be traced
    correctly. Wording is slightly tweaked to ensure it applies
    to the right context.

    Closes-bug: #1324194

    Change-Id: I4b67b5c3584aa33278eb0e9e879ca338ba0ca8b0

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
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.