)]}'
{"specs/ocata/dynamic-log-levels.rst":[{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"41cfefa4b7dca1e2ba4c47540d1d2656016c67b6","unresolved":false,"context_lines":[{"line_number":44,"context_line":"a side product would be when a system administrator wants to confirm Message"},{"line_number":45,"context_line":"Broker connectivity in a service, as the log level query mechanism can be used"},{"line_number":46,"context_line":"as a ping to the service via the Message Broker."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Proposed change"},{"line_number":49,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal is to introduce 2 new admin only REST APIs, one to modify debug"},{"line_number":52,"context_line":"levels and another to query them."}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a74a57a_89e33b05","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":15},"updated":"2016-11-29 22:56:08.000000000","message":"Just to double check, this is only going to set the debug level for the current running process and not persist the log levels between restarts or anything, right?","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8ac963262b038ee2ab3a3336cdf337d7ee2817e0","unresolved":false,"context_lines":[{"line_number":44,"context_line":"a side product would be when a system administrator wants to confirm Message"},{"line_number":45,"context_line":"Broker connectivity in a service, as the log level query mechanism can be used"},{"line_number":46,"context_line":"as a ping to the service via the Message Broker."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Proposed change"},{"line_number":49,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal is to introduce 2 new admin only REST APIs, one to modify debug"},{"line_number":52,"context_line":"levels and another to query them."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_77baff6f","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":15},"in_reply_to":"5a74a57a_89e33b05","updated":"2016-12-02 15:52:26.000000000","message":"Correct.  I did write it at some point, but I must have accidentally deleted the paragraph.  Will add it again.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"41cfefa4b7dca1e2ba4c47540d1d2656016c67b6","unresolved":false,"context_lines":[{"line_number":52,"context_line":"levels and another to query them."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting the log levels will be possible for all Volume, Scheduler, and Backup"},{"line_number":55,"context_line":"services, but limited in the API service to only the service that receives the"},{"line_number":56,"context_line":"request.  This is a reasonable limitation, since API services can be easily"},{"line_number":57,"context_line":"restarted without impacting the cloud because they are only in the control path"},{"line_number":58,"context_line":"and are usually deployed in an Active/Active configuration (and if they are not"},{"line_number":59,"context_line":"and there\u0027s only 1 service running then there\u0027s no problem since it can be"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a74a57a_69889fc7","line":56,"range":{"start_line":55,"start_character":10,"end_line":56,"end_character":8},"updated":"2016-11-29 22:56:08.000000000","message":"I wonder if maybe we should just restrict specifying the api service. Might almost be easier than trying to explain to deployers why not all of their services switched log levels if we just set expecatations that this is *only* for scheduler, volume, and backup. Other than test/dev setups like devstack from what I\u0027ve seen its pretty rare to have single c-api services in a deployment.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8ac963262b038ee2ab3a3336cdf337d7ee2817e0","unresolved":false,"context_lines":[{"line_number":52,"context_line":"levels and another to query them."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting the log levels will be possible for all Volume, Scheduler, and Backup"},{"line_number":55,"context_line":"services, but limited in the API service to only the service that receives the"},{"line_number":56,"context_line":"request.  This is a reasonable limitation, since API services can be easily"},{"line_number":57,"context_line":"restarted without impacting the cloud because they are only in the control path"},{"line_number":58,"context_line":"and are usually deployed in an Active/Active configuration (and if they are not"},{"line_number":59,"context_line":"and there\u0027s only 1 service running then there\u0027s no problem since it can be"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_b773b7e6","line":56,"range":{"start_line":55,"start_character":10,"end_line":56,"end_character":8},"in_reply_to":"3a71b18c_b0ee7a20","updated":"2016-12-02 15:52:26.000000000","message":"I will add an explanation to the spec why I think this is still useful.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"5f08eefa394026cb10d3cb837448a05cbe943f18","unresolved":false,"context_lines":[{"line_number":52,"context_line":"levels and another to query them."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Setting the log levels will be possible for all Volume, Scheduler, and Backup"},{"line_number":55,"context_line":"services, but limited in the API service to only the service that receives the"},{"line_number":56,"context_line":"request.  This is a reasonable limitation, since API services can be easily"},{"line_number":57,"context_line":"restarted without impacting the cloud because they are only in the control path"},{"line_number":58,"context_line":"and are usually deployed in an Active/Active configuration (and if they are not"},{"line_number":59,"context_line":"and there\u0027s only 1 service running then there\u0027s no problem since it can be"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_b0ee7a20","line":56,"range":{"start_line":55,"start_character":10,"end_line":56,"end_character":8},"in_reply_to":"5a74a57a_69889fc7","updated":"2016-12-01 23:46:17.000000000","message":"Agreed. Single API process is rare, and the fact that setting the log level for the API service behaves differently without feedback is very counter-intuitive. Better to give an error.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"5f08eefa394026cb10d3cb837448a05cbe943f18","unresolved":false,"context_lines":[{"line_number":133,"context_line":"Security impact"},{"line_number":134,"context_line":"---------------"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"None"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"Notifications impact"},{"line_number":139,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_b03cba94","line":136,"range":{"start_line":136,"start_character":0,"end_line":136,"end_character":4},"updated":"2016-12-01 23:46:17.000000000","message":"Access control (policy) to this API should be implemented with a default of admin only, since it can be used to set up log spamming DoS attacks if left public.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8ac963262b038ee2ab3a3336cdf337d7ee2817e0","unresolved":false,"context_lines":[{"line_number":133,"context_line":"Security impact"},{"line_number":134,"context_line":"---------------"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"None"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"Notifications impact"},{"line_number":139,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_77103fa9","line":136,"range":{"start_line":136,"start_character":0,"end_line":136,"end_character":4},"in_reply_to":"3a71b18c_b03cba94","updated":"2016-12-02 15:52:26.000000000","message":"On line 51 I mention it\u0027ll only be available to admins, but after I started writing the code I changed it to use the update service ACL, so I will update the spec.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"5f08eefa394026cb10d3cb837448a05cbe943f18","unresolved":false,"context_lines":[{"line_number":138,"context_line":"Notifications impact"},{"line_number":139,"context_line":"--------------------"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"None"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Other end user impact"},{"line_number":144,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_d041360d","line":141,"range":{"start_line":141,"start_character":0,"end_line":141,"end_character":4},"updated":"2016-12-01 23:46:17.000000000","message":"We should emit a notification for changed to log level via the api, for audit purposes.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8ac963262b038ee2ab3a3336cdf337d7ee2817e0","unresolved":false,"context_lines":[{"line_number":138,"context_line":"Notifications impact"},{"line_number":139,"context_line":"--------------------"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"None"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"Other end user impact"},{"line_number":144,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_f7736f13","line":141,"range":{"start_line":141,"start_character":0,"end_line":141,"end_character":4},"in_reply_to":"3a71b18c_d041360d","updated":"2016-12-02 15:52:26.000000000","message":"I thought about it but didn\u0027t think it would be of much interest, but then again I\u0027m no system admin so I\u0027ll follow your recommendation.  ;-)","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"41cfefa4b7dca1e2ba4c47540d1d2656016c67b6","unresolved":false,"context_lines":[{"line_number":170,"context_line":"Primary assignee:"},{"line_number":171,"context_line":"  Gorka Eguileor (geguileo)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"Work Items"},{"line_number":174,"context_line":"----------"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"- Add the set API endpoint and mechanism on the services"},{"line_number":177,"context_line":"- Add the get API endpoint and mechanism on the services"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a74a57a_498763d7","line":177,"range":{"start_line":173,"start_character":0,"end_line":177,"end_character":56},"updated":"2016-11-29 22:56:08.000000000","message":"Will we add support for this in cinderclient/osc?","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"8ac963262b038ee2ab3a3336cdf337d7ee2817e0","unresolved":false,"context_lines":[{"line_number":170,"context_line":"Primary assignee:"},{"line_number":171,"context_line":"  Gorka Eguileor (geguileo)"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"Work Items"},{"line_number":174,"context_line":"----------"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"- Add the set API endpoint and mechanism on the services"},{"line_number":177,"context_line":"- Add the get API endpoint and mechanism on the services"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a71b18c_d2ba0123","line":177,"range":{"start_line":173,"start_character":0,"end_line":177,"end_character":56},"in_reply_to":"5a74a57a_498763d7","updated":"2016-12-02 15:52:26.000000000","message":"I was planning on adding it to cinderclient, but forgot to put it here, will add.","commit_id":"4c3d4856ced8a475239e2c038342bd20ba3cd6d7"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"ad3aed28164e92c107519ef814507b3f5758bd8d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"This is a reasonable limitation, since API services can be easily restarted"},{"line_number":62,"context_line":"without impacting the cloud because they are only in the control path and are"},{"line_number":63,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":64,"context_line":"Active/Passive configuration then there\u0027s only 1 API service running and we"},{"line_number":65,"context_line":"don\u0027t need to propagate the API log level change."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"While some operators may prefer to restart the API services to change the log"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a71b18c_146a45b3","line":64,"updated":"2016-12-05 21:53:00.000000000","message":"Do you mean \"not in an Active/Active configuration....\" ?","commit_id":"2b0da572af9cd63ae512cbfa080b5cd506d5f10b"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"3f6294619758db171efb0ba333a5889d5dfae94f","unresolved":false,"context_lines":[{"line_number":61,"context_line":"This is a reasonable limitation, since API services can be easily restarted"},{"line_number":62,"context_line":"without impacting the cloud because they are only in the control path and are"},{"line_number":63,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":64,"context_line":"Active/Passive configuration then there\u0027s only 1 API service running and we"},{"line_number":65,"context_line":"don\u0027t need to propagate the API log level change."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"While some operators may prefer to restart the API services to change the log"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a71b18c_dd0a6678","line":64,"in_reply_to":"3a71b18c_146a45b3","updated":"2016-12-07 10:38:22.000000000","message":"Yup, this is wrong.","commit_id":"2b0da572af9cd63ae512cbfa080b5cd506d5f10b"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"124d31ba8cd1c0406117e449d6ffd3ff337ef137","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"To change log levels in a service the service\u0027s configuration needs to be"},{"line_number":19,"context_line":"changed and the service restarted.  The restart can done by restarting the"},{"line_number":20,"context_line":"service itself or by requesting it an internal restart via ``SIGHUP`` signal."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"In some services a restart is not a big deal, API and scheduler, because they"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_f0232256","line":19,"range":{"start_line":19,"start_character":48,"end_line":19,"end_character":51},"updated":"2016-12-15 12:21:05.000000000","message":"can be","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ee44ab167a5e1576a226bfff1c2a2b97267ac9f1","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"To change log levels in a service the service\u0027s configuration needs to be"},{"line_number":19,"context_line":"changed and the service restarted.  The restart can done by restarting the"},{"line_number":20,"context_line":"service itself or by requesting it an internal restart via ``SIGHUP`` signal."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"In some services a restart is not a big deal, API and scheduler, because they"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_d319557b","line":19,"range":{"start_line":19,"start_character":48,"end_line":19,"end_character":51},"in_reply_to":"1a6eadb0_f0232256","updated":"2016-12-19 11:09:20.000000000","message":"Done","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"124d31ba8cd1c0406117e449d6ffd3ff337ef137","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"To change log levels in a service the service\u0027s configuration needs to be"},{"line_number":19,"context_line":"changed and the service restarted.  The restart can done by restarting the"},{"line_number":20,"context_line":"service itself or by requesting it an internal restart via ``SIGHUP`` signal."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"In some services a restart is not a big deal, API and scheduler, because they"},{"line_number":23,"context_line":"only operate in the control plane and they don\u0027t perform long running"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_b0136ab7","line":20,"range":{"start_line":20,"start_character":32,"end_line":20,"end_character":34},"updated":"2016-12-15 12:21:05.000000000","message":"not needed","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ee44ab167a5e1576a226bfff1c2a2b97267ac9f1","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"To change log levels in a service the service\u0027s configuration needs to be"},{"line_number":19,"context_line":"changed and the service restarted.  The restart can done by restarting the"},{"line_number":20,"context_line":"service itself or by requesting it an internal restart via ``SIGHUP`` signal."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"In some services a restart is not a big deal, API and scheduler, because they"},{"line_number":23,"context_line":"only operate in the control plane and they don\u0027t perform long running"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_f32491b3","line":20,"range":{"start_line":20,"start_character":32,"end_line":20,"end_character":34},"in_reply_to":"1a6eadb0_b0136ab7","updated":"2016-12-19 11:09:20.000000000","message":"Done","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"124d31ba8cd1c0406117e449d6ffd3ff337ef137","unresolved":false,"context_lines":[{"line_number":189,"context_line":"----------"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"- Add the set API endpoint and mechanism on the services"},{"line_number":192,"context_line":"- Cinder client support for set action"},{"line_number":193,"context_line":"- Add the get API endpoint and mechanism on the services"},{"line_number":194,"context_line":"- Cinder client support for get action"},{"line_number":195,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_db664750","line":192,"range":{"start_line":192,"start_character":2,"end_line":192,"end_character":38},"updated":"2016-12-15 12:21:05.000000000","message":"I\u0027d be good to have a correspondent feature in OSC. As OSC is getting closer to cinder-client compatibility we should work in that direction too.","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ee44ab167a5e1576a226bfff1c2a2b97267ac9f1","unresolved":false,"context_lines":[{"line_number":189,"context_line":"----------"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"- Add the set API endpoint and mechanism on the services"},{"line_number":192,"context_line":"- Cinder client support for set action"},{"line_number":193,"context_line":"- Add the get API endpoint and mechanism on the services"},{"line_number":194,"context_line":"- Cinder client support for get action"},{"line_number":195,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_f3ce51b6","line":192,"range":{"start_line":192,"start_character":2,"end_line":192,"end_character":38},"in_reply_to":"1a6eadb0_db664750","updated":"2016-12-19 11:09:20.000000000","message":"I really hate work duplication, so until we decide we drop the cinder client I won\u0027t be adding features to both clients as work items in the specs.  Of course anybody is free to work on the other client, but I\u0027ll only be adding feature to one client, be it OSC or cinderclient.","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"92d8249dc767e4545d8a74521dbd93bff7773932","unresolved":false,"context_lines":[{"line_number":189,"context_line":"----------"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"- Add the set API endpoint and mechanism on the services"},{"line_number":192,"context_line":"- Cinder client support for set action"},{"line_number":193,"context_line":"- Add the get API endpoint and mechanism on the services"},{"line_number":194,"context_line":"- Cinder client support for get action"},{"line_number":195,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6eadb0_134e6d96","line":192,"range":{"start_line":192,"start_character":2,"end_line":192,"end_character":38},"in_reply_to":"1a6eadb0_f3ce51b6","updated":"2016-12-19 11:19:43.000000000","message":"Ill try add this to OSC team working progress sheet when we have code for testing.","commit_id":"50377b3ea9c5f3d6adcd89c8a5331717557fc64e"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"07cdb8bf0429f82053b8cc9133935c3f7f748e2d","unresolved":false,"context_lines":[{"line_number":28,"context_line":"We should be able to change service log levels dynamically as needed, even if"},{"line_number":29,"context_line":"they will revert back to the defaults on restart."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"A downside to being able to dynamically changing log levels is that we\u0027ll no"},{"line_number":32,"context_line":"longer be sure of what log level a service is running at a given time, so we\u0027ll"},{"line_number":33,"context_line":"also need a mechanism to query current log levels of a service."},{"line_number":34,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"da4df55a_dbe5d2a8","line":31,"range":{"start_line":31,"start_character":40,"end_line":31,"end_character":48},"updated":"2016-12-22 17:45:33.000000000","message":"changing -\u003e change","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4e353c4a7531f8e7ef935dd6dc3d8dadf9f34933","unresolved":false,"context_lines":[{"line_number":28,"context_line":"We should be able to change service log levels dynamically as needed, even if"},{"line_number":29,"context_line":"they will revert back to the defaults on restart."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"A downside to being able to dynamically changing log levels is that we\u0027ll no"},{"line_number":32,"context_line":"longer be sure of what log level a service is running at a given time, so we\u0027ll"},{"line_number":33,"context_line":"also need a mechanism to query current log levels of a service."},{"line_number":34,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a3c09a3_d94b2858","line":31,"range":{"start_line":31,"start_character":40,"end_line":31,"end_character":48},"in_reply_to":"da4df55a_dbe5d2a8","updated":"2017-01-13 15:37:17.000000000","message":"Done","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"46017de44fa986444d7e9d35fb4742459d573559","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal is to introduce 2 new service REST APIs actions, one to modify"},{"line_number":52,"context_line":"debug levels at runtime and another to query them.  The life of the log level"},{"line_number":53,"context_line":"changes will be the current service run, as they will will revert to those"},{"line_number":54,"context_line":"defined in the configuration file upon restart."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Setting the log levels will be possible for all Volume, Scheduler, and Backup"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa48f94a_9230bc09","line":53,"range":{"start_line":53,"start_character":49,"end_line":53,"end_character":58},"updated":"2016-12-21 17:46:19.000000000","message":"Duplicate \"will\".","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4e353c4a7531f8e7ef935dd6dc3d8dadf9f34933","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal is to introduce 2 new service REST APIs actions, one to modify"},{"line_number":52,"context_line":"debug levels at runtime and another to query them.  The life of the log level"},{"line_number":53,"context_line":"changes will be the current service run, as they will will revert to those"},{"line_number":54,"context_line":"defined in the configuration file upon restart."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"Setting the log levels will be possible for all Volume, Scheduler, and Backup"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a3c09a3_39798c94","line":53,"range":{"start_line":53,"start_character":49,"end_line":53,"end_character":58},"in_reply_to":"fa48f94a_9230bc09","updated":"2017-01-13 15:37:17.000000000","message":"Done","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"46017de44fa986444d7e9d35fb4742459d573559","unresolved":false,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"* Set log level: This will be implemented as a service action like ``enable``"},{"line_number":108,"context_line":"  and ``disable``, but will use the ``set-log`` identifier.  It will take"},{"line_number":109,"context_line":"  following parameters in the body:"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"  * ``binary`` (optional): A string parameter indicating the binary of the"},{"line_number":112,"context_line":"    service to change, it can take following values, ``cinder-volume``,"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa48f94a_f5ce52b3","line":109,"updated":"2016-12-21 17:46:19.000000000","message":"Can you write down the URL please.","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ee53e32e4a9fdd43c9593f032ddc9549ac34dafd","unresolved":false,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"* Set log level: This will be implemented as a service action like ``enable``"},{"line_number":108,"context_line":"  and ``disable``, but will use the ``set-log`` identifier.  It will take"},{"line_number":109,"context_line":"  following parameters in the body:"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"  * ``binary`` (optional): A string parameter indicating the binary of the"},{"line_number":112,"context_line":"    service to change, it can take following values, ``cinder-volume``,"}],"source_content_type":"text/x-rst","patch_set":4,"id":"da4df55a_9c7436d0","line":109,"in_reply_to":"fa48f94a_f5ce52b3","updated":"2016-12-22 10:57:21.000000000","message":"You are right, it will be clearer if we have the URLs","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"07cdb8bf0429f82053b8cc9133935c3f7f748e2d","unresolved":false,"context_lines":[{"line_number":117,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":118,"context_line":"    Can be a host or cluster reference - ``host@backend`` or"},{"line_number":119,"context_line":"    ``cluster@backend`` -, or ``null`` or be missing for all servers matching"},{"line_number":120,"context_line":"    the ``binary``.  If ``binary`` is set to all (``*``, ``null``, or missing),"},{"line_number":121,"context_line":"    this field will be ignored."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * ``prefix`` (optional): A string indicating the prefix for the log path, for"},{"line_number":124,"context_line":"    example ``cinder.`` or ``sqlalchemy.engine``.  When not present all logs"}],"source_content_type":"text/x-rst","patch_set":4,"id":"da4df55a_a1045c15","line":121,"range":{"start_line":120,"start_character":21,"end_line":121,"end_character":31},"updated":"2016-12-22 17:45:33.000000000","message":"Why are we ignoring this if the binary is set to all?  Couldn\u0027t you want to do all binaries on one server?","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"fbfd6357a47a44ba37fd0bf4b6622ca6a0fc368f","unresolved":false,"context_lines":[{"line_number":117,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":118,"context_line":"    Can be a host or cluster reference - ``host@backend`` or"},{"line_number":119,"context_line":"    ``cluster@backend`` -, or ``null`` or be missing for all servers matching"},{"line_number":120,"context_line":"    the ``binary``.  If ``binary`` is set to all (``*``, ``null``, or missing),"},{"line_number":121,"context_line":"    this field will be ignored."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * ``prefix`` (optional): A string indicating the prefix for the log path, for"},{"line_number":124,"context_line":"    example ``cinder.`` or ``sqlalchemy.engine``.  When not present all logs"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a3c09a3_7c7222e7","line":121,"range":{"start_line":120,"start_character":21,"end_line":121,"end_character":31},"in_reply_to":"7a3c09a3_bc656ae2","updated":"2017-01-13 15:49:32.000000000","message":"Hmmm.  I don\u0027t want to make the code more complicated.  If no one else is terrible concerned with this, lets not worry about it.  Always something that can be improved in the future if it becomes a necessary use case.  I think it is most likely that people are going to want to restart one binary on all hosts or one binary on one host.  Not being able to do all binaries on one host is not the end of the world.","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4e353c4a7531f8e7ef935dd6dc3d8dadf9f34933","unresolved":false,"context_lines":[{"line_number":117,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":118,"context_line":"    Can be a host or cluster reference - ``host@backend`` or"},{"line_number":119,"context_line":"    ``cluster@backend`` -, or ``null`` or be missing for all servers matching"},{"line_number":120,"context_line":"    the ``binary``.  If ``binary`` is set to all (``*``, ``null``, or missing),"},{"line_number":121,"context_line":"    this field will be ignored."},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"  * ``prefix`` (optional): A string indicating the prefix for the log path, for"},{"line_number":124,"context_line":"    example ``cinder.`` or ``sqlalchemy.engine``.  When not present all logs"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a3c09a3_bc656ae2","line":121,"range":{"start_line":120,"start_character":21,"end_line":121,"end_character":31},"in_reply_to":"da4df55a_a1045c15","updated":"2017-01-13 15:37:17.000000000","message":"This is really caused by the implementation I had in mind.\n\nI was planing to distribute the request to the servicesww using directly the message queues we already have without checking the services in the DB.  Swo we basically can distribute to all services of a binary or to a specific service, what we cannot do is send to all binaries on the same host without checking the DB.\n\nSo sending to binary\u003d* server\u003dhost is not possible and sending to binary\u003d* server\u003dhost@backend will only be successful for one service.\n\nIf you think this is something really important I can change the approach and support all combinations by getting the available services from the DB table and then build the list of topic queues to send to based on that.  But it will be more code.  ;-)","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"46017de44fa986444d7e9d35fb4742459d573559","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    will be changed."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":128,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Get log level: Service action with ``get-log`` identifier accepting the"},{"line_number":131,"context_line":"  following parameters in the body:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa48f94a_952b4633","line":128,"updated":"2016-12-21 17:46:19.000000000","message":"I thought this API is to set DEBUG on and off only?  Isn\u0027t INFO, WARNING, ERROR always logged?","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ee53e32e4a9fdd43c9593f032ddc9549ac34dafd","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    will be changed."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":128,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Get log level: Service action with ``get-log`` identifier accepting the"},{"line_number":131,"context_line":"  following parameters in the body:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"da4df55a_fcd2a233","line":128,"in_reply_to":"fa48f94a_18db0def","updated":"2016-12-22 10:57:21.000000000","message":"From the start the idea was to allow changing it to any valid log level, because it will allow not only setting cinder levels, but also levels for other libraries (ie oslo.messaging, sqlalchemy) and we may want to hide info logs from a library if they are cluttering our logs during a debug session.\n\nTo me it makes more sense to make this feature flexible enough to handle most use cases instead of restricting it now just to add the feature later.  And it\u0027s not like it\u0027s going to add much code (one check for the parameter and one more parameter to pass along).","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"6894b6134186ab3e754075a9e42efb38edbde14d","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    will be changed."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":128,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"* Get log level: Service action with ``get-log`` identifier accepting the"},{"line_number":131,"context_line":"  following parameters in the body:"}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa48f94a_18db0def","line":128,"in_reply_to":"fa48f94a_952b4633","updated":"2016-12-21 18:17:37.000000000","message":"I tend to agree with this.  I\u0027m not sure it makes a lot of sense to allow specific levels to be set?   The cinder.conf could be set to INFO, and this API would allow someone to se it to ERROR, therefore greatly decreasing the logging output.   Not sure why an admin would want to do this.","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"07cdb8bf0429f82053b8cc9133935c3f7f748e2d","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":135,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":138,"context_line":"    Can be a host or a cluster reference - ``host@backend`` or"},{"line_number":139,"context_line":"    ``cluster@backend`` - this parameter is only optional if the ``binary`` is"},{"line_number":140,"context_line":"    ``cinder-api``, otherwise it is required."}],"source_content_type":"text/x-rst","patch_set":4,"id":"da4df55a_013b68d2","line":137,"range":{"start_line":137,"start_character":71,"end_line":137,"end_character":78},"updated":"2016-12-22 17:45:33.000000000","message":"change -\u003e query","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"4e353c4a7531f8e7ef935dd6dc3d8dadf9f34933","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":135,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":138,"context_line":"    Can be a host or a cluster reference - ``host@backend`` or"},{"line_number":139,"context_line":"    ``cluster@backend`` - this parameter is only optional if the ``binary`` is"},{"line_number":140,"context_line":"    ``cinder-api``, otherwise it is required."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a3c09a3_3ca09af4","line":137,"range":{"start_line":137,"start_character":71,"end_line":137,"end_character":78},"in_reply_to":"da4df55a_013b68d2","updated":"2017-01-13 15:37:17.000000000","message":"Done","commit_id":"d86382c7e5c01bd19f50007ed6981eaa51b91b97"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"f6b60bc5a42af600de711881b2a6cbd5b1bd8346","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    will be changed."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":129,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"* Get log level: Service action with ``get-log`` identifier.  Effective URL"},{"line_number":132,"context_line":"  ``/v3/{tenant_id}/os-services/get-log`` will accept the following parameters"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba5201f7_62c5028d","line":129,"updated":"2017-01-03 13:55:46.000000000","message":"I agree that we should be able to set any log level we want. We\u0027ve had issues where libraries (privsep, brick) have spammed the logs.","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"370b4e62a4b67927668833c35e03e0104c0b654b","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    will be changed."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":129,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"* Get log level: Service action with ``get-log`` identifier.  Effective URL"},{"line_number":132,"context_line":"  ``/v3/{tenant_id}/os-services/get-log`` will accept the following parameters"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a3c09a3_392b4c84","line":129,"in_reply_to":"ba5201f7_192e64fa","updated":"2017-01-13 15:37:04.000000000","message":"That\u0027s the use of the prefix field, to be able to set different levels for cinder stuff and for other libraries.","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"60311b8e003f3b320ab3b664fa98cc2d20935f7d","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    will be changed."},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"  * ``level`` (required): A string with the log level to set, case insensitive,"},{"line_number":129,"context_line":"    accepted values are ``INFO``, ``WARNING``, ``ERROR``, ``DEBUG``."},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"* Get log level: Service action with ``get-log`` identifier.  Effective URL"},{"line_number":132,"context_line":"  ``/v3/{tenant_id}/os-services/get-log`` will accept the following parameters"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba5201f7_192e64fa","line":129,"in_reply_to":"ba5201f7_62c5028d","updated":"2017-01-10 16:53:55.000000000","message":"For this reason it would be nice to have log levels for Cinder\u0027s code vs. log levels for libs that cinder uses.","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"d290cb2947ba0b0dd2b6f962eec2d30139c7eb91","unresolved":false,"context_lines":[{"line_number":133,"context_line":"  in the body:"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"  * ``binary`` (required): A string parameter indicating the binary of the"},{"line_number":136,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":137,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba5201f7_14b9c9cb","line":136,"range":{"start_line":136,"start_character":15,"end_line":136,"end_character":21},"updated":"2017-01-10 17:12:50.000000000","message":"query","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"370b4e62a4b67927668833c35e03e0104c0b654b","unresolved":false,"context_lines":[{"line_number":133,"context_line":"  in the body:"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"  * ``binary`` (required): A string parameter indicating the binary of the"},{"line_number":136,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":137,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a3c09a3_d93408e0","line":136,"range":{"start_line":136,"start_character":15,"end_line":136,"end_character":21},"in_reply_to":"ba5201f7_14b9c9cb","updated":"2017-01-13 15:37:04.000000000","message":"Done","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"d290cb2947ba0b0dd2b6f962eec2d30139c7eb91","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":137,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":140,"context_line":"    Can be a host or a cluster reference - ``host@backend`` or"},{"line_number":141,"context_line":"    ``cluster@backend`` - this parameter is only optional if the ``binary`` is"},{"line_number":142,"context_line":"    ``cinder-api``, otherwise it is required."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ba5201f7_d4d661f0","line":139,"range":{"start_line":139,"start_character":71,"end_line":139,"end_character":77},"updated":"2017-01-10 17:12:50.000000000","message":"query","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"370b4e62a4b67927668833c35e03e0104c0b654b","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    service to change, it can take following values, ``cinder-volume``,"},{"line_number":137,"context_line":"    ``cinder-scheduler``, ``cinder-backup``, and ``cinder-api``."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"  * ``server`` (optional): A string parameter indicating the server to change,"},{"line_number":140,"context_line":"    Can be a host or a cluster reference - ``host@backend`` or"},{"line_number":141,"context_line":"    ``cluster@backend`` - this parameter is only optional if the ``binary`` is"},{"line_number":142,"context_line":"    ``cinder-api``, otherwise it is required."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7a3c09a3_59573802","line":139,"range":{"start_line":139,"start_character":71,"end_line":139,"end_character":77},"in_reply_to":"ba5201f7_d4d661f0","updated":"2017-01-13 15:37:04.000000000","message":"Done","commit_id":"1d926085aa3ea2c29365b7bf1168790fec466df3"}],"specs/pike/dynamic-log-levels.rst":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"53b25e7884e57e7ba7cfdcf9c29fa2db6d605458","unresolved":false,"context_lines":[{"line_number":63,"context_line":"without impacting the cloud because they are only in the control plane and are"},{"line_number":64,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":65,"context_line":"Active/Active configuration then there\u0027s only 1 API service running and not"},{"line_number":66,"context_line":"being able to propaget the API log level change isn\u0027t such a big deal."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"While some operators may prefer to restart the API services to change the log"},{"line_number":69,"context_line":"levels, there may be others that prefer to directly make the dynamic log level"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_d9927857","line":66,"range":{"start_line":66,"start_character":13,"end_line":66,"end_character":23},"updated":"2017-04-20 14:59:08.000000000","message":"I am going to push up a patch to fix this and then merge it.","commit_id":"e0833a4f93bf5735d05a9cd6a0ac5cb007b201cd"},{"author":{"_account_id":15100,"name":"zhongjun","email":"jun.zhongjun2@gmail.com","username":"zhongjun2"},"change_message_id":"246e3c433464c4b0db5b72f5ec741367790dcd3b","unresolved":false,"context_lines":[{"line_number":63,"context_line":"without impacting the cloud because they are only in the control plane and are"},{"line_number":64,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":65,"context_line":"Active/Active configuration then there\u0027s only 1 API service running and not"},{"line_number":66,"context_line":"being able to propaget the API log level change isn\u0027t such a big deal."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"While some operators may prefer to restart the API services to change the log"},{"line_number":69,"context_line":"levels, there may be others that prefer to directly make the dynamic log level"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bff0334d_908a99bc","line":66,"range":{"start_line":66,"start_character":13,"end_line":66,"end_character":23},"updated":"2017-04-11 09:41:27.000000000","message":"s/propaget/propagete","commit_id":"e0833a4f93bf5735d05a9cd6a0ac5cb007b201cd"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"7c1ee592fcb283a007427f98dae02bee55e9b697","unresolved":false,"context_lines":[{"line_number":63,"context_line":"without impacting the cloud because they are only in the control plane and are"},{"line_number":64,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":65,"context_line":"Active/Active configuration then there\u0027s only 1 API service running and not"},{"line_number":66,"context_line":"being able to propaget the API log level change isn\u0027t such a big deal."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"While some operators may prefer to restart the API services to change the log"},{"line_number":69,"context_line":"levels, there may be others that prefer to directly make the dynamic log level"}],"source_content_type":"text/x-rst","patch_set":6,"id":"bff0334d_a4ce3560","line":66,"range":{"start_line":66,"start_character":13,"end_line":66,"end_character":23},"in_reply_to":"bff0334d_908a99bc","updated":"2017-04-12 04:21:37.000000000","message":"propagate","commit_id":"e0833a4f93bf5735d05a9cd6a0ac5cb007b201cd"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"242d1effa90846711ac30f31a7eeed5e6f7d1e41","unresolved":false,"context_lines":[{"line_number":63,"context_line":"without impacting the cloud because they are only in the control plane and are"},{"line_number":64,"context_line":"usually deployed in an Active/Active configuration.  And if they are not in an"},{"line_number":65,"context_line":"Active/Active configuration then there\u0027s only 1 API service running and not"},{"line_number":66,"context_line":"being able to propaget the API log level change isn\u0027t such a big deal."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"While some operators may prefer to restart the API services to change the log"},{"line_number":69,"context_line":"levels, there may be others that prefer to directly make the dynamic log level"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7ffa3b31_ebbca46a","line":66,"range":{"start_line":66,"start_character":13,"end_line":66,"end_character":23},"in_reply_to":"bff0334d_a4ce3560","updated":"2017-04-17 17:45:26.000000000","message":"+1","commit_id":"e0833a4f93bf5735d05a9cd6a0ac5cb007b201cd"}]}