)]}'
{"ovsdbapp/schema/ovn_southbound/api.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"69865ec2c66f0877d31fd10a4ea665fe27bb1815","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @abc.abstractmethod"},{"line_number":27,"context_line":"    def chassis_add(self, chassis, encap_types, encap_ip, hostname\u003d\u0027\u0027,"},{"line_number":28,"context_line":"                    may_exist\u003dFalse, **columns):"},{"line_number":29,"context_line":"        \"\"\"Creates  a  new chassis"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ea700bb5_1fc3408b","line":27,"updated":"2026-09-11 12:02:42.000000000","message":"The hostname should be added after may_exist. This can break calls like\n`chassis_add(\u0027chassis\u0027, \u0027encap_types\u0027, \u0027encap_up\u0027, True)` where the last parameter is now added to the hostname.","commit_id":"3caa0b5fce67cb0c69dcdbd1c69827567630282c"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3f72ee4f2f8bd6bf5aafa37a7cff74cf9c5426ac","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    \"\"\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @abc.abstractmethod"},{"line_number":27,"context_line":"    def chassis_add(self, chassis, encap_types, encap_ip, hostname\u003d\u0027\u0027,"},{"line_number":28,"context_line":"                    may_exist\u003dFalse, **columns):"},{"line_number":29,"context_line":"        \"\"\"Creates  a  new chassis"},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"cdaf6390_7d28660a","line":27,"in_reply_to":"ea700bb5_1fc3408b","updated":"2026-09-11 13:03:46.000000000","message":"I had this discussion with myself too:\n* We usually add `may_exist` as the last kwarg in the API methods\n* Because it is a kwarg, it should be called with the variable name, not as a positional argument.\n\nRight now, in OpenStack, this is being used only in the Neutron functional tests. I also took a look in github and I only found this method being called in Neutron.","commit_id":"3caa0b5fce67cb0c69dcdbd1c69827567630282c"}]}