)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"cca0fe662783da108e1869921c45ff4f884afd4e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"A change[1] injected a direct route to metadata ip through a subnet\u0027s"},{"line_number":10,"context_line":"default gateway to avoid Windows mac resolution. Such route is incorrect"},{"line_number":11,"context_line":"when metadata proxy service is disabled on l3 agents and there is no way"},{"line_number":12,"context_line":"to overload it because it\u0027s a route to a /32 subnet."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"More precisely the change[1] breaks the following usecase (separating"},{"line_number":15,"context_line":"admin/management traffic and client/user traffic):"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a4dcd0f_90f11d98","line":12,"updated":"2015-08-15 01:17:28.000000000","message":"Too much details. Please, move all these details to the bug description","commit_id":"043142f52ccfee4ed2bb90b27be53c6850e2d52a"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"cca0fe662783da108e1869921c45ff4f884afd4e","unresolved":false,"context_lines":[{"line_number":28,"context_line":"to use the correct route!"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This change propose to replace current metadata proxy to router:"},{"line_number":31,"context_line":" destination\u003d169.254.169.254/32,nexthop\u003drouter-ip"},{"line_number":32,"context_line":"by:"},{"line_number":33,"context_line":" destination\u003d169.254.169.254/31,nexthop\u003drouter-ip"},{"line_number":34,"context_line":"And VMs will use dhcp metadata proxy service when available[2] and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a4dcd0f_702d71fc","line":31,"updated":"2015-08-15 01:17:28.000000000","message":"I am not sure this is a good change for the project. A lot of libraries and plugins may be affected because they may work under the initial assumption.  Anyway, let\u0027s start by fixing the commit message","commit_id":"043142f52ccfee4ed2bb90b27be53c6850e2d52a"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"c54bb6cb990b0a96ecbf8c9efdf8187713f59126","unresolved":false,"context_lines":[{"line_number":28,"context_line":"to use the correct route!"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"This change propose to replace current metadata proxy to router:"},{"line_number":31,"context_line":" destination\u003d169.254.169.254/32,nexthop\u003drouter-ip"},{"line_number":32,"context_line":"by:"},{"line_number":33,"context_line":" destination\u003d169.254.169.254/31,nexthop\u003drouter-ip"},{"line_number":34,"context_line":"And VMs will use dhcp metadata proxy service when available[2] and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a4dcd0f_306fe9fd","line":31,"in_reply_to":"1a4dcd0f_702d71fc","updated":"2015-08-15 01:37:49.000000000","message":"Initial assumption? Before [1] merge, VMs use dhcp metadata proxies when enabled and otherwise router metadata proxies so this change transforms [1] feature into a backward compatible feature.","commit_id":"043142f52ccfee4ed2bb90b27be53c6850e2d52a"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"c54bb6cb990b0a96ecbf8c9efdf8187713f59126","unresolved":false,"context_lines":[{"line_number":34,"context_line":"And VMs will use dhcp metadata proxy service when available[2] and"},{"line_number":35,"context_line":"router one when available[3]."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Using different masklens avoid to define yet another complicated"},{"line_number":38,"context_line":"configuration option enabling/disabling previous change[1] feature."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"[1] Ic991f9d7f9a8cf942e3e29f2da03d73c0f26470a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a4dcd0f_29d34461","line":37,"updated":"2015-08-15 01:37:49.000000000","message":"I am not sure which approach is the better one ... So i am waiting for feedbacks :)","commit_id":"043142f52ccfee4ed2bb90b27be53c6850e2d52a"}]}