)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12525,"name":"vishwanath jayaraman","email":"vishwanathj@hotmail.com","username":"vish"},"change_message_id":"de89bd5b566ddc995c7f38b39fd649790d0b79d4","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":10,"context_line":"resource before any customer resource on the node is deployed. Nova and Cinder"},{"line_number":11,"context_line":"has the setting of “enable_new_services” in each conf to disable the initial"},{"line_number":12,"context_line":"service status to archive this. This proposal adds enable_new_agents config to"},{"line_number":13,"context_line":"agents in network node."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"7a016987_fca02aa1","line":12,"updated":"2015-05-18 02:16:08.000000000","message":"clarification: did you mean \u0027achieve\u0027 instead of \u0027archive\u0027","commit_id":"133023432382d1c293bada0f4d6b1077541c7c98"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"e73f3eb75834b6cdf2d41ac27900166f0e616817","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":10,"context_line":"resource before any customer resource on the node is deployed. Nova and Cinder"},{"line_number":11,"context_line":"has the setting of “enable_new_services” in each conf to disable the initial"},{"line_number":12,"context_line":"service status to archive this. This proposal adds enable_new_agents config to"},{"line_number":13,"context_line":"agents in network node."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"DocImpact"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"7a016987_91da8d16","line":12,"in_reply_to":"7a016987_fca02aa1","updated":"2015-05-26 13:18:01.000000000","message":"Good catch! Done.","commit_id":"133023432382d1c293bada0f4d6b1077541c7c98"}],"specs/liberty/enable-new-agents.rst":[{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"7f32d64f3555b1675f500bec7c39ce03e4e08372","unresolved":false,"context_lines":[{"line_number":30,"context_line":"all users to create the resources on the node by agent\u0027s admin_state_up\u003dFalse,"},{"line_number":31,"context_line":"but Neutron cannot allow admin user only to create the resources. Admin user"},{"line_number":32,"context_line":"generally has privilege of doing everything. It\u0027s strange that admin user"},{"line_number":33,"context_line":"cannot create resources on node set admin_state_up\u003dFalse."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed Change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa53dc9_703707b1","line":33,"updated":"2015-04-07 01:24:05.000000000","message":"admin can create resources on agents with admin_state_up\u003dFalse if enable_services_on_agents_with_admin_state_down configuration parameter is True.","commit_id":"37bf3b1014e3fc0b11ee2479dc1dce7aac492f16"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"107fa76b85a3fad1df90424732810d9a5b1625bb","unresolved":false,"context_lines":[{"line_number":30,"context_line":"all users to create the resources on the node by agent\u0027s admin_state_up\u003dFalse,"},{"line_number":31,"context_line":"but Neutron cannot allow admin user only to create the resources. Admin user"},{"line_number":32,"context_line":"generally has privilege of doing everything. It\u0027s strange that admin user"},{"line_number":33,"context_line":"cannot create resources on node set admin_state_up\u003dFalse."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed Change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa53dc9_3d1ebd63","line":33,"in_reply_to":"9aa53dc9_703707b1","updated":"2015-04-08 05:24:40.000000000","message":"Done","commit_id":"37bf3b1014e3fc0b11ee2479dc1dce7aac492f16"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"aa62509aad83462a78daa50311b35f8a7b2504e8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"all users to create the resources on the node by agent\u0027s admin_state_up\u003dFalse,"},{"line_number":31,"context_line":"but Neutron cannot allow admin user only to create the resources. Admin user"},{"line_number":32,"context_line":"generally has privilege of doing everything. It\u0027s strange that admin user"},{"line_number":33,"context_line":"cannot create resources on node set admin_state_up\u003dFalse."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Proposed Change"},{"line_number":36,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa53dc9_69184285","line":33,"in_reply_to":"9aa53dc9_703707b1","updated":"2015-04-07 04:42:39.000000000","message":"I missed the option but I also want to change option to default behavior.","commit_id":"37bf3b1014e3fc0b11ee2479dc1dce7aac492f16"},{"author":{"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},"change_message_id":"0b2948306bd8fdb0955542accf7972ec6b0baab9","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"This proposal adds enable_new_agents config for operator maintenance"},{"line_number":40,"context_line":"to l3-agent and dhcp-agent in network nodes. Neutron agent\u0027s"},{"line_number":41,"context_line":"admin_state_up is controlled by this config while starting."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aaa499b_b9afc8e2","line":40,"updated":"2015-04-11 00:30:22.000000000","message":"Why only l3-agent, dhcp-agent?\nThere are many other agents which the scenario applies to.\nThe option can be common for any agents.","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b08322cf21be7cc542012f53fcdd14623047cc8","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"This proposal adds enable_new_agents config for operator maintenance"},{"line_number":40,"context_line":"to l3-agent and dhcp-agent in network nodes. Neutron agent\u0027s"},{"line_number":41,"context_line":"admin_state_up is controlled by this config while starting."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aaa499b_383160a3","line":40,"in_reply_to":"7aaa499b_2cb6c68a","updated":"2015-04-13 07:20:18.000000000","message":"I see. Done.","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"2c18663863e3663557c1826fa2901af4d7bb1a65","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"This proposal adds enable_new_agents config for operator maintenance"},{"line_number":40,"context_line":"to l3-agent and dhcp-agent in network nodes. Neutron agent\u0027s"},{"line_number":41,"context_line":"admin_state_up is controlled by this config while starting."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aaa499b_2cb6c68a","line":40,"in_reply_to":"7aaa499b_4cbfd2d0","updated":"2015-04-13 04:08:15.000000000","message":"neutron-XXXaas stuffs are covered by neutron-specs.","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"8ed158cefa333795f1010483f3e7be5c1752be1d","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"This proposal adds enable_new_agents config for operator maintenance"},{"line_number":40,"context_line":"to l3-agent and dhcp-agent in network nodes. Neutron agent\u0027s"},{"line_number":41,"context_line":"admin_state_up is controlled by this config while starting."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aaa499b_4cbfd2d0","line":40,"in_reply_to":"7aaa499b_b9afc8e2","updated":"2015-04-13 03:28:32.000000000","message":"I know that other agents exist. For example, They are mainly l2-agent(ovs-agent, linuxbridge-agent), metadata-agent and lbaas-agent.\n\nThe scope of proposal prevents user\u0027s resources being scheduled on maintenance agents with admin_state_up False. AFAIK, l2-agent and metadata-agent doesn\u0027t have scheduler logic. Their resources also are used with the other resources(VM, Router, Dnsmasq) which have schedule method. So user\u0027s resources aren\u0027t created on l2-agent and metadata-agent in maintenance node. I suppose their agents don\u0027t need the proposal.\n\nI also think that service-agents including LBaaS should be fixed outside Neutron main-tree. \n\nCan you point out if my thought is wrong?","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"7c22246dc96d12b0706cb2987f92d1dc96ec18d6","unresolved":false,"context_lines":[{"line_number":167,"context_line":"User Documentation"},{"line_number":168,"context_line":"------------------"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"The new config options will be documented."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"Developer Documentation"},{"line_number":173,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa53dc9_80718646","line":170,"updated":"2015-04-10 04:29:36.000000000","message":"\"maintenance scenario\" in Proposed Change section seems a good addition to the documentation.","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"8ed158cefa333795f1010483f3e7be5c1752be1d","unresolved":false,"context_lines":[{"line_number":167,"context_line":"User Documentation"},{"line_number":168,"context_line":"------------------"},{"line_number":169,"context_line":""},{"line_number":170,"context_line":"The new config options will be documented."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"Developer Documentation"},{"line_number":173,"context_line":"-----------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7aaa499b_0c88aae6","line":170,"in_reply_to":"9aa53dc9_80718646","updated":"2015-04-13 03:28:32.000000000","message":"Done","commit_id":"b747feee27b3b7f1e8e28885e29d5b8e166239f9"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"6e996c628538e3d76bc065cb4f07ef92dab6b45a","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"The new config options will be documented."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"The following is maintenance scenario."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"Precondition: Neutron servers run with"},{"line_number":159,"context_line":"enable_services_on_agents_with_admin_state_down\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7aaa499b_871a536d","line":156,"updated":"2015-04-13 04:06:52.000000000","message":"\"The maintenance scenario will be documented\" ?","commit_id":"145a9a92869cccb2a25f5f0c6aa13c859cec7786"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"8272199223eaef0ee587b4b3c4c0d89d87afe7e4","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"The new config options will be documented."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"The following is maintenance scenario."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"Precondition: Neutron servers run with"},{"line_number":159,"context_line":"enable_services_on_agents_with_admin_state_down\u003dTrue."}],"source_content_type":"text/x-rst","patch_set":4,"id":"7aaa499b_180fc4e2","line":156,"in_reply_to":"7aaa499b_871a536d","updated":"2015-04-13 07:20:40.000000000","message":"Done","commit_id":"145a9a92869cccb2a25f5f0c6aa13c859cec7786"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"5dc8ec8264c069c3515628c821bfed160a506684","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"None"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"Functional Tests"},{"line_number":141,"context_line":"----------------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7aaa499b_63dff469","line":140,"updated":"2015-04-14 17:27:37.000000000","message":"Surely we\u0027d want to add a functional test for this new config option and ensure the agents function as you expect.","commit_id":"3e8f6aae6df759e7b60a916ca7181a44cf318195"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"04c838c33501caa08cb52d87cb286eff5167ec32","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"None"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"Functional Tests"},{"line_number":141,"context_line":"----------------"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7aaa499b_6f3b449d","line":140,"in_reply_to":"7aaa499b_63dff469","updated":"2015-04-15 08:00:24.000000000","message":"Done","commit_id":"3e8f6aae6df759e7b60a916ca7181a44cf318195"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"ead097cfd8e751044a2426940bd6524ab741fecf","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":30,"context_line":"prevent all users to create the resources on the agent with"},{"line_number":31,"context_line":"admin_state_up\u003dFalse, but Neutron usualy cannot allow admin user only"},{"line_number":32,"context_line":"to create the resources. Currently, if"},{"line_number":33,"context_line":"enable_services_on_agents_with_admin_state_down configuration"},{"line_number":34,"context_line":"parameter is True, admin can create the resources on the agent."}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_334be8b2","line":31,"updated":"2015-04-19 12:41:53.000000000","message":"s/usualy/usually/","commit_id":"f2fa53cc359b71246602262c4605ceca897c1059"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"22a2293591ab2d25fa9191f4353417f520b9aae9","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":30,"context_line":"prevent all users to create the resources on the agent with"},{"line_number":31,"context_line":"admin_state_up\u003dFalse, but Neutron usualy cannot allow admin user only"},{"line_number":32,"context_line":"to create the resources. Currently, if"},{"line_number":33,"context_line":"enable_services_on_agents_with_admin_state_down configuration"},{"line_number":34,"context_line":"parameter is True, admin can create the resources on the agent."}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_71f2739c","line":31,"in_reply_to":"dac4157f_334be8b2","updated":"2015-04-20 05:30:23.000000000","message":"Done","commit_id":"f2fa53cc359b71246602262c4605ceca897c1059"},{"author":{"_account_id":13997,"name":"SHIGEMATSU Mitsuhiro","email":"shigematsu.mitsuhiro@lab.ntt.co.jp","username":"pshige"},"change_message_id":"ead097cfd8e751044a2426940bd6524ab741fecf","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"This proposal adds enable_new_agents to neutron agent config(dhcp_agent.ini,"},{"line_number":94,"context_line":"l3_agent.ini, etc). This config is set True as default value. It doesn\u0027t change"},{"line_number":95,"context_line":"a traditional befaviour of the agents. We also prefer xxx_agent.ini and"},{"line_number":96,"context_line":"neutron_xxxaas.conf rather than neutron.conf for enable_new_agents because some"},{"line_number":97,"context_line":"agents don\u0027t have scheduler logic."},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_f3f2106f","line":95,"updated":"2015-04-19 12:41:53.000000000","message":"s/befaviour/behaviour/","commit_id":"f2fa53cc359b71246602262c4605ceca897c1059"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"22a2293591ab2d25fa9191f4353417f520b9aae9","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"This proposal adds enable_new_agents to neutron agent config(dhcp_agent.ini,"},{"line_number":94,"context_line":"l3_agent.ini, etc). This config is set True as default value. It doesn\u0027t change"},{"line_number":95,"context_line":"a traditional befaviour of the agents. We also prefer xxx_agent.ini and"},{"line_number":96,"context_line":"neutron_xxxaas.conf rather than neutron.conf for enable_new_agents because some"},{"line_number":97,"context_line":"agents don\u0027t have scheduler logic."},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"dac4157f_51ef6fb0","line":95,"in_reply_to":"dac4157f_f3f2106f","updated":"2015-04-20 05:30:23.000000000","message":"Done","commit_id":"f2fa53cc359b71246602262c4605ceca897c1059"},{"author":{"_account_id":12525,"name":"vishwanath jayaraman","email":"vishwanathj@hotmail.com","username":"vish"},"change_message_id":"de89bd5b566ddc995c7f38b39fd649790d0b79d4","unresolved":false,"context_lines":[{"line_number":24,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":25,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":26,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":27,"context_line":"in each conf to disable the initial service status to archive this."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":30,"context_line":"prevent all users to create the resources on the agent with"}],"source_content_type":"text/x-rst","patch_set":8,"id":"7a016987_7c9afa64","line":27,"updated":"2015-05-18 02:16:08.000000000","message":"clarification: did you intend \u0027achieve\u0027 instead of \u0027archive\u0027?","commit_id":"133023432382d1c293bada0f4d6b1077541c7c98"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"e73f3eb75834b6cdf2d41ac27900166f0e616817","unresolved":false,"context_lines":[{"line_number":24,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":25,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":26,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":27,"context_line":"in each conf to disable the initial service status to archive this."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":30,"context_line":"prevent all users to create the resources on the agent with"}],"source_content_type":"text/x-rst","patch_set":8,"id":"7a016987_5103f576","line":27,"in_reply_to":"7a016987_7c9afa64","updated":"2015-05-26 13:18:01.000000000","message":"Good catch! Done.","commit_id":"133023432382d1c293bada0f4d6b1077541c7c98"},{"author":{"_account_id":6788,"name":"Rossella Sblendido","email":"rsblendido@suse.com","username":"rossella-o"},"change_message_id":"57b1134dbc5687bc5fd394b11e6d3e726ef5f5c9","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":20,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":21,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":22,"context_line":"resources. For example, they may try to create test resources on a new node"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_f272dfac","line":19,"updated":"2015-06-12 13:46:33.000000000","message":"So only test resources are scheduled on the test node, how do you achieve that? You need to manually assign the test resource to the agent running on the test node, right?","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":4149,"name":"Itsuro Oda","email":"oda@valinux.co.jp","username":"oda-g"},"change_message_id":"37566b5cc8615fe9d3f767acc4e9a57f63e10675","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":20,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":21,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":22,"context_line":"resources. For example, they may try to create test resources on a new node"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa32b979_4da1eca8","line":19,"in_reply_to":"1af86dd1_f272dfac","updated":"2015-06-14 22:36:29.000000000","message":"@Rossella, Right. This spec proposes a way that an agent can start without selectable for auto-scheduling but manual-scheduling available (so that a user can test an agent manually). It is not possible now. An agent starts with selectable for auto-scheduling always now.","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"a0f5d0a79f411f6e9fd5e4cc99eb2df8a1ee612c","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":20,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":21,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":22,"context_line":"resources. For example, they may try to create test resources on a new node"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa32b979_b8b84312","line":19,"in_reply_to":"fa32b979_4da1eca8","updated":"2015-06-18 10:30:24.000000000","message":"I agree with Oda.","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"e6bc1828bba364d5a8b9c373dc42eed997ac3acb","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Problem Description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":20,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":21,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":22,"context_line":"resources. For example, they may try to create test resources on a new node"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa32b979_ca841e5f","line":19,"in_reply_to":"fa32b979_b8b84312","updated":"2015-06-19 06:03:08.000000000","message":"I updated the spec using Oda comment.","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":6788,"name":"Rossella Sblendido","email":"rsblendido@suse.com","username":"rossella-o"},"change_message_id":"57b1134dbc5687bc5fd394b11e6d3e726ef5f5c9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":24,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":25,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":26,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":27,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_f2809f48","line":26,"updated":"2015-06-12 13:46:33.000000000","message":"it seems not to work for Nova, any info regarding that? [1]\n\n[1] https://bugs.launchpad.net/nova/+bug/1426332","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"a0f5d0a79f411f6e9fd5e4cc99eb2df8a1ee612c","unresolved":false,"context_lines":[{"line_number":23,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":24,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":25,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":26,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":27,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa32b979_98f0dfca","line":26,"in_reply_to":"1af86dd1_f2809f48","updated":"2015-06-18 10:30:24.000000000","message":"What do you mean?","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":6788,"name":"Rossella Sblendido","email":"rsblendido@suse.com","username":"rossella-o"},"change_message_id":"57b1134dbc5687bc5fd394b11e6d3e726ef5f5c9","unresolved":false,"context_lines":[{"line_number":49,"context_line":"traditional behaviour."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal also presupposes that"},{"line_number":52,"context_line":"enable_services_on_agents_with_admin_state_down is True in Neutron"},{"line_number":53,"context_line":"servers."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Data Model Impact"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_37362904","line":52,"updated":"2015-06-12 13:46:33.000000000","message":"If enable_services_on_agents_with_admin_state_down is True and admin_state_up\u003dFalse the operator can still manually schedule a resource to the agent right? Wouldn\u0027t that cover the case of testing a newly added network node?  Is it necessary to introduce this new conf param? Am I missing something?","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"a0f5d0a79f411f6e9fd5e4cc99eb2df8a1ee612c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"traditional behaviour."},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"The proposal also presupposes that"},{"line_number":52,"context_line":"enable_services_on_agents_with_admin_state_down is True in Neutron"},{"line_number":53,"context_line":"servers."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"Data Model Impact"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fa32b979_7895db30","line":52,"in_reply_to":"1af86dd1_37362904","updated":"2015-06-18 10:30:24.000000000","message":"This spec intends for operators to schedule manually a resource on the test agent.","commit_id":"4a3cf42e840c21199a4d428e2c88a4b2b60cd4cd"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"a6c9731b60bc7a9c0225836fb6159c865d2c8020","unresolved":false,"context_lines":[{"line_number":96,"context_line":"l3_agent.ini, etc). This config is set True as default value. It doesn\u0027t change"},{"line_number":97,"context_line":"a traditional behaviour of the agents. We also prefer xxx_agent.ini and"},{"line_number":98,"context_line":"neutron_xxxaas.conf rather than neutron.conf for enable_new_agents because some"},{"line_number":99,"context_line":"agents don\u0027t have scheduler logic."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Developer Impact"},{"line_number":102,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa32b979_6f017dae","line":99,"updated":"2015-06-25 09:01:19.000000000","message":"i\u0027m not sure about L97-99.\nfor the given use case, isn\u0027t it rather node-global config?","commit_id":"ba94893931425cf18c53431b7c12d6b16617c7be"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"36c329c5ac7a7f1d0b96fc87661597c70c5169ef","unresolved":false,"context_lines":[{"line_number":96,"context_line":"l3_agent.ini, etc). This config is set True as default value. It doesn\u0027t change"},{"line_number":97,"context_line":"a traditional behaviour of the agents. We also prefer xxx_agent.ini and"},{"line_number":98,"context_line":"neutron_xxxaas.conf rather than neutron.conf for enable_new_agents because some"},{"line_number":99,"context_line":"agents don\u0027t have scheduler logic."},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"Developer Impact"},{"line_number":102,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fa32b979_d31b681b","line":99,"in_reply_to":"fa32b979_6f017dae","updated":"2015-06-26 04:05:59.000000000","message":"I examined it again and then I decided to use global config. thanks.","commit_id":"ba94893931425cf18c53431b7c12d6b16617c7be"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"4ccb8bc6bfe186b425325d50b541f791a435e785","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_b4b92dce","line":43,"updated":"2015-07-02 16:48:22.000000000","message":"Let me clarify. Are you proposing to add the new option to the server config or the agent config? It is not clear from your description. I think you are proposing to add it to the server config.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2401dbe96415692a75ec6c6040f56e49141b1325","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_1feafcac","line":43,"in_reply_to":"ba3cc151_b4b92dce","updated":"2015-07-07 01:55:16.000000000","message":"It means server config. I added it to the spec.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"4ccb8bc6bfe186b425325d50b541f791a435e785","unresolved":false,"context_lines":[{"line_number":43,"context_line":"while starting."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."},{"line_number":47,"context_line":"* Neutron agent starts with admin_state_up\u003dFalse when enable_new_agents\u003dFalse."},{"line_number":48,"context_line":"  This behaviour is maintenance mode."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_34ce7d63","line":46,"updated":"2015-07-02 16:48:22.000000000","message":"general -\u003e default?","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2401dbe96415692a75ec6c6040f56e49141b1325","unresolved":false,"context_lines":[{"line_number":43,"context_line":"while starting."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."},{"line_number":47,"context_line":"* Neutron agent starts with admin_state_up\u003dFalse when enable_new_agents\u003dFalse."},{"line_number":48,"context_line":"  This behaviour is maintenance mode."},{"line_number":49,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_7fc4683f","line":46,"in_reply_to":"ba3cc151_34ce7d63","updated":"2015-07-07 01:55:16.000000000","message":"Done","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"4ccb8bc6bfe186b425325d50b541f791a435e785","unresolved":false,"context_lines":[{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."},{"line_number":47,"context_line":"* Neutron agent starts with admin_state_up\u003dFalse when enable_new_agents\u003dFalse."},{"line_number":48,"context_line":"  This behaviour is maintenance mode."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"The default value is True because this proposal don\u0027t intend to change a"},{"line_number":51,"context_line":"traditional behaviour."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_94e989b9","line":48,"updated":"2015-07-02 16:48:22.000000000","message":"I am not sure what is \"general\" and what is \"maintenance mode\".","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2401dbe96415692a75ec6c6040f56e49141b1325","unresolved":false,"context_lines":[{"line_number":45,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":46,"context_line":"  This behaviour is general."},{"line_number":47,"context_line":"* Neutron agent starts with admin_state_up\u003dFalse when enable_new_agents\u003dFalse."},{"line_number":48,"context_line":"  This behaviour is maintenance mode."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"The default value is True because this proposal don\u0027t intend to change a"},{"line_number":51,"context_line":"traditional behaviour."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_dfc5743d","line":48,"in_reply_to":"ba3cc151_94e989b9","updated":"2015-07-07 01:55:16.000000000","message":"I updated so that it\u0027s clear.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"4ccb8bc6bfe186b425325d50b541f791a435e785","unresolved":false,"context_lines":[{"line_number":162,"context_line":""},{"line_number":163,"context_line":"1. Admin prepares a new network node."},{"line_number":164,"context_line":"2. Admin adds enable_new_agents\u003dFalse to neutron.conf and starts their agents."},{"line_number":165,"context_line":"3. They run with admin_state_up\u003dFalse."},{"line_number":166,"context_line":"4. Admin creates network resources on the agents."},{"line_number":167,"context_line":"5. Admin creates VM connected the network resources, then admin"},{"line_number":168,"context_line":"   confirms the capability."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_d41e316f","line":165,"updated":"2015-07-02 16:48:22.000000000","message":"Are all agents affected? What I would like to ask is whether agents like L2 agent (e.g., ovs-agent), metadata agent, LBaaS agent. I think some agent honors admin_state_up.\n\nI would like to clarify what agents needs to be set to admin_state_up\u003dTrue before testing scheduling of network/router/LB.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2401dbe96415692a75ec6c6040f56e49141b1325","unresolved":false,"context_lines":[{"line_number":162,"context_line":""},{"line_number":163,"context_line":"1. Admin prepares a new network node."},{"line_number":164,"context_line":"2. Admin adds enable_new_agents\u003dFalse to neutron.conf and starts their agents."},{"line_number":165,"context_line":"3. They run with admin_state_up\u003dFalse."},{"line_number":166,"context_line":"4. Admin creates network resources on the agents."},{"line_number":167,"context_line":"5. Admin creates VM connected the network resources, then admin"},{"line_number":168,"context_line":"   confirms the capability."}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_ffaab884","line":165,"in_reply_to":"ba3cc151_d41e316f","updated":"2015-07-07 01:55:16.000000000","message":"This proposal aims at preventing user\u0027s resource to be scheduling to test node so the agents is what has scheduling method, that is l3-agent, dhcp-agent and lbaas-agent. I added the explanation to the spec.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"4ccb8bc6bfe186b425325d50b541f791a435e785","unresolved":false,"context_lines":[{"line_number":163,"context_line":"1. Admin prepares a new network node."},{"line_number":164,"context_line":"2. Admin adds enable_new_agents\u003dFalse to neutron.conf and starts their agents."},{"line_number":165,"context_line":"3. They run with admin_state_up\u003dFalse."},{"line_number":166,"context_line":"4. Admin creates network resources on the agents."},{"line_number":167,"context_line":"5. Admin creates VM connected the network resources, then admin"},{"line_number":168,"context_line":"   confirms the capability."},{"line_number":169,"context_line":"6. Admin deletes all resources and update agents to be"}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_b4f12d6d","line":166,"updated":"2015-07-02 16:48:22.000000000","message":"(question) Precisely speaking, admin needs to create a network (or router) and allocates it to a target agent. Correct?","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2401dbe96415692a75ec6c6040f56e49141b1325","unresolved":false,"context_lines":[{"line_number":163,"context_line":"1. Admin prepares a new network node."},{"line_number":164,"context_line":"2. Admin adds enable_new_agents\u003dFalse to neutron.conf and starts their agents."},{"line_number":165,"context_line":"3. They run with admin_state_up\u003dFalse."},{"line_number":166,"context_line":"4. Admin creates network resources on the agents."},{"line_number":167,"context_line":"5. Admin creates VM connected the network resources, then admin"},{"line_number":168,"context_line":"   confirms the capability."},{"line_number":169,"context_line":"6. Admin deletes all resources and update agents to be"}],"source_content_type":"text/x-rst","patch_set":11,"id":"ba3cc151_9fa84c79","line":166,"in_reply_to":"ba3cc151_b4f12d6d","updated":"2015-07-07 01:55:16.000000000","message":"Yes. Done.","commit_id":"05dc051c83051f841d03e898d1d48969f2f3ff82"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_c5955e33","line":22,"updated":"2015-07-08 09:49:17.000000000","message":"s/Cloud/cloud/","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_34a88820","line":22,"in_reply_to":"ba3cc151_c5955e33","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_85b2f6e3","line":23,"updated":"2015-07-08 09:49:17.000000000","message":"prevent users controlling\n--\u003e prevent users from controlling","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Neutron doesn\u0027t have a way to test a newly added network node by deploying test"},{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_94ab941e","line":23,"in_reply_to":"ba3cc151_85b2f6e3","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":27,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":28,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_05d6e6f0","line":25,"updated":"2015-07-08 09:49:17.000000000","message":"prevent users to create\n--\u003e prevent users from creating","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":22,"context_line":"resource before any customer resource on the node is deployed. When Cloud"},{"line_number":23,"context_line":"operators control network resources, they want to prevent users controlling the"},{"line_number":24,"context_line":"resources. For example, they may try to create test resources on a new node"},{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":27,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":28,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_74ae1031","line":25,"in_reply_to":"ba3cc151_05d6e6f0","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":27,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":28,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":29,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_28fbc1c2","line":28,"updated":"2015-07-08 09:49:17.000000000","message":"has --\u003e have","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":25,"context_line":"while deploying the new node. Neutron can prevent users to create the resources"},{"line_number":26,"context_line":"on the node by agent\u0027s admin_state_up\u003dFalse, but Neutron cannot start agent"},{"line_number":27,"context_line":"with admin_state_up\u003dFalse. Neutron always starts agent with"},{"line_number":28,"context_line":"admin_state_up\u003dTrue. Nova and Cinder has the setting of \"enable_new_services\""},{"line_number":29,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_d4b83c78","line":28,"in_reply_to":"ba3cc151_28fbc1c2","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":29,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":32,"context_line":"prevent all users to create the resources on the agent with"},{"line_number":33,"context_line":"admin_state_up\u003dFalse, but Neutron usually cannot allow admin user only"},{"line_number":34,"context_line":"to create the resources. Currently, if"},{"line_number":35,"context_line":"enable_services_on_agents_with_admin_state_down configuration"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_456cee34","line":32,"updated":"2015-07-08 09:49:17.000000000","message":"prevent all users to create\n--\u003e \tprevent all users from creating","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":29,"context_line":"in each conf to disable the initial service status to achieve this."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Neutron also has a problem for the maintenance scenario. Neutron can"},{"line_number":32,"context_line":"prevent all users to create the resources on the agent with"},{"line_number":33,"context_line":"admin_state_up\u003dFalse, but Neutron usually cannot allow admin user only"},{"line_number":34,"context_line":"to create the resources. Currently, if"},{"line_number":35,"context_line":"enable_services_on_agents_with_admin_state_down configuration"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_b4b3389a","line":32,"in_reply_to":"ba3cc151_456cee34","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"339aacab59daaa7b474d8e987dd30d2aed9870eb","unresolved":false,"context_lines":[{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"some agents which has scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":47,"context_line":"  This behaviour is default. User freely creates their resources on the agent."}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_1d8c2b7e","line":44,"updated":"2015-07-09 15:35:40.000000000","message":"how about controlling all agents by the new configuration?\nFrom the perspective of agent API behavior, it looks more consistent.\nI don\u0027t think it is easy to enable agents which are not related to \"scheduling\" (like L2-agent, metadata-agent, metering-agent) after adding new agents.","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"some agents which has scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":47,"context_line":"  This behaviour is default. User freely creates their resources on the agent."}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_c8efb51c","line":44,"updated":"2015-07-08 09:49:17.000000000","message":"which has --\u003e which have","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"some agents which has scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"* Neutron agent starts with admin_state_up\u003dTrue when enable_new_agents\u003dTrue."},{"line_number":47,"context_line":"  This behaviour is default. User freely creates their resources on the agent."}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_548e6cd1","line":44,"in_reply_to":"ba3cc151_1d8c2b7e","updated":"2015-07-10 02:02:40.000000000","message":"New configuration does\u0027t affect non-scheduling agents since this proposal is useless for these agents. I understand that non-scheduling resources is used with scheduling resources, for example, VM and port, router(or dhcp-server) and metadata, etc so this proposal aims l3-agent, dhcp-agent and lbaas-agent even though you feel inconsistent .","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  This behaviour is maintenance mode. In the case, user\u0027s resources cannot be"},{"line_number":51,"context_line":"  created on the agent until admin changes admin_state_up to True."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"The default value is True because this proposal don\u0027t intend to change a"},{"line_number":54,"context_line":"traditional behaviour."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"The proposal also presupposes that"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_c569defa","line":53,"updated":"2015-07-08 09:49:17.000000000","message":"don\u0027t --\u003e doesn\u0027t?","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  This behaviour is maintenance mode. In the case, user\u0027s resources cannot be"},{"line_number":51,"context_line":"  created on the agent until admin changes admin_state_up to True."},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"The default value is True because this proposal don\u0027t intend to change a"},{"line_number":54,"context_line":"traditional behaviour."},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"The proposal also presupposes that"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_f4b9c07d","line":53,"in_reply_to":"ba3cc151_c569defa","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"03966ffd684087f303c53f3811b63244e6ebcee8","unresolved":false,"context_lines":[{"line_number":157,"context_line":""},{"line_number":158,"context_line":"The new config options will be documented."},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"The following is maintenance scenario. The maintenance scenario will be"},{"line_number":161,"context_line":"documented."},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"Precondition: Neutron servers run with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_08183d74","line":160,"updated":"2015-07-08 09:49:17.000000000","message":"maintenance scenario --\u003e a maintenance scenario","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2b1371b9e01e13506f8ca96c90a859144d457fb0","unresolved":false,"context_lines":[{"line_number":157,"context_line":""},{"line_number":158,"context_line":"The new config options will be documented."},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"The following is maintenance scenario. The maintenance scenario will be"},{"line_number":161,"context_line":"documented."},{"line_number":162,"context_line":""},{"line_number":163,"context_line":"Precondition: Neutron servers run with"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba3cc151_14bf8461","line":160,"in_reply_to":"ba3cc151_08183d74","updated":"2015-07-10 02:02:40.000000000","message":"Done","commit_id":"8fd16328588497ee14068d1832d0761501a73b44"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"b8271477ed3fb7053c0a0c4b1d3c2ae75046fd43","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"all agents which have scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)"},{"line_number":45,"context_line":"and don\u0027t have the method(i.e. ovs-agent, metadata-agent) so that deployers"},{"line_number":46,"context_line":"enable agents easily although this proposal gives a benefit to scheduling agents"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3a50d1a3_f649f631","line":43,"updated":"2015-07-23 05:38:43.000000000","message":"(update if there is another spin)\n\"neutron.conf\" is just a filename. I prefer to \"neutron-server\" instead.","commit_id":"a0a1ef66861a5b752a2598f87ece46dc538da1c9"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"7c95bdbdb6932dbbd69949a104a6e07016db9901","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"all agents which have scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)"},{"line_number":45,"context_line":"and don\u0027t have the method(i.e. ovs-agent, metadata-agent) so that deployers"},{"line_number":46,"context_line":"enable agents easily although this proposal gives a benefit to scheduling agents"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3a50d1a3_b9e1212c","line":43,"in_reply_to":"3a50d1a3_36851ece","updated":"2015-07-23 06:54:27.000000000","message":"the description \"Add enable_new_agents to agent conf\" made me think it\u0027s meant to be a config for agents, rather than neutron-server\u0027s.\n\nHirofumi,\ncan you clarify?","commit_id":"a0a1ef66861a5b752a2598f87ece46dc538da1c9"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"27b585ab6d550f7ddcbe4484322bc753dbda74a4","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"all agents which have scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)"},{"line_number":45,"context_line":"and don\u0027t have the method(i.e. ovs-agent, metadata-agent) so that deployers"},{"line_number":46,"context_line":"enable agents easily although this proposal gives a benefit to scheduling agents"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3a50d1a3_4a7cbdeb","line":43,"in_reply_to":"3a50d1a3_b9e1212c","updated":"2015-07-23 07:34:19.000000000","message":"You\u0027re right. I will update it.","commit_id":"a0a1ef66861a5b752a2598f87ece46dc538da1c9"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"fbd2ca96b607447c626c167939920754d2732496","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"This proposal adds enable_new_agents config for operator maintenance to agents"},{"line_number":42,"context_line":"in network nodes. Neutron agent\u0027s admin_state_up is controlled by this config"},{"line_number":43,"context_line":"while starting. The config is added to neutron.conf since the proposal provides"},{"line_number":44,"context_line":"all agents which have scheduling method(i.e.l3-agent, dhcp-agent, lbaas-agent)"},{"line_number":45,"context_line":"and don\u0027t have the method(i.e. ovs-agent, metadata-agent) so that deployers"},{"line_number":46,"context_line":"enable agents easily although this proposal gives a benefit to scheduling agents"}],"source_content_type":"text/x-rst","patch_set":15,"id":"3a50d1a3_36851ece","line":43,"in_reply_to":"3a50d1a3_f649f631","updated":"2015-07-23 05:53:51.000000000","message":"You got it.","commit_id":"a0a1ef66861a5b752a2598f87ece46dc538da1c9"}]}