)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"5aead0c33cdc4972dc08d9632b1c67bebae427bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f51be9fe_b2a8306c","updated":"2021-12-30 10:56:17.000000000","message":"Change is no longer needed - Neutron moved to accept non-UUID https://review.opendev.org/c/openstack/neutron/+/822328 (plus back ports).\nSeena - please abandon this change.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":33826,"name":"Seena Fallah","email":"seenafallah@gmail.com","username":"clwluvw"},"change_message_id":"f0ede802ea7c6c798c6f5e8df585e3d4c8a1195b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"287b0ca4_8eacfcc9","updated":"2021-11-22 10:17:43.000000000","message":"I think it\u0027s a critical bug for OVN deployment because its break the metadata agent registering!","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"f67364adebd3df20552b52b633be5d7cde316b1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"440932b0_5f3fc6d6","updated":"2021-12-16 08:52:01.000000000","message":"There is ongoing discussion about this case in https://bugs.launchpad.net/kolla-ansible/+bug/1952550. \nDiscrepancies between Neutron/OVS/OVN needs to be solved before applying this change to kolla-ansible.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"b040f9fd627c86968dffd6cc3687a418f7d7cb28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f5fa93d9_9725ccee","updated":"2021-12-04 19:52:44.000000000","message":"We test ovn in CI and some have it in prod so I wonder what is causing your issue. We need a detailed bug report on this. Also CC\u0027ing Michal who may know more.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":33826,"name":"Seena Fallah","email":"seenafallah@gmail.com","username":"clwluvw"},"change_message_id":"a4bad31aedda2bc7c8ea0fa36b26fb2b38f4e137","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c4590abb_39f53fff","in_reply_to":"5cea286d_d89158fd","updated":"2021-12-06 18:33:40.000000000","message":"I still think this is a critical bug and is breaking the current and new deployments and you should review it with higher priority! :)\nCurrent users should plan for a migration because the chassis private names will be changed also!","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":33826,"name":"Seena Fallah","email":"seenafallah@gmail.com","username":"clwluvw"},"change_message_id":"d1ca5dc022ae95b1652d3970bb13a2475cc354e4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5cea286d_d89158fd","in_reply_to":"81526408_dbe07852","updated":"2021-12-06 18:30:59.000000000","message":"As I said because of the wrapper there is no error log and as a source code, I share it should UUID unless it will raise an exception!\nYou can deploy a cluster with OVN and run `openstack network agent list` to see there is no metadata agent in your list!","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9ca994e7663ac1d21906a18af2d64e2f83c11c10","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"993d0a2e_a6529abf","in_reply_to":"88a57c5b_7aa2b1e7","updated":"2022-01-04 19:44:41.000000000","message":"Please check the discussion in https://bugs.launchpad.net/neutron/+bug/1952550 - it\u0027s not better, both UUID and other string are accepted - we\u0027re not going to force people to go through the migration if they don\u0027t need to.\nYou\u0027re free to override the variable to whatever you feel proper in your deployment.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"a62fb9071c8424f60544a1bb0f6c73c15cdc144b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"81526408_dbe07852","in_reply_to":"ae540bc3_167ffeea","updated":"2021-12-06 16:59:13.000000000","message":"I haven\u0027t seen any docs around system-id requirements to be UUID, but it seems Neutron has gone with that assumption, given that ovs-ctl/ovn-ctl are setting that.\nStill - I would like to see a bug report with metadata agent error logs.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"f41f91678b4990a28ebe59cf4ffd10e85020031e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7eaaec69_690113fd","in_reply_to":"c4590abb_39f53fff","updated":"2021-12-07 06:33:17.000000000","message":"Still, there\u0027s no bug - and more politeness please.\nSecond of all - since it\u0027s impacting users - you need to provide a proper reno and ideally an additional task that will mitigate the impact of this change.","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":33826,"name":"Seena Fallah","email":"seenafallah@gmail.com","username":"clwluvw"},"change_message_id":"9f716e7a370ce070a697a789725dacc7a93336ea","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"88a57c5b_7aa2b1e7","in_reply_to":"f51be9fe_b2a8306c","updated":"2021-12-30 10:59:49.000000000","message":"Cool but what about the ovs/ovn side? In their documents, it seems it should/better to be UUID!","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"},{"author":{"_account_id":33826,"name":"Seena Fallah","email":"seenafallah@gmail.com","username":"clwluvw"},"change_message_id":"d7ddc0b5f57a54d01cb285b0acabded7137d9d05","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ae540bc3_167ffeea","in_reply_to":"f5fa93d9_9725ccee","updated":"2021-12-04 21:12:56.000000000","message":"Based on these two facts system-id should be UUID:\n1. In ovs-ctl it says that system-id should be UUID: https://github.com/openvswitch/ovs/blob/master/utilities/ovs-ctl.in#L407-L409\n2. In neutron code here it is trying to parse system-id as a UUID and if it\u0027s not UUID it will be failed and so neutron-ovn-metadata-agent won\u0027t be registered because it raises an exception and based on its wrapper it will always retry without any errors: https://github.com/openstack/neutron/blob/master/neutron/agent/ovn/metadata/agent.py#L266","commit_id":"496dfc3ddea2a044e95eb6c7168e2069bcc39e9c"}]}