)]}'
{"specs/kilo/ml2-host-ip-port-binding.rst":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":20,"context_line":"the node where the port should be created in the *host_id* field. The host name"},{"line_number":21,"context_line":"may not always be a fully qualified domain name, and as such it may not always"},{"line_number":22,"context_line":"be possible to determine the IP address of the host (i.e., the mapping is only"},{"line_number":23,"context_line":"in the ``/etc/hosts`` file of the control node). This information may be"},{"line_number":24,"context_line":"necessary in some cases, one such example being the LISP Flow Mapping service"},{"line_number":25,"context_line":"within the OpenDaylight controller."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_fbc79262","line":23,"updated":"2014-11-10 10:13:21.000000000","message":"I would argue that you are trying to work around a deficiency in the ODL controller by providing some information the controller should be able to know through neutron. I would think that ODL knows on which host a port is, especially since this ports should have shown up as an interface on the host virtual bridge.\n\nHowever, given my ignorance with the LISP service for ODL I need more information to comment on this, hence my next comment.","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":22,"context_line":"be possible to determine the IP address of the host (i.e., the mapping is only"},{"line_number":23,"context_line":"in the ``/etc/hosts`` file of the control node). This information may be"},{"line_number":24,"context_line":"necessary in some cases, one such example being the LISP Flow Mapping service"},{"line_number":25,"context_line":"within the OpenDaylight controller."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Proposed Change"},{"line_number":28,"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":1,"id":"5a890539_fb195223","line":25,"updated":"2014-11-10 10:13:21.000000000","message":"reference needed, please","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":37,"context_line":"REST API Impact"},{"line_number":38,"context_line":"---------------"},{"line_number":39,"context_line":"None on the API exposed by Neutron. The OpenDaylight Neutron API should be able"},{"line_number":40,"context_line":"to parse and interpret the new field."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Security Impact"},{"line_number":43,"context_line":"---------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_3b8f5a25","line":40,"updated":"2014-11-10 10:13:21.000000000","message":"The purpose of this section is to describe how you change the rest API, not to reassure users that the API should be able to handle this field.\n\nHow does this new attribute change the port binding data structure? Note that for some reason this extensions has \"input\" and \"output\" attributes. This should be specified as well in your specification.\n\nAlso, what will neutron gain by having this new attribute?","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Alternatives"},{"line_number":71,"context_line":"------------"},{"line_number":72,"context_line":"None."},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Implementation"},{"line_number":75,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_9bb5aecf","line":72,"updated":"2014-11-10 10:13:21.000000000","message":"I was thinking that perhaps one way would be to ensure that deployments which need to leverage this LISP service should ensure FQDN for all hosts are always registered?\n\nWhy isn\u0027t that a valid alternative?","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Implementation"},{"line_number":75,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":76,"context_line":"The host name of the node is part of the ``neutron/extensions/portbindings.py`` extension (*binding:host_id*). A new field, *binding:host_ip* will be added to this extension. ``neutron/plugins/ml2/plugin.py`` will be updated to do DNS resolution when processing port bindings, and store the result in the newly added field. A configuration variable could be added to activate/deactivate this behavior."},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"Assignee(s)"},{"line_number":79,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_5be266d6","line":76,"updated":"2014-11-10 10:13:21.000000000","message":"adding some \\n characters might help rendering of this section","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"e1f5b6d97d333a9f375c155ee078e62d8dedd227","unresolved":false,"context_lines":[{"line_number":82,"context_line":"Work Items"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":"- Add new field to ``neutron/extensions/portbindings.py``"},{"line_number":85,"context_line":"- Add DNS resolution to ``neutron/plugins/ml2/plugin.py``"},{"line_number":86,"context_line":"- Update code working with port binding to take into account the new field"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_423c4094","line":85,"updated":"2014-11-08 22:28:03.000000000","message":"How do you expect this to work?","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":82,"context_line":"Work Items"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":"- Add new field to ``neutron/extensions/portbindings.py``"},{"line_number":85,"context_line":"- Add DNS resolution to ``neutron/plugins/ml2/plugin.py``"},{"line_number":86,"context_line":"- Update code working with port binding to take into account the new field"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_1bd8de1c","line":85,"in_reply_to":"5a890539_423c4094","updated":"2014-11-10 10:13:21.000000000","message":"I am shivering. Especially since the whole point of the blueprint appeared to not do DNS resolution since the host might not send a FQDN that can be resolved.","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"e1f5b6d97d333a9f375c155ee078e62d8dedd227","unresolved":false,"context_lines":[{"line_number":90,"context_line":"None."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Testing"},{"line_number":93,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Tempest Tests"},{"line_number":96,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_62413c0b","line":93,"updated":"2014-11-08 22:28:03.000000000","message":"the work items above indicate there should be some test changes too","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"b87dd52597374b9210e76960e5205289e9196fa1","unresolved":false,"context_lines":[{"line_number":90,"context_line":"None."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"Testing"},{"line_number":93,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Tempest Tests"},{"line_number":96,"context_line":"-------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"5a890539_5b098694","line":93,"in_reply_to":"5a890539_62413c0b","updated":"2014-11-10 10:13:21.000000000","message":"I am glad to know there is no need to test this code.","commit_id":"03c8630b4997c666104186a6e78fd747b26e3cfe"}]}