Commit Graph

72 Commits

Author SHA1 Message Date
Josef Schlehofer afca2f2a71 CI: fix grammar typo when test is succesful
Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>
2020-10-16 08:15:53 -10:00
Michal Hrusecky 7951e29297 CI: Export few helpful variables for CI tests
Export PKG_NAME and PKG_VERSION to make test scripts better readable
and also export a path to the helper script providing colorful output.

Signed-off-by: Michal Hrusecky <michal.hrusecky@turris.com>
2020-10-10 11:16:13 -10:00
Michal Hrusecky 61c4202766 CI: Add helper functions
Add a shell script that can produce colorfull output to help to identify
problems during CI runs.

Signed-off-by: Michal Hrusecky <michal.hrusecky@turris.com>
2020-10-10 11:16:13 -10:00
Rosen Penev ba147d0d40 actions: remove arm_cortex-a9_neon
This is the second ARM NEON target that is tested. It's unlikely that
one will fail and the other succeed.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
2020-10-09 18:50:57 -10:00
Paul Spooren 014e02ab07 CI: use PKG_SOURCE instead of PKG_NAME
The PKG_NAME is the installable name of a package while PKG_SOURCE is
the folder containig both `Makefile` and possibliy `test.sh`

This approach previously worked for packages where both NAME and SOURCE
are the same, e.g. `vim`, however fore more complex packages like
`mariadb` (SOURCE) the NAMES are partly
*mariadb-server-plugin-handlersocket*, which is no existing folder.

With this commit the `PKG_SOURCE` is used to find the `test.sh` script.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-10-01 23:39:04 -10:00
Rosen Penev 527b3e2119 CI: get rid of the arc_archs buildbot
Both ARC platforms are not useful. One is enough for uClibc-ng.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
2020-09-29 22:54:20 -10:00
Rosen Penev 836b4e1c73
github-ci: add --force-removal-of-dependent-packages
Without this option, a package gets installed with its dependencies
but those do not get removed, causing issues later on with other
packages.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
2020-09-19 00:16:45 -07:00
Paul Spooren 4f4207be0c CI: pass only VERSION to test.sh
Currently the passed VERSION includes the release, which is usually not
part of the compiled binary. Removing it simplifies the `grep` command
to check for correct package output during runtime tests.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-17 22:19:49 -10:00
Paul Spooren b36bb173b3 CI: run also on test.sh changes
The CI only tests packages if a Makefile changed, e.g. containing a
version or release bump. This covers package related files as at least
the package release must change whenever a file was touched.

The `test.sh` file is a runtime test used to verify working packages
within OpenWrt containers. This file can independently change and will
never be included in the package ipk files, therefore trigger the CI on
its changes as well.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-17 09:05:16 -10:00
Paul Spooren da111fd095 CI: add runtime tests for packages
Additional to manual runtime tests this CI addition runs a custom test
script per package. Ideally this lowers the errors of package bumps,
something which is time consuming when done manually for multiple
architectures.

This CI uses the official OpenWrt containers and tries to install and
run compiled packages. The run depends on the content of `test.sh`,
which is an `ash` script. It's called with the *packge name* and
*package version* as arguments. This allows different behaviour if
a single package generates multiple IPK files. The version is usable for
the most trivial runtime check, e.g. `tmux -V | grep "$2"`.

The current approach uses the qus project[1] which contains multiple
QEMU binaries to run various architectures.

[1]: https://github.com/dbhi/qus

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-16 10:40:16 -10:00
Paul Spooren 2b5de22202 CI: add super-linter for pull requests
This commit adds a linter which checks most common filetypes, including
Shell and Lua. Ideally this improves the quality of especially `init`
scripts written in Shell.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-14 11:05:33 -10:00
Paul Spooren 43f4ebb532 CI: add compile test for arc_archs
A package recently failed building exclusively for this architecture,
therefore test it as well.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-12 14:16:11 -10:00
Paul Spooren 69c91a23a5 CI: fix compilation of all packages
By using `feeds install -d y` the CI tries to install e.g. `libcxx`
which fails, unrelated to the tested packages. Now follow the approach
of the current CircleCI implementation.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-08 11:02:27 -10:00
Paul Spooren b4c70ed7dd CI: fix artifact folder and add more test SDKs
The CI adds the `packages.git` repository to the `feeds.conf`, which
makes the repository redundant. Once called `packages` including the
upstream status, once `packages_ci` (previously `action`) including the
PRs changes.

This commit changes the binary artifact folder from `packages` to
`packages_ci`, as the SDK choses packages from the modified PR branch
over the `packages` branch.

Also add additional targets to test, as each target only takes a few
minutes to test: aarch64_cortex-a53, arm_cortex-a15_neon-vfpv4 and
i386_pentium4.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-05 13:21:04 -10:00
Paul Spooren b787aa9c81 CI: use GitHub action for Pull Requests
Test all GitHub pull requests for multiple architectures and store build
logs and created packages. This is the first commit of a series of
patches to simplify the life of maintainers.

Signed-off-by: Paul Spooren <mail@aparcar.org>
2020-09-04 15:36:07 -10:00
Daniel Engberg 31179ef2a4 repo: Add more information to the issue template
Rephrase the template a bit and add information about
older releases and forks.

Signed-off-by: Daniel Engberg <daniel.engberg.lists@pyret.net>
2019-04-07 20:18:55 +00:00
Yousong Zhou f552f738e2 github: adapt templates
Remove refs to LEDE and use "OpenWrt" instead of "OpenWRT"

Remove instructions on self-checking pull request content for the
following considerations

 - The checks are now enforced by travis autocheck scripts
 - Github now prompts users to refer to the contributing guide on
   submitting new issue and pull request

Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
2018-01-27 11:24:43 +08:00
Yousong Zhou 0be823dfc9 Add an issue template
So that maintainers can receive at least one notification at the issue
creation time ;)

Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
2016-08-25 20:32:50 +08:00
Etienne CHAMPETIER d749882a68 Improve pull_request_template
commit message is copied at the end of the template when creating PR,
so swap instructions and template part

Signed-off-by: Etienne CHAMPETIER <champetier.etienne@gmail.com>
2016-07-03 13:54:25 +00:00
Etienne CHAMPETIER c3e815cf40 Improve pull_request_template
Signed-off-by: Etienne CHAMPETIER <champetier.etienne@gmail.com>
2016-07-02 06:57:29 +00:00
Etienne CHAMPETIER 2c01e03bb5 Rename pull_request_template (it's text not markdown)
Signed-off-by: Etienne CHAMPETIER <champetier.etienne@gmail.com>
2016-07-02 06:57:12 +00:00
Etienne CHAMPETIER 158b4a630f Add a pull request template
I hope this will help triage PR and reduce commits not following guidelines

Signed-off-by: Etienne CHAMPETIER <champetier.etienne@gmail.com>
2016-06-17 08:02:23 +00:00