make AggregateAPI work in cells

Bug #1161208 reported by Matthew Sherborne
34
This bug affects 5 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Won't Fix
Medium
Unassigned

Bug Description

Make nova.compute.AggregateAPI work in child cells:

    def create_aggregate(self, context, aggregate_name, availability_zone):
    def get_aggregate(self, context, aggregate_id):
    def get_aggregate_list(self, context):
    def update_aggregate(self, context, aggregate_id, values):
    def update_aggregate_metadata(self, context, aggregate_id, metadata):
    def delete_aggregate(self, context, aggregate_id):
    def add_host_to_aggregate(self, context, aggregate_id, host_name):
    def remove_host_from_aggregate(self, context, aggregate_id, host_name):

Should all work through cells.

Changed in nova:
assignee: nobody → Matthew Sherborne (msherborne+openstack)
status: New → In Progress
Revision history for this message
Matthew Sherborne (msherborne+openstack) wrote :
Download full text (5.4 KiB)

In progress: https://review.openstack.org/#/c/25813/

Todo list (* means to do) (+ means done):
********************************************************************************
Host aggregate working in cells
********************************************************************************
 * compute.cells_api.AggregateAPI
     + create_aggregate(self, ctxt, aggregate_name, availability_zone):
     + get_aggregate(self, ctxt, aggregate_id):
     + get_aggregate_list(self, ctxt, cell=None):
     + update_aggregate(self, ctxt, aggregate_id, values):
     + update_aggregate_metadata(self, ctxt, aggregate_id, metadata):
     + delete_aggregate(self, ctxt, aggregate_id):
     + add_host_to_aggregate(self, ctxt, aggregate_id, host_name):
     + remove_host_from_aggregate(self, ctxt, aggregate_id, host_name):
 * TEST - compute.cells_api.AggregateAPI
     * create_aggregate(self, ctxt, aggregate_name, availability_zone):
     * get_aggregate(self, ctxt, aggregate_id):
     * get_aggregate_list(self, ctxt, cell=None):
     * update_aggregate(self, ctxt, aggregate_id, values):
     * update_aggregate_metadata(self, ctxt, aggregate_id, metadata):
     * delete_aggregate(self, ctxt, aggregate_id):
     * add_host_to_aggregate(self, ctxt, aggregate_id, host_name):
     * remove_host_from_aggregate(self, ctxt, aggregate_id, host_name):
 * nova.cells.manager
     + create_aggregate(self, ctxt, aggregate_name, availability_zone):
     * get_aggregate(self, ctxt, aggregate_id):
     * get_aggregate_list(self, ctxt, cell=None):
     * update_aggregate(self, ctxt, aggregate_id, values):
     * update_aggregate_metadata(self, ctxt, aggregate_id, metadata):
     * delete_aggregate(self, ctxt, aggregate_id):
     * add_host_to_aggregate(self, ctxt, aggregate_id, host_name):
     * remove_host_from_aggregate(self, ctxt, aggregate_id, host_name):
 * TEST nova.cells.manager
     * create_aggregate(self, ctxt, aggregate_name, availability_zone):
     * get_aggregate(self, ctxt, aggregate_id):
     * get_aggregate_list(self, ctxt, cell=None):
     * update_aggregate(self, ctxt, aggregate_id, values):
     * update_aggregate_metadata(self, ctxt, aggregate_id, metadata):
     * delete_aggregate(self, ctxt, aggregate_id):
     * add_host_to_aggregate(self, ctxt, aggregate_id, host_name):
     * remove_host_from_aggregate(self, ctxt, aggregate_id, host_name):
 * nova.cells.messaging - Message Runner
     + create_aggregate(self, ctxt, aggregate_name, availability_zone):
     + get_aggregate(self, ctxt, aggregate_id):
     + get_aggregate_list(self, ctxt, cell=None):
     + update_aggregate(self, ctxt, aggregate_id, values):
     + update_aggregate_metadata(self, ctxt, aggregate_id, metadata):
     + delete_aggregate(self, ctxt, aggregate_id):
     + add_host_to_aggregate(self, ctxt, aggregate_id, host_name):
     + remove_host_from_aggregate(self, ctxt, aggregate_id, host_name):
 * TEST nova.cells.messaging - Message Runner
     * create_aggregate(self, ctxt, aggregate_name, availability_zone):
     * get_aggregate(self, ctxt, aggregate_id):
     * get_aggregate_list(self, ctxt, cell=None):
     * update_aggregate(self, ctxt, aggregate_id, values):
     * update_aggregate_m...

Read more...

Revision history for this message
Matt Riedemann (mriedem) wrote :

The patch was abandoned but it sounds like rax still wants to do this in Juno, so marking as confirmed and removing assignee.

Changed in nova:
status: In Progress → Confirmed
assignee: Matthew Sherborne (msherborne+openstack) → nobody
importance: Undecided → Medium
Revision history for this message
Joe Gordon (jogo) wrote :

now that we are planning on replacing the entire cells design with something else and don't want to spend time fixing something we are about to remove, closing this.

Changed in nova:
status: Confirmed → Won't Fix
Revision history for this message
Matt Riedemann (mriedem) wrote :
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.