XenServer port needs to clear out vm-data/networking before issuing resetnetwork command

Bug #747394 reported by Johannes Erdfelt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Low
Unassigned

Bug Description

The guest agent will pull networking information from vm-data/networking in xenstore. If Nova chooses a different name than what is already in that directory directory, the guest agent will apply all configurations, even if they are old or conflicting.

Tags: xenserver

Related branches

Thierry Carrez (ttx)
Changed in nova:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Joel wineland (joel-wineland) wrote : Re: [Bug 747394] Re: XenServer port needs to clear out vm-data/networking before issuing resetnetwork command

Thierry Carrez <email address hidden> wrote:

** Changed in: nova
   Importance: Undecided => Medium

** Changed in: nova
       Status: New => Confirmed

--
You received this bug notification because you are subscribed to
OpenStack.
https://bugs.launchpad.net/bugs/747394

Title:
  XenServer port needs to clear out vm-data/networking before issuing
  resetnetwork command

Status in OpenStack Compute (Nova):
  Confirmed

Bug description:
  The guest agent will pull networking information from vm-
  data/networking in xenstore. If Nova chooses a different name than
  what is already in that directory directory, the guest agent will
  apply all configurations, even if they are old or conflicting.

Confidentiality Notice: This e-mail message (including any attached or
embedded documents) is intended for the exclusive and confidential use of the
individual or entity to which this message is addressed, and unless otherwise
expressly indicated, is confidential and privileged information of Rackspace.
Any dissemination, distribution or copying of the enclosed material is prohibited.
If you receive this transmission in error, please notify us immediately by e-mail
at <email address hidden>, and delete the original message.
Your cooperation is appreciated.

Changed in nova:
assignee: nobody → Johannes Erdfelt (johannes.erdfelt)
Thierry Carrez (ttx)
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
Thierry Carrez (ttx) wrote :

@Johannes: still working on that ? If yes could you refresh your merge proposal with the latest comments and trunk ? If no, please unassign yourself from the bug so that someone else can have a try.

Revision history for this message
Johannes Erdfelt (johannes.erdfelt) wrote :

The bug is fixed, but the changes necessary to the test suite are what is holding this up. I haven't had a chance to refactor the code to handle the changes necessary to fix this bug.

I may have some time to work on that this week.

Revision history for this message
Thierry Carrez (ttx) wrote :

@Johannes: did you have time to refresh that one ?

Revision history for this message
Mark McLoughlin (markmc) wrote :

de-assigning since this is pretty stale now

Changed in nova:
status: In Progress → Confirmed
assignee: Johannes Erdfelt (johannes.erdfelt) → nobody
tags: added: xenserver
Revision history for this message
John Garbutt (johngarbutt) wrote :
Revision history for this message
John Garbutt (johngarbutt) wrote :

Sorry, wrong bug

Revision history for this message
John Garbutt (johngarbutt) wrote :

Given the moves towards cloud-init this is much less important, and does seemed to have caused any problems so far.

Changed in nova:
importance: Medium → Low
Revision history for this message
John Garbutt (johngarbutt) wrote :

Hmm, thinking about this, its related to issues with inject_network_info.

Currently people call inject_network_info, and potentially alter xenstore, then call resetnetwork.

So, sadly, this behaviour is now being used as a feature.

Changed in nova:
status: Confirmed → Invalid
Revision history for this message
Johannes Erdfelt (johannes.erdfelt) wrote :

It looks like it's fixed rather than invalid.

When VIFs are removed from an instance, the xenstore entry is removed too (this wasn't always the case).

So that effectively fixes the problem, which probably also explains why we haven't seen this problem in a while.

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.