scheduler: Change servicegroup API is_up() command to accept host and topic (partial)

Registered by Jay Pipes

Currently, the servicegroup API's is_up() API call, which is used to determine if a service is up or down, accepts a dictionary of service information. With the detach-service-from-compute-node blueprint, it became clear that the is_up() API call should really just accept a host and a topic parameter, not a dictionary of information.

The reason for this is because the servicegroup API's current is_up() API call was built with various assumptions that the driver for the API was a database. For ZooKeeper and Memcache servicegroup drivers, there is no need to pass a dictionary of service information. We should only supply a host and a topic key for the service and the drivers themselves should be free to implement whatever underlying data model makes sense.

Blueprint information

Status:
Complete
Approver:
John Garbutt
Priority:
Low
Drafter:
Jay Pipes
Direction:
Approved
Assignee:
Jay Pipes
Definition:
Approved
Series goal:
Accepted for kilo
Implementation:
Implemented
Milestone target:
milestone icon 2015.1.0
Started by
John Garbutt
Completed by
John Garbutt

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bug/1414536,n,z

Addressed by: https://review.openstack.org/149921
    Break base service group driver class out from API

Addressed by: https://review.openstack.org/149922
    Pull singleton config check cruft out of SG API

Addressed by: https://review.openstack.org/149924
    Remove useless get_one() method in SG API

Addressed by: https://review.openstack.org/149925
    Make the servicegroup get_all() method consistent

Loosely, this is helping with the scheduler priority efforts, so approving for kilo. --johnthetubaguy 13th Feb 2015

Since we have hit feature proposal freeze, marking this as partial and ready to review. Please no more "new code" should be uploaded as we try to get the current items merged before feature freeze. --johnthetubaguy 10th March 2015

Gerrit topic: https://review.openstack.org/#q,topic:bug/1414513,n,z

All changes now approved, and now merged. --johnthetubaguy 19th March 2015

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.