xenapi: image upload does not need to propagate usage_agent key

Bug #1204933 reported by John Garbutt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Brian Elliott

Bug Description

Bad use of system metadata means xenapi_use_agent key is never propagated to the snapshots.

However compute.api will automatically inherit all image properties that are saved into system metadata, so this logic can be removed.

Tags: xenserver
Changed in nova:
status: New → Triaged
importance: Undecided → Medium
Changed in nova:
status: Triaged → In Progress
Revision history for this message
John Garbutt (johngarbutt) wrote :
summary: - xenapi: image upload fails to add usage_agent key
+ xenapi: image upload does not need to propagate usage_agent key
description: updated
Changed in nova:
assignee: John Garbutt (johngarbutt) → Brian Elliott (belliott)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/38637
Committed: http://github.com/openstack/nova/commit/79420db5c9aa2d29c78058a38a82a5fa049bf1e8
Submitter: Jenkins
Branch: master

commit 79420db5c9aa2d29c78058a38a82a5fa049bf1e8
Author: John Garbutt <email address hidden>
Date: Thu Jul 25 15:36:24 2013 +0100

    xenapi: remove propagate xenapi_use_agent key

    The compute.api class will already propagate then xenapi_use_agent key
    when required, so we can remove the xenapi specific code that tries to
    propagate this value. The code did not correctly access the system
    metadata, would have failed to propagate the key anyway.

    Fixes bug 1204933
    Change-Id: Ib291a265318d80705129e565155d4250fd704b75

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-3 → 2013.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.