On Thu, Mar 19, 2020 at 6:09 AM Aaron Conole <acon...@redhat.com> wrote: > > William Tu <u9012...@gmail.com> writes: > > > So far we only use travis to do run 'make check' per commit. > > This enables per-commit check for 'make check-system-userspace' test. > > We can think about what others to add using github actions. > > > > Example run: > > https://github.com/williamtu/ovs-travis/runs/514822181?check_suite_focus=true > > > > Signed-off-by: William Tu <u9012...@gmail.com> > > --- > > This is a cool feature - thanks for proposing it. > > I've only done a little bit of looking at it - I think I may need to > explicitly enable it for the bot repositories (but I still don't quite > understand the settings in 'Action permissions'). I'll spend more time > digging into it. > > Are there any thoughts to using the scripts in .travis and maybe > consolidating it to something more generic? >
Thanks. This is independent of travis CI so I don't think we need to change .travis. As soon as you push this .github/workflows/ovs.yml to your own github master branch, the CI runner will kick off. Currently travis takes about 1 hour to finish our tests, so I'm thinking about moving or adding tests here at github actions. William > > .github/workflows/ovs.yml | 25 +++++++++++++++++++++++++ > > Makefile.am | 1 + > > 2 files changed, 26 insertions(+) > > create mode 100644 .github/workflows/ovs.yml > > > > diff --git a/.github/workflows/ovs.yml b/.github/workflows/ovs.yml > > new file mode 100644 > > index 000000000000..1e056aceed2c > > --- /dev/null > > +++ b/.github/workflows/ovs.yml > > @@ -0,0 +1,25 @@ > > +name: OVS CI > > + > > +on: > > + push: > > + branches: [ master ] > > + pull_request: > > + branches: [ master ] > > + > > +jobs: > > + build: > > + runs-on: ubuntu-latest > > + steps: > > + - uses: actions/checkout@v2 > > + - name: configure > > + run: ./boot.sh; ./configure > > + - name: make > > + run: make -j2 > > + - name: check-system-userspace > > + run: sudo make check-system-userspace TESTSUITEFLAGS='1-30' > > RECHECK=yes > > + - name: Upload artifact > > + uses: actions/upload-artifact@v1.0.0 > > + if: failure() > > + with: > > + name: system-userspace > > + path: tests/system-userspace-testsuite.dir/ > > diff --git a/Makefile.am b/Makefile.am > > index b279303d186c..80448d0c31c1 100644 > > --- a/Makefile.am > > +++ b/Makefile.am > > @@ -92,6 +92,7 @@ EXTRA_DIST = \ > > $(MAN_ROOTS) \ > > Vagrantfile \ > > Vagrantfile-FreeBSD \ > > + .github/workflows/ovs.yml \ > > .mailmap > > bin_PROGRAMS = > > sbin_PROGRAMS = > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev