)]}'
{"vif_plug_ovs/ovsdb/impl_idl.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b4ee7592138cf0f9959cc81571016424effd9a1a","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# os-vif does not use the connection to monitor the state of the ovs-db"},{"line_number":32,"context_line":"# it only uses it to write to the ovs-db. As a result we do not need to"},{"line_number":33,"context_line":"# actually poll ovs so mock out the Poller used by ovsdbapp."},{"line_number":34,"context_line":"# TODO(sean-k-mooney): modify ovsdbapp to allow us to disable polling"},{"line_number":35,"context_line":"# or kick the polling to a real pthread instead of this hack."},{"line_number":36,"context_line":"@mock.patch(\"ovs.poller.Poller\", new\u003dmock.MagicMock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"74d3451a_8371d5a7","line":33,"range":{"start_line":33,"start_character":36,"end_line":33,"end_character":42},"updated":"2021-06-10 12:57:36.000000000","message":"But this is also used to execute transactions [1]. Without that, you cannot send/receive anything to the DB\n\nAm I missing something here?\n\n[1]https://github.com/openstack/ovsdbapp/blob/5d24facd36ca06b1f5f793c834da2efa4fa3a508/ovsdbapp/backend/ovs_idl/connection.py#L95-L137","commit_id":"721d775988cd6109c55ace55f0d489d656858058"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bee6b93c9ba5aadf4c7ac911f3fd7fe67b1693d3","unresolved":true,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# os-vif does not use the connection to monitor the state of the ovs-db"},{"line_number":32,"context_line":"# it only uses it to write to the ovs-db. As a result we do not need to"},{"line_number":33,"context_line":"# actually poll ovs so mock out the Poller used by ovsdbapp."},{"line_number":34,"context_line":"# TODO(sean-k-mooney): modify ovsdbapp to allow us to disable polling"},{"line_number":35,"context_line":"# or kick the polling to a real pthread instead of this hack."},{"line_number":36,"context_line":"@mock.patch(\"ovs.poller.Poller\", new\u003dmock.MagicMock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"22a63ee0_10adbb4e","line":33,"range":{"start_line":33,"start_character":36,"end_line":33,"end_character":42},"in_reply_to":"74d3451a_8371d5a7","updated":"2021-06-10 13:12:20.000000000","message":"Talked via IRC: Sean is right, this poller is only used by Connection to retrieve the events from the DB. That will populate the local DB cache","commit_id":"721d775988cd6109c55ace55f0d489d656858058"}]}