harryzcy
82c3c2df1a
Upgrade Go to 1.21 and bump other dependencies ( #330 )
...
Co-authored-by: harryzcy <harry@harryzheng.com >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/330
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: harryzcy <harryzcy@noreply.gitea.com >
Co-committed-by: harryzcy <harryzcy@noreply.gitea.com >
2023-08-10 01:45:25 +00:00
sillyguodong
9fc823e4b1
Fix "http: invalid Host header" issue ( #319 )
...
This issue is caused by the addition of validation logic for `Host` in `http.Request` in golang 1.20.6 (see https://go-review.googlesource.com/c/go/+/507357/6/src/net/http/request.go )
In `act`, when execute `ContainerExecAttach()`(see 22d91e3ac3/pkg/container/docker_run.go (L594)https://github.com/nektos/act/pull/1917 and revert https://gitea.com/gitea/act_runner/pulls/295
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/319
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: sillyguodong <gedong_1994@163.com >
Co-committed-by: sillyguodong <gedong_1994@163.com >
2023-08-02 04:34:36 +00:00
Thomas E Lackey
dcf84d8a53
Avoid https://github.com/nektos/act/issues/1908 by specifying golang 1.20.5. ( #295 )
...
Without this, actions fail when attempting to clone the repo:
```
2023-07-18 17:11:02 [Smoke Test/Run basic test suite] failed to attach to exec: http: invalid Host header
```
This appears to be the same as https://github.com/nektos/act/issues/1908 , and only shows up with golang 1.20.6. Staying with golang 1.20.5 is a temporary solution to avoid the bug until it is fixed upstream.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/295
Co-authored-by: Thomas E Lackey <telackey@bozemanpass.com >
Co-committed-by: Thomas E Lackey <telackey@bozemanpass.com >
2023-07-19 05:12:07 +00:00
ccureau
341d49a24d
implement act_runner rootless image ( #208 )
...
This PR creates a rootless Docker image that runs both `dockerd` and `act_runner` using `supervisord`. It has been tested locally for a few days and seems stable.
Co-authored-by: ccureau <ccureau@noreply.gitea.io >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/208
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: ccureau <ccureau@noreply.gitea.com >
Co-committed-by: ccureau <ccureau@noreply.gitea.com >
2023-06-12 06:35:27 +00:00
Jason Song
a29307a9d9
Remove hadolint and improve Dockerfile ( #234 )
...
Replace #190
See:
- https://gitea.com/gitea/act_runner/pulls/190#issuecomment-741196
- https://gitea.com/gitea/act_runner/pulls/208#issuecomment-741049
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/234
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: silverwind <silverwind@noreply.gitea.com >
Reviewed-by: delvh <dev.lh@web.de >
2023-06-09 02:50:30 +00:00
sillyguodong
d5e4baed54
Fix missing runner version when building images from a Dockerfile ( #181 )
...
regression of https://gitea.com/gitea/act_runner/pulls/172
https://gitea.com/gitea/act_runner/actions/runs/400/jobs/1
In the step of `Build and push`, log output that `git: not found`. This lead to runner's version not being injected when go compile.
Co-authored-by: Jason Song <i@wolfogre.com >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/181
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: sillyguodong <gedong_1994@163.com >
Co-committed-by: sillyguodong <gedong_1994@163.com >
2023-05-09 16:44:31 +08:00
sando38
d4caa7e065
Dockerfile: Improve signal handling by adding a runtime init ( #180 )
...
This adds tini as a runtime init (https://github.com/krallin/tini ). It improves signal handling for the container, see https://github.com/krallin/tini#why-tini .
An alternative could be to run the container with `docker run --init ...` which also places tini as a runtime init as PID 1.
Co-authored-by: sando38 <sandomir@tutanota.com >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/180
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: sando38 <sando38@noreply.gitea.io >
Co-committed-by: sando38 <sando38@noreply.gitea.io >
2023-05-09 16:09:48 +08:00
Bo-Yi Wu
c40b651873
chore: improve Dockerfile, README, and testing settings ( #172 )
...
- Remove `git=2.38.5-r0` from the `apk add` command in Dockerfile
- Update the download link for act_runner in README.md to be a clickable link
Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/172
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: Bo-Yi Wu <appleboy.tw@gmail.com >
Co-committed-by: Bo-Yi Wu <appleboy.tw@gmail.com >
2023-05-04 09:36:31 +08:00
appleboy
0d727eb262
build: optimize Dockerfile and update dependencies ( #162 )
...
- Update base images to golang:1.20-alpine3.17 and alpine:3.17
- Replace `--update-cache` with `--no-cache` in apk add command
- Specify exact versions for make, git, and bash packages
Signed-off-by: appleboy <appleboy.tw@gmail.com >
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/162
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Co-authored-by: appleboy <appleboy.tw@gmail.com >
Co-committed-by: appleboy <appleboy.tw@gmail.com >
2023-04-29 12:07:15 +08:00
Thomas E Lackey
5a8134410d
Run as a container ( #8 ) including Docker-in-Docker. ( #84 )
...
This adds a very simple Dockerfile and run script for running `act_runner` as a container.
It also allows setting `Privileged` and `ContainerOptions` flags via the new config file when spawning task containers. The combination makes it possible to use Docker-in-Docker (which requires `privileged` mode) as well as pass any other options child Docker containers may require.
For example, if Gitea is running in Docker on the same machine, for the `checkout` action to behave as expected from a task container launched by `act_runner`, it might be necessary to map the hostname via something like:
```
container:
network_mode: bridge
privileged: true
options: --add-host=my.gitea.hostname:host-gateway
```
> NOTE: Description updated to reflect latest code.
> NOTE: Description updated to reflect latest code (again).
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/84
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com >
Reviewed-by: Jason Song <i@wolfogre.com >
Co-authored-by: Thomas E Lackey <telackey@bozemanpass.com >
Co-committed-by: Thomas E Lackey <telackey@bozemanpass.com >
2023-04-11 10:58:12 +08:00
