Message ID | 20240410132625.1067214-1-amusil@redhat.com |
---|---|
State | Changes Requested |
Headers | show |
Series | [ovs-dev] ci: Rename OvS master branch to main. | 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, Apr 10, 2024 at 03:26:25PM +0200, Ales Musil wrote: > There was recent switch in OvS from master to main branch. > Use main instead in ovn-fake-mutlinode tests. > > Signed-off-by: Ales Musil <amusil@redhat.com> Acked-by: Simon Horman <horms@ovn.org>
Hi Ales, Simon, On 4/10/24 15:39, Simon Horman wrote: > On Wed, Apr 10, 2024 at 03:26:25PM +0200, Ales Musil wrote: >> There was recent switch in OvS from master to main branch. >> Use main instead in ovn-fake-mutlinode tests. Nit: typo :) >> >> Signed-off-by: Ales Musil <amusil@redhat.com> > > Acked-by: Simon Horman <horms@ovn.org> > There are two more references to OVS "master" branch, in ovn-sandbox.rst and in README.rst. Regards, Dumitru
On Wed, Apr 10, 2024 at 3:42 PM Dumitru Ceara <dceara@redhat.com> wrote: > Hi Ales, Simon, > > On 4/10/24 15:39, Simon Horman wrote: > > On Wed, Apr 10, 2024 at 03:26:25PM +0200, Ales Musil wrote: > >> There was recent switch in OvS from master to main branch. > >> Use main instead in ovn-fake-mutlinode tests. > > Nit: typo :) > > >> > >> Signed-off-by: Ales Musil <amusil@redhat.com> > > > > Acked-by: Simon Horman <horms@ovn.org> > > > > There are two more references to OVS "master" branch, in ovn-sandbox.rst > and in README.rst. > > Regards, > Dumitru > > You are right, posted v2. Thanks, Ales
diff --git a/.github/workflows/ovn-fake-multinode-tests.yml b/.github/workflows/ovn-fake-multinode-tests.yml index 79b6c4253..ed4d6c4ef 100644 --- a/.github/workflows/ovn-fake-multinode-tests.yml +++ b/.github/workflows/ovn-fake-multinode-tests.yml @@ -35,12 +35,12 @@ jobs: # Check out ovn and ovs separately inside ovn-fake-multinode/ovn and ovn-fake-multinode/ovs # ovn-fake-multinode builds and installs ovs from ovn-fake-multinode/ovs # and it builds and installs ovn from ovn-fake-multinode/ovn. It uses the ovs submodule for ovn compilation. - - name: Check out ovs master + - name: Check out ovs main uses: actions/checkout@v4 with: path: 'ovn-fake-multinode/ovs' repository: 'openvswitch/ovs' - ref: 'master' + ref: 'main' - name: Check out ovn ${{ matrix.cfg.branch }} uses: actions/checkout@v4
There was recent switch in OvS from master to main branch. Use main instead in ovn-fake-mutlinode tests. Signed-off-by: Ales Musil <amusil@redhat.com> --- .github/workflows/ovn-fake-multinode-tests.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)