)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"948d887091215916e4fcfce49595af4149006f1b","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Network Cascade Deletion Extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I41491e7d85647df94ceffda4ccb1d469ecf1ac4a"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Fix line error, inconsistent naming and constants.py"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib26526650b61062f753a470f5b33b8f53c0a7947"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"8ffe0058_5cf51245","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":52},"updated":"2022-07-14 13:21:43.000000000","message":"Please remove one of the Change-IDs from the commit msg, It can cause hard times for gerrit and for reviewers :-)","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"e12b7445843ffb7c8514a58bdcdbd93e7717539f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Network Cascade Deletion Extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I41491e7d85647df94ceffda4ccb1d469ecf1ac4a"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Fix line error, inconsistent naming and constants.py"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib26526650b61062f753a470f5b33b8f53c0a7947"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"66639ad8_ebe58b63","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":52},"in_reply_to":"8ffe0058_5cf51245","updated":"2022-07-15 14:20:41.000000000","message":"Done","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"948d887091215916e4fcfce49595af4149006f1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1ac4df7d_29651520","updated":"2022-07-14 13:21:43.000000000","message":"Cool, please add a release-note also.\nYou can generate the relesenotes page with tox -ereleasenotes , and in that tox virtualenv (activate it with . .tox/releasenotes/bin/activate, ) you can use the reno package to generate a new one.\n$ reno new my-fancy-new-api-extension","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"c9f9f5e20144e2980e983585da5ff8a0940536fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c45cf89b_d43692aa","in_reply_to":"15e65481_94742140","updated":"2022-07-27 10:43:07.000000000","message":"I don\u0027t remember running sudo with tox at any point. Also, there\u0027s only one user so I don\u0027t understand. Haven\u0027t found any helpful answers on the internet too, but it\u0027s cool, I guess reno is the way. :D","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"cfbd974a114b2a834f39aaa82e0326a8082a625e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7e904ac1_913af041","in_reply_to":"1ac4df7d_29651520","updated":"2022-07-15 15:46:34.000000000","message":"Ack","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"a3503268a9c10919660884d72212507d3c0937aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e7529c89_00da6b24","in_reply_to":"7e904ac1_913af041","updated":"2022-07-21 17:18:25.000000000","message":"I just generated a new one with reno. Couldn\u0027t activate it with tox; got this error: `.tox/pep8/bin/activate: Permission denied`. Any idea why?","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"9cfb9ea35a70ab0da3fbfcc0b6bcab8b29286582","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"15e65481_94742140","in_reply_to":"e7529c89_00da6b24","updated":"2022-07-22 07:56:15.000000000","message":"Sounds like for some reason you need privileges to run the activate file. The reason why this is happening might be that at some point you ran tox with sudo instead of just tox. Maybe checking who owns the activate file could clarify a bit the reason why this is happening","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"03ce2e9ff8fd5357a4a6247757b5711dad03c1e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2e8d5c0a_df6a55f7","updated":"2022-07-19 08:08:13.000000000","message":"functional job fails due to https://bugs.launchpad.net/neutron/+bug/1981963","commit_id":"5dc39ff6845632d70d3575150cfd421d25b59d12"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b839b040ae9b8767569f3e12de1821f5f11ce1a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"16859343_cef6bbc8","updated":"2022-07-21 07:37:26.000000000","message":"please add release notes and api-ref also as I commented previously","commit_id":"5dc39ff6845632d70d3575150cfd421d25b59d12"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"b0f9df42e631355f35461baa3e1e718c113c1b43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d8177632_cf066254","updated":"2022-07-21 07:36:58.000000000","message":"recheck\nhttps://review.opendev.org/c/openstack/requirements/+/850295 to block 0.7.1 pyroute2 is merged","commit_id":"5dc39ff6845632d70d3575150cfd421d25b59d12"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"8f784320368ca370df118657f0c0884c1e8cf073","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8d17541b_b2fb8609","in_reply_to":"d8177632_cf066254","updated":"2022-07-21 17:24:14.000000000","message":"Nice.","commit_id":"5dc39ff6845632d70d3575150cfd421d25b59d12"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2c5c209d2f5fbaeb5c50faba641f697e65eabbaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"43724064_32d209df","updated":"2022-07-22 08:35:15.000000000","message":"Please add unit tests also","commit_id":"f7299b416d29403e11031719ee0e0064ea54d011"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f3d29e8955b6c90924f5b8ba141060e0ab3b11d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e283a1b6_6e5b9d5e","updated":"2022-07-22 12:27:30.000000000","message":"Unit tests examples for extensions: https://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/tests/unit/api/definitions","commit_id":"f7299b416d29403e11031719ee0e0064ea54d011"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7092b315f41c078a35d0d72197ae8ee326ef02b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a4f4ba45_61051dd8","updated":"2022-07-26 13:37:34.000000000","message":"Looks ok","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"379e40c43010c04a09dc5c32f59069fbcc108c89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"2eaa1ba1_aecd393c","updated":"2022-08-05 08:21:56.000000000","message":"\u003e Patch Set 11:\n\u003e \n\u003e This change is ready for review.\n\n\n","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e1273808e62dfc8afdbed9c5b06c0842679e9c34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"38da59e2_2a8449c7","updated":"2022-08-04 08:32:06.000000000","message":"Is it still \"Work In Progress\"? For me it looks like it is good to go now. Please let me know if it is still missing something and we should wait for some more time.","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"195d09919be48ca33c2339870a4cabf972ef68d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"08846faf_121e4db6","updated":"2022-07-29 12:58:29.000000000","message":"Looks, ok","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2b3c92337ef64b4c8b99f060c7cf6e846ec2d737","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"27f8de3b_e6bdeb82","updated":"2022-08-04 10:56:26.000000000","message":"Ping Sharon, can you remove the WIP flag? The patch looks OK and ready to be merged.","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"e789a127fbcffa77d9e1d0bf1e6f080e5bbc0cbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"1ca4e724_f35305b1","updated":"2022-08-05 08:14:26.000000000","message":"Thanks, Sharon! :)","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"379e40c43010c04a09dc5c32f59069fbcc108c89","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"164b5629_13896eaf","in_reply_to":"1ca4e724_f35305b1","updated":"2022-08-05 08:21:56.000000000","message":"No, thank you! 😊\nThank you all for all the help and reviews.","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"1a9a5450f3ff02ae22e3e41a1e0e09d13b56908a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"341cd4c9_cda74a8c","in_reply_to":"27f8de3b_e6bdeb82","updated":"2022-08-05 08:02:25.000000000","message":"Seems like there\u0027s a concensus that the patch is ready. Lifting the WIP flag now. :)","commit_id":"19cb07025b9f4bb74ab4288a19e2651aead0c530"}],"api-ref/source/v2/networks.inc":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"948d887091215916e4fcfce49595af4149006f1b","unresolved":true,"context_lines":[{"line_number":149,"context_line":"the VLAN transparency of the network. If the service does not support VLAN"},{"line_number":150,"context_line":"transparency and a user requests a VLAN transparent network,"},{"line_number":151,"context_line":"the plug-in refuses to create one and returns an appropriate error to the user."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"Show network details"},{"line_number":154,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":155,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"13ccb62e_19e199c4","line":152,"updated":"2022-07-14 13:21:43.000000000","message":"please add here a section about the extension, something like this:\nNetwork Cascade Delete\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nblah, blah.....\n\nYou can build and check it with tox -eapi-ref, and open the rendered html in browser (it will be in neutron-lib/api-ref/build directory)","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"8c1db8505153407fba32180b74e73e85306cb9ee","unresolved":false,"context_lines":[{"line_number":149,"context_line":"the VLAN transparency of the network. If the service does not support VLAN"},{"line_number":150,"context_line":"transparency and a user requests a VLAN transparent network,"},{"line_number":151,"context_line":"the plug-in refuses to create one and returns an appropriate error to the user."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"Show network details"},{"line_number":154,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":155,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"afd795d4_5f754666","line":152,"in_reply_to":"13ccb62e_19e199c4","updated":"2022-07-15 14:21:37.000000000","message":"Ack","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"e0d55b953fbc9122b86888563b7f802f3b60f6a8","unresolved":true,"context_lines":[{"line_number":149,"context_line":"the VLAN transparency of the network. If the service does not support VLAN"},{"line_number":150,"context_line":"transparency and a user requests a VLAN transparent network,"},{"line_number":151,"context_line":"the plug-in refuses to create one and returns an appropriate error to the user."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"Show network details"},{"line_number":154,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":155,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"bf5e3581_79fbfef1","line":152,"in_reply_to":"afd795d4_5f754666","updated":"2022-07-15 15:30:10.000000000","message":"\u003e Ack","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"7b66e64d759b4e4a06c99caf17221b5d288d34b9","unresolved":false,"context_lines":[{"line_number":149,"context_line":"the VLAN transparency of the network. If the service does not support VLAN"},{"line_number":150,"context_line":"transparency and a user requests a VLAN transparent network,"},{"line_number":151,"context_line":"the plug-in refuses to create one and returns an appropriate error to the user."},{"line_number":152,"context_line":""},{"line_number":153,"context_line":"Show network details"},{"line_number":154,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":155,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"9565886d_970255ea","line":152,"in_reply_to":"bf5e3581_79fbfef1","updated":"2022-07-22 16:23:52.000000000","message":"Seems like they should be in alphabetical order.","commit_id":"54c171e796b9d74a72f6f73517cc6e6053b2cf1e"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7092b315f41c078a35d0d72197ae8ee326ef02b9","unresolved":true,"context_lines":[{"line_number":95,"context_line":"Network cascade delete extension"},{"line_number":96,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"The ``network-cascade-delete`` shim extension adds to networks the optional"},{"line_number":99,"context_line":"boolean attribute, ``cascade``, that when defined as ``true``, removes all"},{"line_number":100,"context_line":"child objects of a network upon its deletion."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Port security"},{"line_number":103,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"3085c4aa_e5f94840","line":100,"range":{"start_line":98,"start_character":0,"end_line":100,"end_character":45},"updated":"2022-07-26 13:37:34.000000000","message":"+1","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"350830a7ea2f70c35289dc275ee331588c74e8c2","unresolved":false,"context_lines":[{"line_number":95,"context_line":"Network cascade delete extension"},{"line_number":96,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"The ``network-cascade-delete`` shim extension adds to networks the optional"},{"line_number":99,"context_line":"boolean attribute, ``cascade``, that when defined as ``true``, removes all"},{"line_number":100,"context_line":"child objects of a network upon its deletion."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Port security"},{"line_number":103,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"14e98b2b_abcc679d","line":100,"range":{"start_line":98,"start_character":0,"end_line":100,"end_character":45},"in_reply_to":"3085c4aa_e5f94840","updated":"2022-08-05 08:03:52.000000000","message":"Ack","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"}],"neutron_lib/api/definitions/cascade_delete.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"644ef0a7f3aa38a820c53e9cd3c39193eafe6421","unresolved":true,"context_lines":[{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"ALIAS \u003d \u0027cascade-delete\u0027"},{"line_number":15,"context_line":"IS_SHIM_EXTENSION \u003d True"},{"line_number":16,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":17,"context_line":"NAME \u003d \u0027Network Cascade Deletion Extension\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"e959a254_95d7aedc","line":14,"range":{"start_line":14,"start_character":9,"end_line":14,"end_character":23},"updated":"2022-07-12 15:03:10.000000000","message":"network-cascade-deletion\n\nand the file should have the same name","commit_id":"aaf56cdd62eff1a1e24dd4f009cc8278cf4ede82"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"c0f4d1e0ebbcfc9c43a22a3260772d3d5ad3a75c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"ALIAS \u003d \u0027cascade-delete\u0027"},{"line_number":15,"context_line":"IS_SHIM_EXTENSION \u003d True"},{"line_number":16,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":17,"context_line":"NAME \u003d \u0027Network Cascade Deletion Extension\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"e6e7e211_3003ff1a","line":14,"range":{"start_line":14,"start_character":9,"end_line":14,"end_character":23},"in_reply_to":"e959a254_95d7aedc","updated":"2022-07-14 10:53:15.000000000","message":"Ack","commit_id":"aaf56cdd62eff1a1e24dd4f009cc8278cf4ede82"}],"neutron_lib/constants.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"644ef0a7f3aa38a820c53e9cd3c39193eafe6421","unresolved":true,"context_lines":[{"line_number":331,"context_line":"MIN_GRE_ID \u003d 1"},{"line_number":332,"context_line":"MAX_GRE_ID \u003d 2 ** 32 - 1"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"# For Geneve Tunnel"},{"line_number":335,"context_line":"MIN_GENEVE_VNI \u003d 1"},{"line_number":336,"context_line":"MAX_GENEVE_VNI \u003d 2 ** 24 - 1"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"# For VXLAN Tunnel"},{"line_number":339,"context_line":"MIN_VXLAN_VNI \u003d 1"}],"source_content_type":"text/x-python","patch_set":2,"id":"c2e58e80_f04337de","line":336,"range":{"start_line":334,"start_character":1,"end_line":336,"end_character":28},"updated":"2022-07-12 15:03:10.000000000","message":"unrelated (and unneeded) change","commit_id":"aaf56cdd62eff1a1e24dd4f009cc8278cf4ede82"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"c0f4d1e0ebbcfc9c43a22a3260772d3d5ad3a75c","unresolved":false,"context_lines":[{"line_number":331,"context_line":"MIN_GRE_ID \u003d 1"},{"line_number":332,"context_line":"MAX_GRE_ID \u003d 2 ** 32 - 1"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"# For Geneve Tunnel"},{"line_number":335,"context_line":"MIN_GENEVE_VNI \u003d 1"},{"line_number":336,"context_line":"MAX_GENEVE_VNI \u003d 2 ** 24 - 1"},{"line_number":337,"context_line":""},{"line_number":338,"context_line":"# For VXLAN Tunnel"},{"line_number":339,"context_line":"MIN_VXLAN_VNI \u003d 1"}],"source_content_type":"text/x-python","patch_set":2,"id":"0f1fdf6d_6e91c5c1","line":336,"range":{"start_line":334,"start_character":1,"end_line":336,"end_character":28},"in_reply_to":"c2e58e80_f04337de","updated":"2022-07-14 10:53:15.000000000","message":"Ack.\nI changed the order to get something to test on DevStack a while back, didn\u0027t know the order matters.","commit_id":"aaf56cdd62eff1a1e24dd4f009cc8278cf4ede82"}],"neutron_lib/tests/unit/api/definitions/test_network_cascade_delete.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c2d39b6683a77e099ea76281b0d245ba16133bdc","unresolved":true,"context_lines":[{"line_number":1,"context_line":"from neutron_lib.api.definitions import network_cascade_delete"},{"line_number":2,"context_line":"from neutron_lib.tests.unit.api.definitions import base"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"17aa42d5_43cf6aaf","line":1,"updated":"2022-07-26 14:54:16.000000000","message":"Missing license header, I missed the missing lines this afternoon.\nStrange that pep8 haven\u0027t warned of it","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"691bf6a8621844f039a551ffe734a038cc232d33","unresolved":false,"context_lines":[{"line_number":1,"context_line":"from neutron_lib.api.definitions import network_cascade_delete"},{"line_number":2,"context_line":"from neutron_lib.tests.unit.api.definitions import base"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"93bfaaaf_b37993f7","line":1,"in_reply_to":"087c3ee4_7d1a45e1","updated":"2022-07-27 10:34:07.000000000","message":"Didn\u0027t know that pep8 checked for the license header too. Is it part of the style guide for open-source python files, or is it a custom requirement for Neutron?","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"427de7267dbb16ba03ba1412ae9029387fe5ff7e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"from neutron_lib.api.definitions import network_cascade_delete"},{"line_number":2,"context_line":"from neutron_lib.tests.unit.api.definitions import base"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"087c3ee4_7d1a45e1","line":1,"in_reply_to":"17aa42d5_43cf6aaf","updated":"2022-07-27 09:46:24.000000000","message":"Done","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"3b1b61430b506937b1bd4159f0dc98ac0025f7d7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"from neutron_lib.api.definitions import network_cascade_delete"},{"line_number":2,"context_line":"from neutron_lib.tests.unit.api.definitions import base"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"cd09d6cc_c2a5769a","line":1,"in_reply_to":"93bfaaaf_b37993f7","updated":"2022-07-27 10:36:40.000000000","message":"Also, would it make sense to configure the tox.ini file to require pep8 to check for it?\n\nIf so, I\u0027d be happy to do it.","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"5ddcf26696770636894cf81a7cfc297b325477bb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"from neutron_lib.api.definitions import network_cascade_delete"},{"line_number":2,"context_line":"from neutron_lib.tests.unit.api.definitions import base"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"07729cfd_31dfc19f","line":1,"in_reply_to":"cd09d6cc_c2a5769a","updated":"2022-07-27 13:47:01.000000000","message":"seems the rule is there but not for tests folder....","commit_id":"e399e043a6023788bb2de6d50893076dda5db5bf"}],"releasenotes/notes/add-network-cascade-delete-api-extension-f418e44b37c2b2ed.yaml":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2c5c209d2f5fbaeb5c50faba641f697e65eabbaf","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add API extension ``network-cascade-deletion`` that will allow users"},{"line_number":5,"context_line":"    to automatically delete associated ports and subports when"},{"line_number":6,"context_line":"    deleting networks."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"000ef271_01daa147","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":57},"updated":"2022-07-22 08:35:15.000000000","message":"subnets?","commit_id":"f7299b416d29403e11031719ee0e0064ea54d011"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"8cf69fb014074b6d1456cb4b48ee515e4aef642f","unresolved":true,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add API extension ``network-cascade-deletion`` that will allow users"},{"line_number":5,"context_line":"    to automatically delete associated ports and subports when"},{"line_number":6,"context_line":"    deleting networks."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"de6dd9c5_fe8789f9","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":57},"in_reply_to":"000ef271_01daa147","updated":"2022-07-22 14:03:41.000000000","message":"No, subnets are removed when the network is deleted. I meant like ports attached to trunks, but I guess just \"ports\" covers it. Ports and routers, maybe?","commit_id":"f7299b416d29403e11031719ee0e0064ea54d011"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"d40dd32e7ac5d08cdbc337cd53082a53eb1328de","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add API extension ``network-cascade-deletion`` that will allow users"},{"line_number":5,"context_line":"    to automatically delete associated ports and subports when"},{"line_number":6,"context_line":"    deleting networks."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"751b1e46_f2a35bdf","line":5,"range":{"start_line":5,"start_character":49,"end_line":5,"end_character":57},"in_reply_to":"de6dd9c5_fe8789f9","updated":"2022-07-25 17:12:02.000000000","message":"\u003e No, subnets are removed when the network is deleted. I meant like ports attached to trunks, but I guess just \"ports\" covers it. Ports and routers, maybe?","commit_id":"f7299b416d29403e11031719ee0e0064ea54d011"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a4bb59599c74cd050f92a58f20074a0093a70f7f","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add API extension ``network-cascade-delete`` that will allow users"},{"line_number":5,"context_line":"    to automatically delete all associated ports when"},{"line_number":6,"context_line":"    deleting networks."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"f5c301c6_2a4ce42f","line":4,"range":{"start_line":4,"start_character":54,"end_line":4,"end_character":64},"updated":"2022-07-27 13:42:49.000000000","message":"I think it should be \"allows\"","commit_id":"8eda92f821c43def3e2fa3167912462c89f6eb94"},{"author":{"_account_id":34728,"name":"Sharon Koech","display_name":"skoech","email":"sharon.koech@canonical.com","username":"skoech"},"change_message_id":"ef5f3ba4f6d7706a0e449286c0ae1e7ad342cb49","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add API extension ``network-cascade-delete`` that will allow users"},{"line_number":5,"context_line":"    to automatically delete all associated ports when"},{"line_number":6,"context_line":"    deleting networks."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"78267e3d_e6addf5a","line":4,"range":{"start_line":4,"start_character":54,"end_line":4,"end_character":64},"in_reply_to":"f5c301c6_2a4ce42f","updated":"2022-07-28 14:49:29.000000000","message":"Makes sense. :)","commit_id":"8eda92f821c43def3e2fa3167912462c89f6eb94"}]}