<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">Hey Radomir<div><br></div><div>My replies keep getting blocked because they’re too big. I’ll summarise and then you will all get spammed if someone decides to approve my other replies. In our case, I had already tried that patch and it doesn’t resolve this particular issue. The problem I see is that we need to compare the fields in our params with what already exists on the port. Then not send fields that are unchanged. This diff resolved the issue we’re discussing here:</div><div><br></div><div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">❯ diff horizon/openstack_dashboard/<wbr>dashboards/project/networks/<wbr>ports/workflows.py workflows.py</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">411c411</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">< params = self._construct_parameters(<wbr>data)</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">---</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> params = self._construct_parameters(<wbr>request, data)</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">420c420</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">< def _construct_parameters(self, data):</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">---</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> def _construct_parameters(self, request, data):</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">424a425,426</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> LOG.info("bshephar - Data variable = %s", data)</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> initial = api.neutron.port_get(request, self.context['port_id'])</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">431c433,434</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">< if data['port_security_enabled'] is not None:</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">---</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> if (data['port_security_enabled'] is not None and data['port_security_enabled']</font></div><div style="caret-color: rgb(80, 0, 80); color: rgb(80, 0, 80); font-family: Arial, Helvetica, sans-serif;"><font face="Menlo">> != initial['port_security_<wbr>enabled']):</font></div></div><div><br><div><br></div><div>This probably isn’t a solution you guys would want to merge. But it demonstrates a working solution to the problem that we’re discussing.</div><div><br></div><div>
Brendan Shephard<br>Senior Software Engineer<br>Brisbane, Australia<br><br><br>
</div>
<div><br></div></div></body></html>