Message ID | 3447131715189836@miiwldkc3tzczqhd.vla.yp-c.yandex.net |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev] docs: Typo. Remove duplicated "to". | expand |
Context | Check | Description |
---|---|---|
ovsrobot/apply-robot | success | apply and check: success |
ovsrobot/github-robot-_Build_and_Test | success | github build: passed |
ovsrobot/github-robot-_ovn-kubernetes | success | github build: passed |
On Wed, May 8, 2024 at 1:38 PM Igor Zhukov <fsb4000@yandex.ru> wrote: > > I came across a new patch recently: https://patchwork.ozlabs.org/project/ovn/patch/20240423095006.24429-1-martin.kalcok@canonical.com/ > After that, I searched the OVN source code for instances of "to to" and discovered two more occurrences of the same typo. Thanks. Applied to main. Numan > > Signed-off-by: Igor Zhukov <fsb4000@yandex.ru> > --- > MAINTAINERS.rst | 2 +- > controller/if-status.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst > index 624aebb9d..f72c8b5ed 100644 > --- a/MAINTAINERS.rst > +++ b/MAINTAINERS.rst > @@ -26,7 +26,7 @@ Committers > ========== > > OVN committers are the people who have been granted access to push > -changes to to the OVN git repository. > +changes to the OVN git repository. > > The responsibilities of an OVN committer are documented here: > |responsibilities|. > diff --git a/controller/if-status.c b/controller/if-status.c > index 6e8aa1f7e..9a7488057 100644 > --- a/controller/if-status.c > +++ b/controller/if-status.c > @@ -582,7 +582,7 @@ if_status_mgr_update(struct if_status_mgr *mgr, > if (!sb_readonly) { > HMAPX_FOR_EACH_SAFE (node, &mgr->ifaces_per_state[OIF_CLAIMED]) { > struct ovs_iface *iface = node->data; > - /* No need to to update pb->chassis as already done > + /* No need to update pb->chassis as already done > * in if_status_handle_claims or if_status_mgr_claim_iface > */ > if (iface->is_vif) { > -- > 2.43.0.windows.1 > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
Great! Thank you for the fast review! Could you also look at https://github.com/ovn-org/ovn-website/pull/81 ? > On Wed, May 8, 2024 at 1:38 PM Igor Zhukov <fsb4000@yandex.ru> wrote: > >> I came across a new patch recently: https://patchwork.ozlabs.org/project/ovn/patch/20240423095006.24429-1-martin.kalcok@canonical.com/ >> After that, I searched the OVN source code for instances of "to to" and discovered two more occurrences of the same typo. > > Thanks. Applied to main. > > Numan > >> Signed-off-by: Igor Zhukov <fsb4000@yandex.ru> >> --- >> MAINTAINERS.rst | 2 +- >> controller/if-status.c | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst >> index 624aebb9d..f72c8b5ed 100644 >> --- a/MAINTAINERS.rst >> +++ b/MAINTAINERS.rst >> @@ -26,7 +26,7 @@ Committers >> ========== >> >> OVN committers are the people who have been granted access to push >> -changes to to the OVN git repository. >> +changes to the OVN git repository. >> >> The responsibilities of an OVN committer are documented here: >> |responsibilities|. >> diff --git a/controller/if-status.c b/controller/if-status.c >> index 6e8aa1f7e..9a7488057 100644 >> --- a/controller/if-status.c >> +++ b/controller/if-status.c >> @@ -582,7 +582,7 @@ if_status_mgr_update(struct if_status_mgr *mgr, >> if (!sb_readonly) { >> HMAPX_FOR_EACH_SAFE (node, &mgr->ifaces_per_state[OIF_CLAIMED]) { >> struct ovs_iface *iface = node->data; >> - /* No need to to update pb->chassis as already done >> + /* No need to update pb->chassis as already done >> * in if_status_handle_claims or if_status_mgr_claim_iface >> */ >> if (iface->is_vif) { >> -- >> 2.43.0.windows.1 >> _______________________________________________ >> dev mailing list >> dev@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff --git a/MAINTAINERS.rst b/MAINTAINERS.rst index 624aebb9d..f72c8b5ed 100644 --- a/MAINTAINERS.rst +++ b/MAINTAINERS.rst @@ -26,7 +26,7 @@ Committers ========== OVN committers are the people who have been granted access to push -changes to to the OVN git repository. +changes to the OVN git repository. The responsibilities of an OVN committer are documented here: |responsibilities|. diff --git a/controller/if-status.c b/controller/if-status.c index 6e8aa1f7e..9a7488057 100644 --- a/controller/if-status.c +++ b/controller/if-status.c @@ -582,7 +582,7 @@ if_status_mgr_update(struct if_status_mgr *mgr, if (!sb_readonly) { HMAPX_FOR_EACH_SAFE (node, &mgr->ifaces_per_state[OIF_CLAIMED]) { struct ovs_iface *iface = node->data; - /* No need to to update pb->chassis as already done + /* No need to update pb->chassis as already done * in if_status_handle_claims or if_status_mgr_claim_iface */ if (iface->is_vif) {
I came across a new patch recently: https://patchwork.ozlabs.org/project/ovn/patch/20240423095006.24429-1-martin.kalcok@canonical.com/ After that, I searched the OVN source code for instances of "to to" and discovered two more occurrences of the same typo. Signed-off-by: Igor Zhukov <fsb4000@yandex.ru> --- MAINTAINERS.rst | 2 +- controller/if-status.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)