cells: hypervisor API extension can't find compute_node services

Bug #1427351 reported by Andrew Laski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
High
Sylvain Bauza

Bug Description

After the conversion to use Service objects in the hypervisor API extension the lookups for services are happening in the parent cell, not the child cells. This is due to cells redirects not being implemented in the Service object.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
assignee: nobody → Andrew Laski (alaski)
status: New → In Progress
Changed in nova:
importance: Undecided → Low
Changed in nova:
assignee: Andrew Laski (alaski) → Sylvain Bauza (sylvain-bauza)
Changed in nova:
assignee: Sylvain Bauza (sylvain-bauza) → Andrew Laski (alaski)
Changed in nova:
assignee: Andrew Laski (alaski) → Sylvain Bauza (sylvain-bauza)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Changed in nova:
assignee: Sylvain Bauza (sylvain-bauza) → RedBaron (dheeraj-gupta4)
melanie witt (melwitt)
tags: added: cells
Changed in nova:
assignee: RedBaron (dheeraj-gupta4) → Sylvain Bauza (sylvain-bauza)
Changed in nova:
milestone: none → kilo-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Dheeraj Gupta (<email address hidden>) on branch: master
Review: https://review.openstack.org/167564
Reason: This issue has now been addressed by https://review.openstack.org/157185

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

don't think we should block the kilo release on this bug, removing from kilo-rc1

Changed in nova:
milestone: kilo-rc1 → none
Changed in nova:
assignee: Sylvain Bauza (sylvain-bauza) → Andrew Laski (alaski)
Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

As discussed during Nova meeting, attempt to merge it by RC1 as it would help the cells job to be green.

Changed in nova:
milestone: none → kilo-rc1
Changed in nova:
importance: Low → High
tags: added: kilo-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to nova (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/171306

Changed in nova:
assignee: Andrew Laski (alaski) → Sylvain Bauza (sylvain-bauza)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.openstack.org/171306
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=471699d140da84af28624cfee078ad8dd80b0f52
Submitter: Jenkins
Branch: master

commit 471699d140da84af28624cfee078ad8dd80b0f52
Author: Sylvain Bauza <email address hidden>
Date: Tue Apr 7 12:58:21 2015 +0200

    Use cells.utils.ServiceProxy object within cells_api

    Now that all proxy objects are provided by cells.utils, we can directly use
    them instead of using a duplicate object.

    Change-Id: I184e143495140bf96381e6958fa68280da15d8d6
    Related-Bug: #1427351

Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

Moving it to critical as it blocks the cells job to be green for Kilo

The patch needed to be merged is https://review.openstack.org/160506

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

This can't be critical as it doesn't affect all users.

I am unsure about blocking the release on this bug, given we already warn people in the logs that cells is untested.

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

Moving this out of RC1, its something we might want for RC2, to prove cells goes green in kilo-stable, but at this point, it seems unlikely to get merged.

Changed in nova:
milestone: kilo-rc1 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/160506
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=c689923c036d0a5292a8a31be3f4f94311eb5314
Submitter: Jenkins
Branch: master

commit c689923c036d0a5292a8a31be3f4f94311eb5314
Author: Sylvain Bauza <email address hidden>
Date: Mon Mar 2 14:24:33 2015 -0500

    Fix how service objects are looked up for Cells

    When querying for services in the hypervisor API extension the queries
    need to be proxied through to the child cells.

    Note: We need to proxy the calls thru the HostAPI in order to have the
    same method for both non-cells and cells.

    Co-Authored-By: Andrew Laski <email address hidden>

    Change-Id: I3ff3976d24aef108a990fb8ff64afd29c96a1e2f
    Closes-Bug: 1427351

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