Fix missing parentid on rule delete journal record
This fixes an error raised when deleting a security group if there are
pending rules to be deleted of that group as records in the journal
that were not triggered by the group deletion operation itself so that
they did not have the parent security group id as data of the record.
This happens when the rule and the group are deleted independently but
in rapid succession. It does not happen when the rules are deleted as a
consequence of deleting the group since the parentid is correctly set
in that case, neither when the group is deleted some time after the
rules when their records might have already been processed and removed.
This problem was encountered in OPNFV functest environment but may also
fix [1] and complement [2].
Reviewed: https:/ /review. openstack. org/538352 /git.openstack. org/cgit/ openstack/ networking- odl/commit/ ?id=684c46437db eb97bb71dde248d 39db4dbebcd65e
Committed: https:/
Submitter: Zuul
Branch: stable/pike
commit 684c46437dbeb97 bb71dde248d39db 4dbebcd65e
Author: Jaime Caamaño Ruiz <email address hidden>
Date: Tue Jan 23 19:44:12 2018 +0100
Fix missing parentid on rule delete journal record
This fixes an error raised when deleting a security group if there are
pending rules to be deleted of that group as records in the journal
that were not triggered by the group deletion operation itself so that
they did not have the parent security group id as data of the record.
This happens when the rule and the group are deleted independently but
in rapid succession. It does not happen when the rules are deleted as a
consequence of deleting the group since the parentid is correctly set
in that case, neither when the group is deleted some time after the
rules when their records might have already been processed and removed.
This problem was encountered in OPNFV functest environment but may also
fix [1] and complement [2].
[1] https:/ /jira.opendayli ght.org/ browse/ NETVIRT- 896 /review. openstack. org/#/c/ 536935
[2] https:/
Change-Id: I5ccdbdea9d8033 34c8ae9416b40e3 c4b48a8144b e363a15c53ac44d edfae7e0ce)
Closes-Bug: 1744966
Signed-off-by: Jaime Caamaño Ruiz <email address hidden>
(cherry picked from commit 2c15a7926dfd24e