Re: [ovs-dev] [PATCH ovn v2] Add weekly CI job that uses OVS master.
On 2/4/21 2:01 PM, Dumitru Ceara wrote: On 1/28/21 9:35 PM, Mark Michelson wrote: Signed-off-by: Mark Michelson --- This patch is based on the "Include OVS as a git submodule." patch, which, at this time has not been merged into OVN master yet. --- .github/workflows/test.yml | 37 - 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 916b14da2..f3a53a8b6 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,11 @@ name: Build and Test -on: [push, pull_request] +on: + push: + pull_request: + schedule: + # Run Sunday at midnight + - cron: '0 0 * * 0' jobs: build-linux: @@ -49,10 +54,26 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive + # For weekly runs, don't update submodules + - name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + + # Weekly runs test using OVS master instead of the + # submodule. + - name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: + repository: 'openvswitch/ovs' + path: 'ovs' + ref: 'master' + - name: update APT cache run: sudo apt update @@ -105,9 +126,23 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive + # For weekly runs, don't update submodules + - name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + # Weekly runs test using OVS master instead of the + # submodule. + - name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: + repository: 'openvswitch/ovs' + path: 'ovs' + ref: 'master' - name: install dependencies run: brew install automake libtool - name: prepare Hi Mark, I didn't try out the schedule part but the change looks good to me. Github CI on "push" passes fine. Acked-by: Dumitru Ceara Thanks, Dumitru I have merged this to master. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] [PATCH ovn v2] Add weekly CI job that uses OVS master.
On 1/28/21 9:35 PM, Mark Michelson wrote: Signed-off-by: Mark Michelson --- This patch is based on the "Include OVS as a git submodule." patch, which, at this time has not been merged into OVN master yet. --- .github/workflows/test.yml | 37 - 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 916b14da2..f3a53a8b6 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,11 @@ name: Build and Test -on: [push, pull_request] +on: + push: + pull_request: + schedule: +# Run Sunday at midnight +- cron: '0 0 * * 0' jobs: build-linux: @@ -49,10 +54,26 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive +# For weekly runs, don't update submodules +- name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + +# Weekly runs test using OVS master instead of the +# submodule. +- name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: +repository: 'openvswitch/ovs' +path: 'ovs' +ref: 'master' + - name: update APT cache run: sudo apt update @@ -105,9 +126,23 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive +# For weekly runs, don't update submodules +- name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 +# Weekly runs test using OVS master instead of the +# submodule. +- name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: +repository: 'openvswitch/ovs' +path: 'ovs' +ref: 'master' - name: install dependencies run: brew install automake libtool - name: prepare Hi Mark, I didn't try out the schedule part but the change looks good to me. Github CI on "push" passes fine. Acked-by: Dumitru Ceara Thanks, Dumitru ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Re: [ovs-dev] [PATCH ovn v2] Add weekly CI job that uses OVS master.
Bleep bloop. Greetings Mark Michelson, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: Failed to merge in the changes. hint: Use 'git am --show-current-patch' to see the failed patch Patch failed at 0001 Add weekly CI job that uses OVS master. When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort". Please check this out. If you feel there has been an error, please email acon...@redhat.com Thanks, 0-day Robot ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
[ovs-dev] [PATCH ovn v2] Add weekly CI job that uses OVS master.
Signed-off-by: Mark Michelson --- This patch is based on the "Include OVS as a git submodule." patch, which, at this time has not been merged into OVN master yet. --- .github/workflows/test.yml | 37 - 1 file changed, 36 insertions(+), 1 deletion(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 916b14da2..f3a53a8b6 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,11 @@ name: Build and Test -on: [push, pull_request] +on: + push: + pull_request: + schedule: +# Run Sunday at midnight +- cron: '0 0 * * 0' jobs: build-linux: @@ -49,10 +54,26 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive +# For weekly runs, don't update submodules +- name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + +# Weekly runs test using OVS master instead of the +# submodule. +- name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: +repository: 'openvswitch/ovs' +path: 'ovs' +ref: 'master' + - name: update APT cache run: sudo apt update @@ -105,9 +126,23 @@ jobs: steps: - name: checkout + if: github.event_name == 'push' || github.event_name == 'pull_request' uses: actions/checkout@v2 with: submodules: recursive +# For weekly runs, don't update submodules +- name: checkout without submodule + if: github.event_name == 'schedule' + uses: actions/checkout@v2 +# Weekly runs test using OVS master instead of the +# submodule. +- name: checkout OVS master + if: github.event_name == 'schedule' + uses: actions/checkout@v2 + with: +repository: 'openvswitch/ovs' +path: 'ovs' +ref: 'master' - name: install dependencies run: brew install automake libtool - name: prepare -- 2.29.2 ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
