mbox series

[ovs-dev,0/2] ovn-ctl: stop databases with stop_ovn_daemon()

Message ID fe103d8cbfeebe899fee66ed677edd5bdc0b324a.camel@redhat.com
Headers show
Series ovn-ctl: stop databases with stop_ovn_daemon() | expand

Message

Dan Williams April 12, 2021, 2:58 p.m. UTC 
Wait for the databases to actually exit before returning to
the caller of ovn-ctl.

v1 -> v2
-----
  * No code changes
  * Fixed stop_ovn_daemon() name in patch headers
  * Added "ovn" to patch subjects for correct patchwork

Dan Williams (2):
  ovn-lib: harmonize stop_ovn_daemon() with ovs-lib
  ovn-ctl: stop databases with stop_ovn_daemon()

 utilities/ovn-ctl    | 16 +++++-----------
 utilities/ovn-lib.in | 40 ++++++++++++++++++++++++++++++++++------
 2 files changed, 39 insertions(+), 17 deletions(-)

Comments

Mark Michelson April 20, 2021, 5:05 p.m. UTC | #1 
Hi Dan,

Acked-by: Mark Michelson <mmichels@redhat.com>

for both patches in the series.

On 4/12/21 10:58 AM, Dan Williams wrote:
> Wait for the databases to actually exit before returning to
> the caller of ovn-ctl.
> 
> v1 -> v2
> -----
>    * No code changes
>    * Fixed stop_ovn_daemon() name in patch headers
>    * Added "ovn" to patch subjects for correct patchwork
> 
> Dan Williams (2):
>    ovn-lib: harmonize stop_ovn_daemon() with ovs-lib
>    ovn-ctl: stop databases with stop_ovn_daemon()
> 
>   utilities/ovn-ctl    | 16 +++++-----------
>   utilities/ovn-lib.in | 40 ++++++++++++++++++++++++++++++++++------
>   2 files changed, 39 insertions(+), 17 deletions(-)
>
Numan Siddique April 21, 2021, 10:12 a.m. UTC | #2 
On Tue, Apr 20, 2021 at 1:06 PM Mark Michelson <mmichels@redhat.com> wrote:
>
> Hi Dan,
>
> Acked-by: Mark Michelson <mmichels@redhat.com>
>
> for both patches in the series.

Thanks Dan (and Mark).  I applied the patches to the main branch.

Numan

>
> On 4/12/21 10:58 AM, Dan Williams wrote:
> > Wait for the databases to actually exit before returning to
> > the caller of ovn-ctl.
> >
> > v1 -> v2
> > -----
> >    * No code changes
> >    * Fixed stop_ovn_daemon() name in patch headers
> >    * Added "ovn" to patch subjects for correct patchwork
> >
> > Dan Williams (2):
> >    ovn-lib: harmonize stop_ovn_daemon() with ovs-lib
> >    ovn-ctl: stop databases with stop_ovn_daemon()
> >
> >   utilities/ovn-ctl    | 16 +++++-----------
> >   utilities/ovn-lib.in | 40 ++++++++++++++++++++++++++++++++++------
> >   2 files changed, 39 insertions(+), 17 deletions(-)
> >
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>