)]}'
{"specs/train/probe-api-and-agent-extension.rst":[{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"link status through API. And we meet some ssh failures [1]_ during"},{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_9baa5b17","line":14,"range":{"start_line":14,"start_character":25,"end_line":14,"end_character":54},"updated":"2019-05-31 20:17:18.000000000","message":"It might be clearer to say \"we frequently encounter ssh failures\"","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"e2d988491b5edcccb8cda4128b9d4674d13b60f3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"link status through API. And we meet some ssh failures [1]_ during"},{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_d32af78e","line":14,"range":{"start_line":14,"start_character":25,"end_line":14,"end_character":54},"in_reply_to":"9fb8cfa7_9baa5b17","updated":"2019-06-01 08:39:46.000000000","message":"Done","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"},{"line_number":18,"context_line":"API which will partial relay on neutron-debug [2]_."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_db7533ce","line":18,"range":{"start_line":18,"start_character":15,"end_line":18,"end_character":28},"updated":"2019-05-31 20:17:18.000000000","message":"Do you mean \"partially rely\"?","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"link status through API. And we meet some ssh failures [1]_ during"},{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"},{"line_number":18,"context_line":"API which will partial relay on neutron-debug [2]_."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_9b91fbf4","line":18,"range":{"start_line":13,"start_character":0,"end_line":18,"end_character":51},"updated":"2019-05-31 20:17:18.000000000","message":"My problem with this proposal is that it turns Neutron into a monitoring system.  The division of responsibilities has always been:\n\n- neutron: this link is supposed to be up\n- monasca: this link is actually up; I tested it\n\nYou can substitute any other kind of monitoring system instead of monasca there, but the point remains true: Neutron\u0027s goal is to orchestrate the configuration of the system, not to healthcheck all the resources all the time.  I see this proposal taking Neutron away from what it is good at, and trying to make it a general purpose do-everything solution rather than optimizing it to be the best at the tasks for which it was intended.","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"488bbcf84adf1345cbf345b885b71157daf726f6","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"link status through API. And we meet some ssh failures [1]_ during"},{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"},{"line_number":18,"context_line":"API which will partial relay on neutron-debug [2]_."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_b31ae314","line":18,"range":{"start_line":13,"start_character":0,"end_line":18,"end_character":51},"in_reply_to":"9fb8cfa7_9b91fbf4","updated":"2019-06-01 06:08:42.000000000","message":"IMO, you are overthinking about this, it is a simple detecting API, no persistent process resources, no extra RPC or notification to monitor system. And this does not report any data to any data collector. It only returns the detect result to user by API response.","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"e2d988491b5edcccb8cda4128b9d4674d13b60f3","unresolved":false,"context_lines":[{"line_number":15,"context_line":"the neutron CI test. It is black-box testing, we can not do much"},{"line_number":16,"context_line":"detect work in the compute node or network node since tempest are"},{"line_number":17,"context_line":"mostly running by calling API. So this RFE is aiming to add such"},{"line_number":18,"context_line":"API which will partial relay on neutron-debug [2]_."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_7319cb86","line":18,"range":{"start_line":18,"start_character":15,"end_line":18,"end_character":28},"in_reply_to":"9fb8cfa7_db7533ce","updated":"2019-06-01 08:39:46.000000000","message":"Done","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"We noticed that sometimes the guest OS is not fully UP, but test"},{"line_number":26,"context_line":"case is trying to login it. A simple idea is to ping it first,"},{"line_number":27,"context_line":"then try to login. So we hope to find a way for tempest to"},{"line_number":28,"context_line":"verify the neutron port link state. In high probability, the"},{"line_number":29,"context_line":"DB resource state is not reliable. We need an independent"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_bb7cffb5","line":26,"range":{"start_line":26,"start_character":15,"end_line":26,"end_character":26},"updated":"2019-05-31 20:17:18.000000000","message":"to login to it","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"e2d988491b5edcccb8cda4128b9d4674d13b60f3","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"We noticed that sometimes the guest OS is not fully UP, but test"},{"line_number":26,"context_line":"case is trying to login it. A simple idea is to ping it first,"},{"line_number":27,"context_line":"then try to login. So we hope to find a way for tempest to"},{"line_number":28,"context_line":"verify the neutron port link state. In high probability, the"},{"line_number":29,"context_line":"DB resource state is not reliable. We need an independent"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_931ebf6e","line":26,"range":{"start_line":26,"start_character":15,"end_line":26,"end_character":26},"in_reply_to":"9fb8cfa7_bb7cffb5","updated":"2019-06-01 08:39:46.000000000","message":"Done","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":38,"context_line":"will be a general agent extension, you can enable it to the L2-agent,"},{"line_number":39,"context_line":"L3-agent or DHCP-agent. Once you have such \"probe\" resource in"},{"line_number":40,"context_line":"the agent side, you will be able to run any command (or some"},{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_7651be0e","line":41,"range":{"start_line":41,"start_character":0,"end_line":41,"end_character":9},"updated":"2019-05-31 20:17:18.000000000","message":"Do you mean \"specific\" here?  The word \"statutory\" means \"enacted, regulated, or imposed by a legal authority\" and doesn\u0027t really apply unless the commands are dictated by national laws.","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"488bbcf84adf1345cbf345b885b71157daf726f6","unresolved":false,"context_lines":[{"line_number":38,"context_line":"will be a general agent extension, you can enable it to the L2-agent,"},{"line_number":39,"context_line":"L3-agent or DHCP-agent. Once you have such \"probe\" resource in"},{"line_number":40,"context_line":"the agent side, you will be able to run any command (or some"},{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_933b1f9b","line":41,"range":{"start_line":41,"start_character":0,"end_line":41,"end_character":9},"in_reply_to":"9fb8cfa7_7651be0e","updated":"2019-06-01 06:08:42.000000000","message":"I\u0027m not a native English speaker, here my thought is to allow the user only run some defined  (or hard code, or configured) commands, not any commands from API.","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":38,"context_line":"will be a general agent extension, you can enable it to the L2-agent,"},{"line_number":39,"context_line":"L3-agent or DHCP-agent. Once you have such \"probe\" resource in"},{"line_number":40,"context_line":"the agent side, you will be able to run any command (or some"},{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."},{"line_number":45,"context_line":"But we can remain that \"probe\" mechanism."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_56835a57","line":42,"range":{"start_line":41,"start_character":27,"end_line":42,"end_character":23},"updated":"2019-05-31 20:17:18.000000000","message":"So if I understand the neutron-debug functionality correctly, it calls neutron APIs to set up a port and interface on a specific subnet and plug it in.  Are you saying that when this funtionality is in use all VMs would have an additional port for use for this monitoring/backdoor access?","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"488bbcf84adf1345cbf345b885b71157daf726f6","unresolved":false,"context_lines":[{"line_number":38,"context_line":"will be a general agent extension, you can enable it to the L2-agent,"},{"line_number":39,"context_line":"L3-agent or DHCP-agent. Once you have such \"probe\" resource in"},{"line_number":40,"context_line":"the agent side, you will be able to run any command (or some"},{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."},{"line_number":45,"context_line":"But we can remain that \"probe\" mechanism."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_f3493b32","line":42,"range":{"start_line":41,"start_character":27,"end_line":42,"end_character":23},"in_reply_to":"9fb8cfa7_56835a57","updated":"2019-06-01 06:08:42.000000000","message":"Basically right, maybe you can enjoy the neutron-debug tool first:\nhttp://paste.openstack.org/show/752395/","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"67cb94cc643c6a46e2cdce682c8ad156f6bb7910","unresolved":false,"context_lines":[{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."},{"line_number":45,"context_line":"But we can remain that \"probe\" mechanism."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_f6936ea2","line":45,"range":{"start_line":44,"start_character":0,"end_line":45,"end_character":41},"updated":"2019-05-31 20:17:18.000000000","message":"Looking at the code [1] the probe is just a port.  If an instance is not up yet, how would an additional port help you determine that more effectively than the primary port already assigned to it?\n\n[1] https://opendev.org/openstack/neutron/src/branch/master/neutron/debug/debug_agent.py#L44-L66","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"488bbcf84adf1345cbf345b885b71157daf726f6","unresolved":false,"context_lines":[{"line_number":41,"context_line":"statutory commands) in it. This will be useful for neutron CI to"},{"line_number":42,"context_line":"check the VM link state."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Since [3]_ marked the neutron-debug tool as deprecated."},{"line_number":45,"context_line":"But we can remain that \"probe\" mechanism."},{"line_number":46,"context_line":""},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9fb8cfa7_d369978b","line":45,"range":{"start_line":44,"start_character":0,"end_line":45,"end_character":41},"in_reply_to":"9fb8cfa7_f6936ea2","updated":"2019-06-01 06:08:42.000000000","message":"It is irrelevant for any resource not involved to the probe mechanism. This means, probe can be created at any time, WHEN you want to detect the network resource connectivity, you can use it. Your concern is the `when`, the users or the CI tests should take care about when they try to use such probe detecting. For instance, when port is ACTIVE status, or when VM is ACTIVE status. A more interesting use case is when users try to shutdown an instance, it can use such mechanism to detect if it is real powered off.","commit_id":"1f9c6af6491b5572cdaf7c45ecf52115a3e07d06"}],"specs/ussuri/detection-api-and-agent.rst":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":34,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":35,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":36,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":37,"context_line":"such detection work in a remote host."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Then we need an independent detection mechanism to check the port link"},{"line_number":40,"context_line":"status. With such mechanism when the users or testing cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_1a4b3b5f","line":37,"updated":"2019-12-06 10:54:32.000000000","message":"described here use case related to tempest testing can be in many cases workarounded by logging VM\u0027s console log where You usually can see if it got IP address from DHCP or not, if it was able to connect to Metadata service or not and so on.\nSo maybe we can also e.g. add some user-data script which will check connectivity from the VM to the DHCP/GW IPs and this would be logged there too?","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":34,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":35,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":36,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":37,"context_line":"such detection work in a remote host."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Then we need an independent detection mechanism to check the port link"},{"line_number":40,"context_line":"status. With such mechanism when the users or testing cases"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ff570b3c_ec0f3685","line":37,"in_reply_to":"3fa7e38b_1a4b3b5f","updated":"2020-05-20 15:39:37.000000000","message":"I may say the neutron CI should be one case, this should be mainly for the production environment. When the operators want to detect the VM connectitity, no need to login to the host, something can be run by call API. Neutron upstream CI can leverage this new mechanism to detect the data plane of the VM.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":49,"context_line":"be a generic neutron port link state detection mechanism, it can be used"},{"line_number":50,"context_line":"in some scenarios else, not just CI. If the user or operators want to"},{"line_number":51,"context_line":"confirm a traffic state, such API will significantly get efficiency"},{"line_number":52,"context_line":"improvement."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":55,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_ba3b07c6","line":52,"updated":"2019-12-06 10:54:32.000000000","message":"where this probe will be installed? On same host as VM is or somewhere else?\nIf it will be on same host You will not detect e.g. problems with configuration of tunnels.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"be a generic neutron port link state detection mechanism, it can be used"},{"line_number":50,"context_line":"in some scenarios else, not just CI. If the user or operators want to"},{"line_number":51,"context_line":"confirm a traffic state, such API will significantly get efficiency"},{"line_number":52,"context_line":"improvement."},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":55,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ff570b3c_ec01f66b","line":52,"in_reply_to":"3fa7e38b_ba3b07c6","updated":"2020-05-20 15:39:37.000000000","message":"The probe can be installed in same host or others.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":61,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":62,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":63,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":64,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":65,"context_line":"status of CI again and again 3) we can more directly say there are"},{"line_number":66,"context_line":"problems with other services or resources."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_ba6467e6","line":64,"range":{"start_line":64,"start_character":44,"end_line":64,"end_character":60},"updated":"2019-12-06 10:54:32.000000000","message":"troubleshooting","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":62,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":63,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":64,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":65,"context_line":"status of CI again and again 3) we can more directly say there are"},{"line_number":66,"context_line":"problems with other services or resources."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Proposed Change"},{"line_number":69,"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":3,"id":"3fa7e38b_3a9e37b8","line":66,"range":{"start_line":65,"start_character":32,"end_line":66,"end_character":41},"updated":"2019-12-06 10:54:32.000000000","message":"can You elaborate more on this?","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":63,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":64,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":65,"context_line":"status of CI again and again 3) we can more directly say there are"},{"line_number":66,"context_line":"problems with other services or resources."},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Proposed Change"},{"line_number":69,"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":3,"id":"ff570b3c_accc1e35","line":66,"range":{"start_line":65,"start_character":32,"end_line":66,"end_character":41},"in_reply_to":"3fa7e38b_3a9e37b8","updated":"2020-05-20 15:39:37.000000000","message":"Maybe I should delete this, nothing technically. Everytime the VM is not reachable or the login is failed, the users may say the problem is in Neutron in first glance. But if we detect everything if normal, we can quiclly and directly say no to them.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":92,"context_line":"++++++++++++++++++"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":95,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":96,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":97,"context_line":"  custom commands execution available."},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_baa9c7dd","line":95,"updated":"2019-12-06 10:54:32.000000000","message":"can this be done as L2 agent extension?","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":92,"context_line":"++++++++++++++++++"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":95,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":96,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":97,"context_line":"  custom commands execution available."},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ff570b3c_ac51be38","line":95,"in_reply_to":"3fa7e38b_baa9c7dd","updated":"2020-05-20 15:39:37.000000000","message":"It can I guess. But this is a new resouce manager, the ovs-agent is mainly for ports. If this is implemented as a L2 agent, the L2 agent may need to do something like: create the namespace, plug the new device, procesing the related L2 stuffs. It is a bit heavy for L2 agent.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"  * check if the port status is ACTIVE"},{"line_number":139,"context_line":"  * create the detector for the VM network by calling API introduced by this"},{"line_number":140,"context_line":"  * run ping command or ssh command to detect the link"},{"line_number":141,"context_line":"  * wait until the VM is reachable or timeout"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* do next step of the testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_5a8bf367","line":140,"updated":"2019-12-06 10:54:32.000000000","message":"most (or even all) tempest tests already are doing it. So how this will be different from what we have now?","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"  * check if the port status is ACTIVE"},{"line_number":139,"context_line":"  * create the detector for the VM network by calling API introduced by this"},{"line_number":140,"context_line":"  * run ping command or ssh command to detect the link"},{"line_number":141,"context_line":"  * wait until the VM is reachable or timeout"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"* do next step of the testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"ff570b3c_4e1bb771","line":140,"in_reply_to":"3fa7e38b_5a8bf367","updated":"2020-05-20 15:39:37.000000000","message":"Sorry, it should be using \u0027nc\u0027 to dectect the sshd listen port, or ping to detect the VM is reachable.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5e07be350f7df34ee8919a699353219286b27b23","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                           nullable\u003dFalse)"},{"line_number":161,"context_line":"    port_id \u003d sa.Column(sa.String(36),"},{"line_number":162,"context_line":"                        sa.ForeignKey(\"ports.id\", ondelete\u003d\u0027CASCADE\u0027),"},{"line_number":163,"context_line":"                        nullable\u003dFalse)"},{"line_number":164,"context_line":"    status \u003d sa.Column(sa.String(16))"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_faf17f01","line":163,"updated":"2019-12-06 10:54:32.000000000","message":"why we need both port_id and network_id here?","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"c82945fccde02f80c87dc5a27b608e5067f2c04e","unresolved":false,"context_lines":[{"line_number":160,"context_line":"                           nullable\u003dFalse)"},{"line_number":161,"context_line":"    port_id \u003d sa.Column(sa.String(36),"},{"line_number":162,"context_line":"                        sa.ForeignKey(\"ports.id\", ondelete\u003d\u0027CASCADE\u0027),"},{"line_number":163,"context_line":"                        nullable\u003dFalse)"},{"line_number":164,"context_line":"    status \u003d sa.Column(sa.String(16))"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"3fa7e38b_7edc3117","line":163,"in_reply_to":"3fa7e38b_faf17f01","updated":"2020-05-20 15:39:37.000000000","message":"A new port will be created in this network, it will be plugged in the detector namespace.","commit_id":"bc6cd7f0ce2d4d8f013944b664cc0c8c16ec8bb4"}],"specs/victoria/detection-api-and-agent.rst":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"network resource link status through API. For a mature and robust"},{"line_number":15,"context_line":"software, a state checking mechanism is necessary, such as ``mysqlcheck``"},{"line_number":16,"context_line":"for database status, ``rabbitmqctl status`` for message queue status,"},{"line_number":17,"context_line":"and ``ovs-appctl ofproto/trace`` for tracing packets inside OpenvSwitch."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_7375e263","line":14,"range":{"start_line":14,"start_character":37,"end_line":14,"end_character":40},"updated":"2020-07-13 19:35:28.000000000","message":"s/the API","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"RFE: https://bugs.launchpad.net/neutron/+bug/1830014"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"There is no general mechanism for neutron users/operators to detect"},{"line_number":14,"context_line":"network resource link status through API. For a mature and robust"},{"line_number":15,"context_line":"software, a state checking mechanism is necessary, such as ``mysqlcheck``"},{"line_number":16,"context_line":"for database status, ``rabbitmqctl status`` for message queue status,"},{"line_number":17,"context_line":"and ``ovs-appctl ofproto/trace`` for tracing packets inside OpenvSwitch."}],"source_content_type":"text/x-rst","patch_set":5,"id":"d7507a73_5ad431f6","line":14,"range":{"start_line":14,"start_character":37,"end_line":14,"end_character":40},"in_reply_to":"bf51134e_7375e263","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"for database status, ``rabbitmqctl status`` for message queue status,"},{"line_number":17,"context_line":"and ``ovs-appctl ofproto/trace`` for tracing packets inside OpenvSwitch."},{"line_number":18,"context_line":"For neutron, we can add a more flexible detection mechanism for"},{"line_number":19,"context_line":"administrators or users, so that users can more easily to know the"},{"line_number":20,"context_line":"link status."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_53709e72","line":19,"range":{"start_line":19,"start_character":33,"end_line":19,"end_character":38},"updated":"2020-07-13 19:35:28.000000000","message":"s/they","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":16,"context_line":"for database status, ``rabbitmqctl status`` for message queue status,"},{"line_number":17,"context_line":"and ``ovs-appctl ofproto/trace`` for tracing packets inside OpenvSwitch."},{"line_number":18,"context_line":"For neutron, we can add a more flexible detection mechanism for"},{"line_number":19,"context_line":"administrators or users, so that users can more easily to know the"},{"line_number":20,"context_line":"link status."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"8b67f7cb_0d87f814","line":19,"range":{"start_line":19,"start_character":33,"end_line":19,"end_character":38},"in_reply_to":"bf51134e_53709e72","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Problem Description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_9366b6ba","line":26,"range":{"start_line":26,"start_character":4,"end_line":26,"end_character":7},"updated":"2020-07-13 19:35:28.000000000","message":"s/a","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Problem Description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f30612ce","line":26,"range":{"start_line":26,"start_character":37,"end_line":26,"end_character":55},"updated":"2020-07-13 19:35:28.000000000","message":"s/an operator wants","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Problem Description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"a2fd2078_aa1b86a9","line":26,"range":{"start_line":26,"start_character":4,"end_line":26,"end_character":7},"in_reply_to":"bf51134e_9366b6ba","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Problem Description"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"}],"source_content_type":"text/x-rst","patch_set":5,"id":"f3350617_5daf7553","line":26,"range":{"start_line":26,"start_character":37,"end_line":26,"end_character":55},"in_reply_to":"bf51134e_f30612ce","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_53f53eca","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":6},"updated":"2020-07-13 19:35:28.000000000","message":"s/VM (remove \u0027the\u0027)","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_13290636","line":27,"range":{"start_line":27,"start_character":28,"end_line":27,"end_character":40},"updated":"2020-07-13 19:35:28.000000000","message":"s/logging-into a\n\nAnd this is \u0027host\u0027 since we\u0027re talking about a compute node for example?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"}],"source_content_type":"text/x-rst","patch_set":5,"id":"6298de05_a7aadc17","line":27,"range":{"start_line":27,"start_character":28,"end_line":27,"end_character":40},"in_reply_to":"bf51134e_13290636","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5ad14e8f_2c40fb3c","line":27,"range":{"start_line":27,"start_character":0,"end_line":27,"end_character":6},"in_reply_to":"bf51134e_53f53eca","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_73232257","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":16},"updated":"2020-07-13 19:35:28.000000000","message":"s/an operation via an API call directly","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3f32764d_19c086b8","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":16},"in_reply_to":"bf51134e_73232257","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_73e842e1","line":29,"range":{"start_line":29,"start_character":51,"end_line":29,"end_character":60},"updated":"2020-07-13 19:35:28.000000000","message":"s/a mechanism,","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":26,"context_line":"For the production environment, when the operators want to detect"},{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b236d826_186c824a","line":29,"range":{"start_line":29,"start_character":51,"end_line":29,"end_character":60},"in_reply_to":"bf51134e_73e842e1","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_53e37e05","line":31,"range":{"start_line":30,"start_character":54,"end_line":31,"end_character":11},"updated":"2020-07-13 19:35:28.000000000","message":"They\u0027re not calling the api in a host, just the API","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":27,"context_line":"the VM connectitity without login to the host, something they want is to"},{"line_number":28,"context_line":"run something by call API directly. We need an independent detection"},{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"3a452537_668f0dc6","line":31,"range":{"start_line":30,"start_character":54,"end_line":31,"end_character":11},"in_reply_to":"bf51134e_53e37e05","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f3a292ad","line":32,"range":{"start_line":32,"start_character":54,"end_line":32,"end_character":67},"updated":"2020-07-13 19:35:28.000000000","message":"So by traffic state do you mean that the neutron port can pass traffic?  Can\u0027t that already be done with say a ping of the instance?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":29,"context_line":"mechanism to check the port link status. With such mechanism when the"},{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"}],"source_content_type":"text/x-rst","patch_set":5,"id":"94460ee8_349be5b6","line":32,"range":{"start_line":32,"start_character":54,"end_line":32,"end_character":67},"in_reply_to":"bf51134e_f3a292ad","updated":"2021-01-12 09:19:09.000000000","message":"Ping can be one \u0027operation\u0027 for opeartors to run in the compute nodes. But now if the instance does not have a floating IP, we need to ping it in the dhcp namespace or router namespace.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_73d16223","line":33,"range":{"start_line":33,"start_character":28,"end_line":33,"end_character":31},"updated":"2020-07-13 19:35:28.000000000","message":"s/get an","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":30,"context_line":"users or testing cases need to verify the link state, just call the API"},{"line_number":31,"context_line":"in any host. This should be a generic neutron port link state detection"},{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""}],"source_content_type":"text/x-rst","patch_set":5,"id":"e4dc733e_1440d055","line":33,"range":{"start_line":33,"start_character":28,"end_line":33,"end_character":31},"in_reply_to":"bf51134e_73d16223","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":32,"context_line":"mechanism, if the user or operators want to confirm a traffic state,"},{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_b333faf5","line":35,"range":{"start_line":35,"start_character":56,"end_line":35,"end_character":61},"updated":"2020-07-13 19:35:28.000000000","message":"s/rely ?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":33,"context_line":"such API will significantly get efficiency improvement."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_13ac669d","line":36,"range":{"start_line":36,"start_character":53,"end_line":36,"end_character":60},"updated":"2020-07-13 19:35:28.000000000","message":"s/includes (or should this be \u0027creates\u0027 ?)","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_5391be51","line":37,"range":{"start_line":37,"start_character":42,"end_line":37,"end_character":46},"updated":"2020-07-13 19:35:28.000000000","message":"s/an","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_9397d645","line":38,"range":{"start_line":38,"start_character":17,"end_line":38,"end_character":23},"updated":"2020-07-13 19:35:28.000000000","message":"s/add (remove \u0027to\u0027)","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f38bb21b","line":38,"range":{"start_line":38,"start_character":37,"end_line":38,"end_character":59},"updated":"2020-07-13 19:35:28.000000000","message":"s/namespaces","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"6eab58d5_2c577dc8","line":38,"range":{"start_line":38,"start_character":17,"end_line":38,"end_character":23},"in_reply_to":"bf51134e_9397d645","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":35,"context_line":"This RFE is aiming to add such API which will partially relay on"},{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"2e5c9bc0_86e9e225","line":38,"range":{"start_line":38,"start_character":37,"end_line":38,"end_character":59},"in_reply_to":"bf51134e_f38bb21b","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Neutron upstream CI can leverage this new mechanism to detect the data"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_33824a01","line":39,"range":{"start_line":39,"start_character":0,"end_line":39,"end_character":2},"updated":"2020-07-13 19:35:28.000000000","message":"s/On the","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":36,"context_line":"neutron-debug [2]_. We have neutron-debug tool which include a \"probe\""},{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Neutron upstream CI can leverage this new mechanism to detect the data"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_737fa214","line":39,"range":{"start_line":39,"start_character":29,"end_line":39,"end_character":33},"updated":"2020-07-13 19:35:28.000000000","message":"s/this","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":37,"context_line":"namespace in the agent side. We could add some API to neutron, and let"},{"line_number":38,"context_line":"the proper agent to add such \"probe\" namespace installation for us."},{"line_number":39,"context_line":"In agent side, once you have such \"probe\" resource you will be able to"},{"line_number":40,"context_line":"run any command (or some defined/configured commands) in it."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Neutron upstream CI can leverage this new mechanism to detect the data"},{"line_number":43,"context_line":"plane of the VM."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_13b3c6ea","line":40,"updated":"2020-07-13 19:35:28.000000000","message":"So are we talking about adding an API where a user can give a command to run?  That seems like a security issue if it\u0027s running inside a namespace on a host since it would have escalated privileges to do that, right?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2244a634c250364eed1a561d4938808aa51b70ce","unresolved":false,"context_lines":[{"line_number":43,"context_line":"plane of the VM."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_a31e93b7","line":46,"range":{"start_line":46,"start_character":56,"end_line":46,"end_character":63},"updated":"2020-06-17 13:09:26.000000000","message":"nit: s/manually","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":43,"context_line":"plane of the VM."},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f3de120f","line":46,"range":{"start_line":46,"start_character":48,"end_line":46,"end_character":52},"updated":"2020-07-13 19:35:28.000000000","message":"s/needs","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."},{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_13362652","line":47,"range":{"start_line":47,"start_character":10,"end_line":47,"end_character":13},"updated":"2020-07-13 19:35:28.000000000","message":"s/Also, since bug [3]_, the neutron-debug tool has been marked as deprecated.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":45,"context_line":"Why not directly use the neutron-debug tool?"},{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."},{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."},{"line_number":51,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_53cd3e5b","line":48,"range":{"start_line":48,"start_character":67,"end_line":48,"end_character":74},"updated":"2020-07-13 19:35:28.000000000","message":"s/commands","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."},{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_93d35601","line":49,"range":{"start_line":49,"start_character":16,"end_line":49,"end_character":23},"updated":"2020-07-13 19:35:28.000000000","message":"s/host ?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":46,"context_line":"Firstly, because the current neutron-debug tool need to mannuly run in"},{"line_number":47,"context_line":"the host. And since bug [3]_ marked the neutron-debug tool as deprecated."},{"line_number":48,"context_line":"For some deployments with hierarchical roles, those who cannot run command"},{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."},{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_73fb2278","line":49,"range":{"start_line":49,"start_character":46,"end_line":49,"end_character":49},"updated":"2020-07-13 19:35:28.000000000","message":"s/this","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":49,"context_line":"directly in the cluster can also benefit from the API."},{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_b3465abf","line":53,"range":{"start_line":52,"start_character":63,"end_line":53,"end_character":2},"updated":"2020-07-13 19:35:28.000000000","message":"s/If (remove And)","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":50,"context_line":"But We will do some refactor work and reuse some of that \"probe\" mechanism."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_330c0a93","line":53,"range":{"start_line":53,"start_character":21,"end_line":53,"end_character":23},"updated":"2020-07-13 19:35:28.000000000","message":"s/for","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_138466d2","line":55,"range":{"start_line":55,"start_character":44,"end_line":55,"end_character":60},"updated":"2020-07-13 19:35:28.000000000","message":"nit: s/trouble-shooting","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_931a36c3","line":55,"range":{"start_line":55,"start_character":34,"end_line":55,"end_character":39},"updated":"2020-07-13 19:35:28.000000000","message":"s/time","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"c9183957_59a3e7f4","line":55,"range":{"start_line":55,"start_character":44,"end_line":55,"end_character":60},"in_reply_to":"bf51134e_138466d2","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":52,"context_line":"This will be useful for neutron CI to check the VM link state. And"},{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":5,"id":"d871a843_c2dc369a","line":55,"range":{"start_line":55,"start_character":34,"end_line":55,"end_character":39},"in_reply_to":"bf51134e_931a36c3","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"},{"line_number":59,"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":5,"id":"bf51134e_53525e68","line":56,"updated":"2020-07-13 19:35:28.000000000","message":"Missing period here.\n\nCan you explain a little more how this would be used?  Do you mean if some resource is not working correctly, we would use this to probe it\u0027s current state to help with debugging the tests?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":53,"context_line":"if we implement this to some critical test cases, we will benefit"},{"line_number":54,"context_line":"from it, something like this: 1) record and realize if neutron works"},{"line_number":55,"context_line":"fine for the network link 2) save times for trouble shooting the link"},{"line_number":56,"context_line":"status of CI again and again"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"Proposed Change"},{"line_number":59,"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":5,"id":"3d8cdf03_66aa9087","line":56,"in_reply_to":"bf51134e_53525e68","updated":"2021-01-12 09:19:09.000000000","message":"The section `The neutron CI use case workflow` can be the details for how to detect the network state for test cases.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":59,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_53d7febf","line":62,"range":{"start_line":62,"start_character":49,"end_line":62,"end_character":54},"updated":"2020-07-13 19:35:28.000000000","message":"s/The agent","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":58,"context_line":"Proposed Change"},{"line_number":59,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_73dcc2e3","line":62,"range":{"start_line":61,"start_character":65,"end_line":62,"end_character":4},"updated":"2020-07-13 19:35:28.000000000","message":"s/consist of a","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":59,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"e0698bda_13ba5229","line":62,"range":{"start_line":62,"start_character":49,"end_line":62,"end_character":54},"in_reply_to":"bf51134e_53d7febf","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":58,"context_line":"Proposed Change"},{"line_number":59,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"2f368d60_09d356f9","line":62,"range":{"start_line":61,"start_character":65,"end_line":62,"end_character":4},"in_reply_to":"bf51134e_73dcc2e3","updated":"2021-01-12 09:19:09.000000000","message":"It\u0027s not one API, but some CURD for the new resource, and a \u0027detect\u0027 API for running commands.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"This can be used for not only neutron agents, but also some other networking"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_93dd16df","line":63,"range":{"start_line":63,"start_character":0,"end_line":63,"end_character":4},"updated":"2020-07-13 19:35:28.000000000","message":"s/a","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"This can be used for not only neutron agents, but also some other networking"},{"line_number":67,"context_line":"mechanism drivers like OVN, ODL etc."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_d3f4ae5d","line":64,"range":{"start_line":63,"start_character":49,"end_line":64,"end_character":18},"updated":"2020-07-13 19:35:28.000000000","message":"s/and run the detection","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"For this proposal, a new neutron resource will be added, it will contain"},{"line_number":62,"context_line":"some new API, plugin, extension and OVO objects. Agent side will also need"},{"line_number":63,"context_line":"some change to install the namespace and device, and most important is"},{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"This can be used for not only neutron agents, but also some other networking"},{"line_number":67,"context_line":"mechanism drivers like OVN, ODL etc."}],"source_content_type":"text/x-rst","patch_set":5,"id":"ebec63a5_157f9bfd","line":64,"range":{"start_line":63,"start_character":49,"end_line":64,"end_character":18},"in_reply_to":"bf51134e_d3f4ae5d","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2244a634c250364eed1a561d4938808aa51b70ce","unresolved":false,"context_lines":[{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"This can be used for not only neutron agents, but also some other networking"},{"line_number":67,"context_line":"mechanism drivers like OVN, ODL etc."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Solution Proposed"},{"line_number":70,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_03f01f67","line":67,"range":{"start_line":67,"start_character":23,"end_line":67,"end_character":26},"updated":"2020-06-17 13:09:26.000000000","message":"IIUC correctly You are proposing new agent which will be run on all (or some) hosts. How this can work with OVN or ODL which don\u0027t have agents on hosts? Can You elaborate more about it?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"bd33e9a61eaebda7377998b4bba40013e2fe5c11","unresolved":false,"context_lines":[{"line_number":64,"context_line":"to run the detect commands."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"This can be used for not only neutron agents, but also some other networking"},{"line_number":67,"context_line":"mechanism drivers like OVN, ODL etc."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Solution Proposed"},{"line_number":70,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_fea04794","line":67,"range":{"start_line":67,"start_character":23,"end_line":67,"end_character":26},"in_reply_to":"bf51134e_03f01f67","updated":"2020-07-01 03:07:09.000000000","message":"OK, this should be the API reusing. I will make it most accurate.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":82,"context_line":"++++++++++++++++++"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":85,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":86,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":87,"context_line":"  custom commands execution available."},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_d3994e1f","line":85,"updated":"2020-07-13 19:35:28.000000000","message":"Missing period\n\nAlso, should this be \u0027creates the related device\u0027 ?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":82,"context_line":"++++++++++++++++++"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":85,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":86,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":87,"context_line":"  custom commands execution available."},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"db623a8f_c46cb671","line":85,"in_reply_to":"bf51134e_d3994e1f","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2244a634c250364eed1a561d4938808aa51b70ce","unresolved":false,"context_lines":[{"line_number":84,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":85,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":86,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":87,"context_line":"  custom commands execution available."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Potential configurations"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_0395bfe4","line":87,"updated":"2020-06-17 13:09:26.000000000","message":"I think we already have notifications mechanism between server side and agents. We are using RPC for that. What else You want to add?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"bd33e9a61eaebda7377998b4bba40013e2fe5c11","unresolved":false,"context_lines":[{"line_number":84,"context_line":"* Add a simple detector-agent which will install the \"probe\" namespace"},{"line_number":85,"context_line":"  reused from neutron-debug-tool and related device"},{"line_number":86,"context_line":"* Add notification mechanism between server side plugin and agent to make"},{"line_number":87,"context_line":"  custom commands execution available."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Potential configurations"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_de9d8358","line":87,"in_reply_to":"bf51134e_0395bfe4","updated":"2020-07-01 03:07:09.000000000","message":"This is just a new rpc method. I will update this.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":108,"context_line":"The neutron basic workflow"},{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_b3a93a25","line":111,"range":{"start_line":111,"start_character":3,"end_line":111,"end_character":13},"updated":"2020-07-13 19:35:28.000000000","message":"s/user calls","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2244a634c250364eed1a561d4938808aa51b70ce","unresolved":false,"context_lines":[{"line_number":108,"context_line":"The neutron basic workflow"},{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_03be5f64","line":111,"updated":"2020-06-17 13:09:26.000000000","message":"will it be admin only or for all users by default?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"bd33e9a61eaebda7377998b4bba40013e2fe5c11","unresolved":false,"context_lines":[{"line_number":108,"context_line":"The neutron basic workflow"},{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_dec6234a","line":111,"in_reply_to":"bf51134e_03be5f64","updated":"2020-07-01 03:07:09.000000000","message":"It should be changable by policy for those hierarchical operation management.","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":108,"context_line":"The neutron basic workflow"},{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b5527890_4e513d46","line":111,"range":{"start_line":111,"start_character":3,"end_line":111,"end_character":13},"in_reply_to":"bf51134e_b3a93a25","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_b3901aef","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":24},"updated":"2020-07-13 19:35:28.000000000","message":"s/creates","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":109,"context_line":"--------------------------"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"}],"source_content_type":"text/x-rst","patch_set":5,"id":"ac3d6c70_ca435408","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":24},"in_reply_to":"bf51134e_b3901aef","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_d3c2eee8","line":113,"range":{"start_line":113,"start_character":18,"end_line":113,"end_character":34},"updated":"2020-07-13 19:35:28.000000000","message":"s/schedules a","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":110,"context_line":""},{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"299c75b4_7cec6d77","line":113,"range":{"start_line":113,"start_character":18,"end_line":113,"end_character":34},"in_reply_to":"bf51134e_d3c2eee8","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_33b6ca8b","line":114,"range":{"start_line":114,"start_character":14,"end_line":114,"end_character":21},"updated":"2020-07-13 19:35:28.000000000","message":"s/creates","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":111,"context_line":"1. users call API to create a detector resource"},{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1c570242_06a73ae6","line":114,"range":{"start_line":114,"start_character":14,"end_line":114,"end_character":21},"in_reply_to":"bf51134e_33b6ca8b","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2244a634c250364eed1a561d4938808aa51b70ce","unresolved":false,"context_lines":[{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_465a25f3","line":115,"updated":"2020-06-17 13:09:26.000000000","message":"can You add description of this API also?","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_93a4f64c","line":115,"range":{"start_line":115,"start_character":3,"end_line":115,"end_character":7},"updated":"2020-07-13 19:35:28.000000000","message":"s/user calls","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"bd33e9a61eaebda7377998b4bba40013e2fe5c11","unresolved":false,"context_lines":[{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_fec9675a","line":115,"in_reply_to":"bf51134e_465a25f3","updated":"2020-07-01 03:07:09.000000000","message":"This is something like this:\nPOST neutron-server:9696/v2.0/dector/\u003cxxxx\u003e/dectect -d\n{\n \"tcp\": \"80\"\n}","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"570753bb_dd7f8cbd","line":115,"range":{"start_line":115,"start_character":3,"end_line":115,"end_character":7},"in_reply_to":"bf51134e_93a4f64c","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":112,"context_line":"2. neutron server create the DB records"},{"line_number":113,"context_line":"3. neutron server start scheduling detector to a valid host"},{"line_number":114,"context_line":"4. agent side create the namespace and device"},{"line_number":115,"context_line":"5. call API to run custom commands in that \"probe\" namespace"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"}],"source_content_type":"text/x-rst","patch_set":5,"id":"b724c276_eb7742ee","line":115,"in_reply_to":"bf51134e_fec9675a","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_d3678e24","line":120,"range":{"start_line":120,"start_character":22,"end_line":120,"end_character":31},"updated":"2020-07-13 19:35:28.000000000","message":"s/sometimes have","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":117,"context_line":"The neutron CI use case workflow"},{"line_number":118,"context_line":"--------------------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"}],"source_content_type":"text/x-rst","patch_set":5,"id":"abf88882_a4d3f550","line":120,"range":{"start_line":120,"start_character":22,"end_line":120,"end_character":31},"in_reply_to":"bf51134e_d3678e24","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":118,"context_line":"--------------------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_336b6af1","line":121,"range":{"start_line":121,"start_character":44,"end_line":121,"end_character":48},"updated":"2020-07-13 19:35:28.000000000","message":"s/the test","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":118,"context_line":"--------------------------------"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"}],"source_content_type":"text/x-rst","patch_set":5,"id":"f74913d0_d347e617","line":121,"range":{"start_line":121,"start_character":44,"end_line":121,"end_character":48},"in_reply_to":"bf51134e_336b6af1","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_b37e5aac","line":123,"range":{"start_line":123,"start_character":48,"end_line":123,"end_character":59},"updated":"2020-07-13 19:35:28.000000000","message":"s/to detect if it is working","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":120,"context_line":"Neutron CI test cases meet some ssh failures [1]_. We noticed that"},{"line_number":121,"context_line":"sometimes the guest OS is not fully UP, but test case is trying to"},{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"}],"source_content_type":"text/x-rst","patch_set":5,"id":"97e0ccad_395fee72","line":123,"range":{"start_line":123,"start_character":48,"end_line":123,"end_character":59},"in_reply_to":"bf51134e_b37e5aac","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f34d729a","line":125,"range":{"start_line":125,"start_character":11,"end_line":125,"end_character":14},"updated":"2020-07-13 19:35:28.000000000","message":"s/the","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":122,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bd659f3b_ba33542e","line":125,"range":{"start_line":125,"start_character":11,"end_line":125,"end_character":14},"in_reply_to":"bf51134e_f34d729a","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_33440a74","line":126,"range":{"start_line":126,"start_character":27,"end_line":126,"end_character":33},"updated":"2020-07-13 19:35:28.000000000","message":"s/be in","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":123,"context_line":"But it is black-box testing, we can not do much detect work"},{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"}],"source_content_type":"text/x-rst","patch_set":5,"id":"4c80d5c4_0bb3cb90","line":126,"range":{"start_line":126,"start_character":27,"end_line":126,"end_character":33},"in_reply_to":"bf51134e_33440a74","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_13474679","line":127,"range":{"start_line":127,"start_character":25,"end_line":127,"end_character":30},"updated":"2020-07-13 19:35:28.000000000","message":"s/starting a","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":124,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":125,"context_line":"by calling API. And in high probability, the DB resource states are"},{"line_number":126,"context_line":"offen not reliable, it may become ACTIVE status, but still doing"},{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"}],"source_content_type":"text/x-rst","patch_set":5,"id":"52f3421b_476e21b1","line":127,"range":{"start_line":127,"start_character":25,"end_line":127,"end_character":30},"in_reply_to":"bf51134e_13474679","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":131,"context_line":"such detection work in a remote host."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"For one test case, it is basicly like this:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f332920f","line":130,"range":{"start_line":130,"start_character":47,"end_line":130,"end_character":56},"updated":"2020-07-13 19:35:28.000000000","message":"s/and we","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":131,"context_line":"such detection work in a remote host."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"For one test case, it is basicly like this:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_93523634","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":6},"updated":"2020-07-13 19:35:28.000000000","message":"s/detection","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":131,"context_line":"such detection work in a remote host."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"For one test case, it is basicly like this:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"53258d5d_d6e98d9a","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":6},"in_reply_to":"bf51134e_93523634","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":127,"context_line":"some extra work, such as start the service process inside the guest OS."},{"line_number":128,"context_line":"Moreover we can not use current network resources under the existing"},{"line_number":129,"context_line":"mechanism, such as qdhcp-namepace or qrouter-namepace, to do such"},{"line_number":130,"context_line":"detect, because tempest can be run in any host we surely can not do"},{"line_number":131,"context_line":"such detection work in a remote host."},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"For one test case, it is basicly like this:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"237eb79e_61804f4f","line":130,"range":{"start_line":130,"start_character":47,"end_line":130,"end_character":56},"in_reply_to":"bf51134e_f332920f","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"* create tempest resources"},{"line_number":136,"context_line":"* create test VM"},{"line_number":137,"context_line":"* create router and connected to the VM\u0027s subnet"},{"line_number":138,"context_line":"* create a floating IP and associate to the VM\u0027s port"},{"line_number":139,"context_line":"* test case try to ssh the VM via floating IP"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_3339aaf1","line":137,"range":{"start_line":137,"start_character":20,"end_line":137,"end_character":29},"updated":"2020-07-13 19:35:28.000000000","message":"s/connect (or attach)","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":134,"context_line":""},{"line_number":135,"context_line":"* create tempest resources"},{"line_number":136,"context_line":"* create test VM"},{"line_number":137,"context_line":"* create router and connected to the VM\u0027s subnet"},{"line_number":138,"context_line":"* create a floating IP and associate to the VM\u0027s port"},{"line_number":139,"context_line":"* test case try to ssh the VM via floating IP"},{"line_number":140,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"14dae432_8d0d8cb9","line":137,"range":{"start_line":137,"start_character":20,"end_line":137,"end_character":29},"in_reply_to":"bf51134e_3339aaf1","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":136,"context_line":"* create test VM"},{"line_number":137,"context_line":"* create router and connected to the VM\u0027s subnet"},{"line_number":138,"context_line":"* create a floating IP and associate to the VM\u0027s port"},{"line_number":139,"context_line":"* test case try to ssh the VM via floating IP"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"  * check if the port status is ACTIVE"},{"line_number":142,"context_line":"  * create the detector for the VM network by calling API introduced by this"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_133c66df","line":139,"range":{"start_line":139,"start_character":12,"end_line":139,"end_character":15},"updated":"2020-07-13 19:35:28.000000000","message":"s/tries to ssh to the VM via its floating IP","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":136,"context_line":"* create test VM"},{"line_number":137,"context_line":"* create router and connected to the VM\u0027s subnet"},{"line_number":138,"context_line":"* create a floating IP and associate to the VM\u0027s port"},{"line_number":139,"context_line":"* test case try to ssh the VM via floating IP"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"  * check if the port status is ACTIVE"},{"line_number":142,"context_line":"  * create the detector for the VM network by calling API introduced by this"}],"source_content_type":"text/x-rst","patch_set":5,"id":"f58030d9_b3d4ad2c","line":139,"range":{"start_line":139,"start_character":12,"end_line":139,"end_character":15},"in_reply_to":"bf51134e_133c66df","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":210,"context_line":"Alternatives"},{"line_number":211,"context_line":"------------"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_732682d0","line":213,"range":{"start_line":213,"start_character":49,"end_line":213,"end_character":58},"updated":"2020-07-13 19:35:28.000000000","message":"s/, which was mentioned","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":210,"context_line":"Alternatives"},{"line_number":211,"context_line":"------------"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."}],"source_content_type":"text/x-rst","patch_set":5,"id":"7fda8125_520adf74","line":213,"range":{"start_line":213,"start_character":49,"end_line":213,"end_character":58},"in_reply_to":"bf51134e_732682d0","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":212,"context_line":""},{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_f31bb27d","line":215,"range":{"start_line":215,"start_character":58,"end_line":215,"end_character":68},"updated":"2020-07-13 19:35:28.000000000","message":"s/installation of","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":212,"context_line":""},{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":5,"id":"382c9524_6bc94160","line":215,"range":{"start_line":215,"start_character":58,"end_line":215,"end_character":68},"in_reply_to":"bf51134e_f31bb27d","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"Implementation"},{"line_number":219,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_33124a63","line":216,"range":{"start_line":216,"start_character":25,"end_line":216,"end_character":30},"updated":"2020-07-13 19:35:28.000000000","message":"s/the","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":213,"context_line":"The project skydive [4]_ could be an alternative mentioned during the"},{"line_number":214,"context_line":"Shanghai PTG related session. It provides real-time network topology"},{"line_number":215,"context_line":"with a nice web-UI. For OpenStack networking, it requires to install"},{"line_number":216,"context_line":"the skydive-agent to all nodes you want to investigate."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"Implementation"},{"line_number":219,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"e4ecbef7_1b6ae65a","line":216,"range":{"start_line":216,"start_character":25,"end_line":216,"end_character":30},"in_reply_to":"bf51134e_33124a63","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3c715488ae347f9524346acb4e9801d3c11b66c8","unresolved":false,"context_lines":[{"line_number":227,"context_line":"Work Items"},{"line_number":228,"context_line":"----------"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"* Implemente API extension detector."},{"line_number":231,"context_line":"* Implemente detector service plugin"},{"line_number":232,"context_line":"* Implemente detector agent and its RPC"},{"line_number":233,"context_line":"* Testing."}],"source_content_type":"text/x-rst","patch_set":5,"id":"bf51134e_93007604","line":230,"range":{"start_line":230,"start_character":2,"end_line":230,"end_character":12},"updated":"2020-07-13 19:35:28.000000000","message":"s/Implement, below too","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"},{"author":{"_account_id":9531,"name":"liuyulong","display_name":"LIU Yulong","email":"i@liuyulong.me","username":"LIU-Yulong"},"change_message_id":"70f4d2850f7a08af649ec303d92587dbc9064bc0","unresolved":false,"context_lines":[{"line_number":227,"context_line":"Work Items"},{"line_number":228,"context_line":"----------"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"* Implemente API extension detector."},{"line_number":231,"context_line":"* Implemente detector service plugin"},{"line_number":232,"context_line":"* Implemente detector agent and its RPC"},{"line_number":233,"context_line":"* Testing."}],"source_content_type":"text/x-rst","patch_set":5,"id":"3422980e_0847bbb9","line":230,"range":{"start_line":230,"start_character":2,"end_line":230,"end_character":12},"in_reply_to":"bf51134e_93007604","updated":"2021-01-12 09:19:09.000000000","message":"Done","commit_id":"afba448c05deb7f0ab3418ba2a20daa6c0b63bef"}],"specs/wallaby/detection-api-and-agent.rst":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e5d70450962ff59f82ae77842ab331c88ae48bbb","unresolved":true,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"Neutron CI test cases sometimes have ssh failures [1]_. We noticed that"},{"line_number":141,"context_line":"sometimes the guest OS is not fully UP, but the test case is trying to"},{"line_number":142,"context_line":"login to it. A simple idea is to ping it first, then try to login."},{"line_number":143,"context_line":"But it is black-box testing, we can not do much to detect if it is working"},{"line_number":144,"context_line":"in the compute node or network node since tempest are mostly run"},{"line_number":145,"context_line":"by calling the API. And in high probability, the DB resource states are"}],"source_content_type":"text/x-rst","patch_set":6,"id":"03a9597d_9e78f1e9","line":142,"updated":"2021-09-09 10:04:15.000000000","message":"actually we workarounded that problem some time ago by checking console output from vm: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/758968 - it helped a lot and seems to be working pretty ok in the CI.","commit_id":"cb68a59600cfe49f8510721af476952ef4d910f0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e5d70450962ff59f82ae77842ab331c88ae48bbb","unresolved":true,"context_lines":[{"line_number":163,"context_line":"  * run ping command or netcat command to detect the network state"},{"line_number":164,"context_line":"  * wait until the VM is reachable or timeout"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"* do next step of the testing"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"Data Model Impact"},{"line_number":169,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"87b7ecc2_c77d3699","line":166,"updated":"2021-09-09 10:04:15.000000000","message":"I\u0027m not really sure it that can really useful for CI cases but I do see some benefit e.g. for support teams which may check if e.g. vm responds on some specific port.\nBut from the other side, the number of use cases which this can help to debug is IMO a bit small. So I\u0027m not sure if it\u0027s worth to put a lot of code in the main tree just to help a bit with debugging connectivity issues.","commit_id":"cb68a59600cfe49f8510721af476952ef4d910f0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e5d70450962ff59f82ae77842ab331c88ae48bbb","unresolved":true,"context_lines":[{"line_number":177,"context_line":"    description \u003d sa.Column(sa.String(db_const.LONG_DESCRIPTION_FIELD_SIZE))"},{"line_number":178,"context_line":"    agent_id \u003d sa.Column(sa.String(36),"},{"line_number":179,"context_line":"                         sa.ForeignKey(\"agents.id\", ondelete\u003d\u0027CASCADE\u0027),"},{"line_number":180,"context_line":"                         nullable\u003dFalse)"},{"line_number":181,"context_line":"    network_id \u003d sa.Column(sa.String(36),"},{"line_number":182,"context_line":"                           sa.ForeignKey(\"networks.id\", ondelete\u003d\u0027CASCADE\u0027),"},{"line_number":183,"context_line":"                           nullable\u003dFalse)"}],"source_content_type":"text/x-rst","patch_set":6,"id":"dc8ed95c_f2a8b7dd","line":180,"updated":"2021-09-09 10:04:15.000000000","message":"did You read about \"Local IP\" RFE: https://review.opendev.org/c/openstack/neutron-specs/+/797798 ? With this You would be maybe able to create only one detector per network and it could be installed on all hosts where ports from that network are.","commit_id":"cb68a59600cfe49f8510721af476952ef4d910f0"}]}