)]}'
{"specs/train/atomic-extraroute.rst":[{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"88452a29a5b0934a190a4d6e6634f2eaf56a8c23","unresolved":false,"context_lines":[{"line_number":27,"context_line":"API the routes attribute of a router (containing all routing table"},{"line_number":28,"context_line":"entries) must be updated at once. Therefore additions and deletions"},{"line_number":29,"context_line":"must be performed on the client side. Therefore multiple clients race"},{"line_number":30,"context_line":"to update the routes attribute and updates may get lost."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Proposed change"},{"line_number":33,"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":"dfbec78f_5a9e1308","line":30,"updated":"2019-05-03 16:32:00.000000000","message":"From discussion at the PTG: another problem is that routes are identified only by their data (destination/nexthop/router id); there\u0027s no way to uniquely identify the source of a route. So if two things add the same route then if one is deleted then the other will be broken (or alternatively, we could fail if two Heat stacks try to create the same route, but that\u0027s also bad and doesn\u0027t prevent other tools from doing the same).","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"82a68867b5bb7f1f0e1a6655bea223c9f62d984c","unresolved":false,"context_lines":[{"line_number":27,"context_line":"API the routes attribute of a router (containing all routing table"},{"line_number":28,"context_line":"entries) must be updated at once. Therefore additions and deletions"},{"line_number":29,"context_line":"must be performed on the client side. Therefore multiple clients race"},{"line_number":30,"context_line":"to update the routes attribute and updates may get lost."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Proposed change"},{"line_number":33,"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":"dfbec78f_a50da291","line":30,"in_reply_to":"dfbec78f_5a9e1308","updated":"2019-05-04 19:40:45.000000000","message":"I did raise this problem in the Neutron PTG. And added a proposal addressing this to the Neutron spec.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"82a68867b5bb7f1f0e1a6655bea223c9f62d984c","unresolved":false,"context_lines":[{"line_number":33,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"Change OS::Neutron::ExtraRoute to use the new Neutron API (extension"},{"line_number":36,"context_line":"extraroute-atomic) as it is specified in the `Atomic Extraroute API"},{"line_number":37,"context_line":"Neutron RFE \u003chttps://bugs.launchpad.net/neutron/+bug/1826396\u003e`_."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Keep the resource template format as it is today, that is:"}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_2521b2fc","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":17},"updated":"2019-05-04 19:40:45.000000000","message":"This changed in the new proposal to extraroute-firstclass.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"88452a29a5b0934a190a4d6e6634f2eaf56a8c23","unresolved":false,"context_lines":[{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"We could introduce a OS::Neutron::ExtraRoutes resource (please note the"},{"line_number":58,"context_line":"plural). Internally the the resource implementation would still use the"},{"line_number":59,"context_line":"improved Neutron extraroute-atomic API. But the template syntax could allow"},{"line_number":60,"context_line":"multiple routes in one resource;"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_7aaf5794","line":58,"range":{"start_line":57,"start_character":56,"end_line":58,"end_character":6},"updated":"2019-05-03 16:32:00.000000000","message":"Experience with SoftwareDeployments is that this leads to massive confusion where you always have to clarify whether you\u0027re talking about a series of ExtraRoute resources vs. a single ExtraRoutes resource. Something like ExtraRouteList might be better.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2a40c1158d0d267afabfc427b1df9b0525337d32","unresolved":false,"context_lines":[{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"We could introduce a OS::Neutron::ExtraRoutes resource (please note the"},{"line_number":58,"context_line":"plural). Internally the the resource implementation would still use the"},{"line_number":59,"context_line":"improved Neutron extraroute-atomic API. But the template syntax could allow"},{"line_number":60,"context_line":"multiple routes in one resource;"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_75fae142","line":58,"range":{"start_line":58,"start_character":24,"end_line":58,"end_character":27},"updated":"2019-05-07 07:40:57.000000000","message":"nit: extra \"the\" (not extraroute :-)))","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"82a68867b5bb7f1f0e1a6655bea223c9f62d984c","unresolved":false,"context_lines":[{"line_number":54,"context_line":"Alternatives"},{"line_number":55,"context_line":"------------"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"We could introduce a OS::Neutron::ExtraRoutes resource (please note the"},{"line_number":58,"context_line":"plural). Internally the the resource implementation would still use the"},{"line_number":59,"context_line":"improved Neutron extraroute-atomic API. But the template syntax could allow"},{"line_number":60,"context_line":"multiple routes in one resource;"},{"line_number":61,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_c560d635","line":58,"range":{"start_line":57,"start_character":56,"end_line":58,"end_character":6},"in_reply_to":"dfbec78f_7aaf5794","updated":"2019-05-04 19:40:45.000000000","message":"I think I will change this to OS::Neutron::ExtraRouteSet.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"88452a29a5b0934a190a4d6e6634f2eaf56a8c23","unresolved":false,"context_lines":[{"line_number":57,"context_line":"We could introduce a OS::Neutron::ExtraRoutes resource (please note the"},{"line_number":58,"context_line":"plural). Internally the the resource implementation would still use the"},{"line_number":59,"context_line":"improved Neutron extraroute-atomic API. But the template syntax could allow"},{"line_number":60,"context_line":"multiple routes in one resource;"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"::"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_fa55871e","line":60,"updated":"2019-05-03 16:32:00.000000000","message":"This is probably a good idea, because it allows users to use e.g. the repeat function or just passing data from parameters to create a variable number of routes.\n\nHowever, it\u0027s very difficult for Heat to store a *list* of UUIDs as the physical ID of a resource. So if we\u0027re assigning a UUID to deal with the idempotency problem identified above, then we can really only do this if we have a single UUID for the group.\n\nIn other words, we can only do it this way if the Neutron API also supports a *group* of extra routes as the resource identified by a UUID.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"},{"author":{"_account_id":15554,"name":"Bence Romsics","email":"bence.romsics@gmail.com","username":"ebenrom","status":"working for Ericsson, UTC+1 (+DST)"},"change_message_id":"82a68867b5bb7f1f0e1a6655bea223c9f62d984c","unresolved":false,"context_lines":[{"line_number":57,"context_line":"We could introduce a OS::Neutron::ExtraRoutes resource (please note the"},{"line_number":58,"context_line":"plural). Internally the the resource implementation would still use the"},{"line_number":59,"context_line":"improved Neutron extraroute-atomic API. But the template syntax could allow"},{"line_number":60,"context_line":"multiple routes in one resource;"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"::"},{"line_number":63,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"dfbec78f_0567ce4f","line":60,"in_reply_to":"dfbec78f_fa55871e","updated":"2019-05-04 19:40:45.000000000","message":"In order to keep the neutron api relatively simple I did remove the option of multi-update in ps5 of the neutron spec. I\u0027ll come back to the question of ExtraRoute or ExtraRouteSet when the neutron api is finalized.","commit_id":"6b3f2ff94df327038a6a3323db787c08c40cbfa5"}]}