scheduler: Change servicegroup API is_up() command to accept host and topic (partial)
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-
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:
- 2015.1.0
- Started by
- John Garbutt
- Completed by
- John Garbutt
Related branches
Related bugs
Sprints
Whiteboard
Gerrit topic: https:/
Addressed by: https:/
Break base service group driver class out from API
Addressed by: https:/
Pull singleton config check cruft out of SG API
Addressed by: https:/
Remove useless get_one() method in SG API
Addressed by: https:/
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:/
All changes now approved, and now merged. --johnthetubaguy 19th March 2015