)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"e721989234d9fae5a599fa3e4820fcffe53d867e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Chance and Simple scheduler functionality can be archived using"},{"line_number":10,"context_line":"FilterScheduler. The implementation has been deprecated in Juno,"},{"line_number":11,"context_line":"so we can remove it safely now."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I92b302ea0404a35328bd7a36739c3e7846898392"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a41bdd9_9e515511","line":12,"updated":"2015-07-17 03:38:30.000000000","message":"Can you verify if the docstring in simple.py is documented?  Then we can remove it as long as user has something to refer to (how to simulate Simple/Chance scheduler).","commit_id":"f235fd1e0d63ff430de8f73fb36c4e005c3acb14"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"f64b9e449e20b44b2959d1783d29f1b1e39f30d6","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Chance and Simple scheduler functionality can be archived using"},{"line_number":10,"context_line":"FilterScheduler. The implementation has been deprecated in Juno,"},{"line_number":11,"context_line":"so we can remove it safely now."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I92b302ea0404a35328bd7a36739c3e7846898392"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_e031a80c","line":12,"in_reply_to":"3a50d1a3_30aad01a","updated":"2015-07-21 09:29:07.000000000","message":"So Simple and Chance scheduler were actually removed in commit 289f63934459baccae1e7c547890cc5f4e9f26b3, although simple.py remains in cinder/scheduler directory.  The reason I asked about documentation was because the docstring in simple.py can be considered as documentation, removing it is OK but I hope there is a place where people can find something about Simple/Chance scheduler.\n\nAt least we can add a DocImpact flag here?","commit_id":"f235fd1e0d63ff430de8f73fb36c4e005c3acb14"},{"author":{"_account_id":1313,"name":"yaguang","display_name":"Yaguang Tang","email":"heut2008@gmail.com","username":"heut2008"},"change_message_id":"a849f975d7b0a0b52c3eb1aaf6fe07ce08663c9a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Chance and Simple scheduler functionality can be archived using"},{"line_number":10,"context_line":"FilterScheduler. The implementation has been deprecated in Juno,"},{"line_number":11,"context_line":"so we can remove it safely now."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I92b302ea0404a35328bd7a36739c3e7846898392"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_72f55c81","line":12,"in_reply_to":"3a50d1a3_e031a80c","updated":"2015-07-21 13:39:22.000000000","message":"adding Chance scheduler config description url  in commit message.","commit_id":"f235fd1e0d63ff430de8f73fb36c4e005c3acb14"},{"author":{"_account_id":1313,"name":"yaguang","display_name":"Yaguang Tang","email":"heut2008@gmail.com","username":"heut2008"},"change_message_id":"c47dc1c95ca0ba551ae995afca430eaa3f5ed2a9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Chance and Simple scheduler functionality can be archived using"},{"line_number":10,"context_line":"FilterScheduler. The implementation has been deprecated in Juno,"},{"line_number":11,"context_line":"so we can remove it safely now."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I92b302ea0404a35328bd7a36739c3e7846898392"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_30aad01a","line":12,"in_reply_to":"9a41bdd9_9e515511","updated":"2015-07-19 04:53:38.000000000","message":"Sadly, I don\u0027t find any official document describes how each filters and weighers can be used to archive specific scheduler strategy for cloud admin,  but that fact is that the FilterScheduler has been default scheduler for a long time, according to git log,from Grizzly. AFAIK, the Simple/Chance scheduler exists when cinder project is created , code forked from Nova. and it\u0027s been default Scheduler only for Folsom release. So I believe this code is mostly a legacy issue in our develop cycle. almost no user impact.","commit_id":"f235fd1e0d63ff430de8f73fb36c4e005c3acb14"}],"cinder/tests/unit/scheduler/test_scheduler.py":[{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"c9fc97cae28e7c6b9bd2260dbc2d9b780b65ae37","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        # scheduler_default_filters and scheduler_default_weighers when"},{"line_number":199,"context_line":"        # ChanceScheduler or SimpleScheduler is configured as scheduler_driver."},{"line_number":200,"context_line":"        chance \u003d \u0027cinder.scheduler.chance.ChanceScheduler\u0027"},{"line_number":201,"context_line":"        simple \u003d \u0027cinder.scheduler.simple.SimpleScheduler\u0027"},{"line_number":202,"context_line":"        default_filters \u003d [\u0027AvailabilityZoneFilter\u0027,"},{"line_number":203,"context_line":"                           \u0027CapacityFilter\u0027,"},{"line_number":204,"context_line":"                           \u0027CapabilitiesFilter\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_95c38138","side":"PARENT","line":201,"updated":"2015-07-24 08:40:12.000000000","message":"Anton, Chance scheduler was also removed and should be gone too.","commit_id":"c84976fd3d0dd453878faf2d8842ef187bb037f9"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"708d40d0a8a7e412dde283d561b1ef6d57549cf1","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        # scheduler_default_filters and scheduler_default_weighers when"},{"line_number":199,"context_line":"        # ChanceScheduler or SimpleScheduler is configured as scheduler_driver."},{"line_number":200,"context_line":"        chance \u003d \u0027cinder.scheduler.chance.ChanceScheduler\u0027"},{"line_number":201,"context_line":"        simple \u003d \u0027cinder.scheduler.simple.SimpleScheduler\u0027"},{"line_number":202,"context_line":"        default_filters \u003d [\u0027AvailabilityZoneFilter\u0027,"},{"line_number":203,"context_line":"                           \u0027CapacityFilter\u0027,"},{"line_number":204,"context_line":"                           \u0027CapabilitiesFilter\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_6892a1f8","side":"PARENT","line":201,"updated":"2015-07-24 08:00:34.000000000","message":"imo we should delete only simple scheduler from this test case","commit_id":"c84976fd3d0dd453878faf2d8842ef187bb037f9"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"50ae228469fb12293e546ee321329e8d299dd827","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        # scheduler_default_filters and scheduler_default_weighers when"},{"line_number":199,"context_line":"        # ChanceScheduler or SimpleScheduler is configured as scheduler_driver."},{"line_number":200,"context_line":"        chance \u003d \u0027cinder.scheduler.chance.ChanceScheduler\u0027"},{"line_number":201,"context_line":"        simple \u003d \u0027cinder.scheduler.simple.SimpleScheduler\u0027"},{"line_number":202,"context_line":"        default_filters \u003d [\u0027AvailabilityZoneFilter\u0027,"},{"line_number":203,"context_line":"                           \u0027CapacityFilter\u0027,"},{"line_number":204,"context_line":"                           \u0027CapabilitiesFilter\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a50d1a3_d56ca938","side":"PARENT","line":201,"in_reply_to":"3a50d1a3_95c38138","updated":"2015-07-24 08:43:41.000000000","message":"Yep, missed it, thanks","commit_id":"c84976fd3d0dd453878faf2d8842ef187bb037f9"}]}