Renovate Bot and techknowlogick
0adfc1c7cc
chore(deps): update actions/setup-go action to v4 ( #418 )
...
Co-authored-by: Renovate Bot <[email protected] [email protected]
2023-11-24 03:05:33 +00:00
Renovate Bot and techknowlogick
e3c68668fa
chore(deps): update docker/build-push-action action to v5 ( #419 )
...
Co-authored-by: Renovate Bot <[email protected] [email protected]
2023-11-24 03:05:28 +00:00
Renovate Bot and techknowlogick
f1b27d5274
chore(deps): update docker/login-action action to v3 ( #420 )
...
Co-authored-by: Renovate Bot <[email protected] [email protected]
2023-11-24 03:05:22 +00:00
Renovate Bot and techknowlogick
655a39fd61
chore(deps): update docker/setup-buildx-action action to v3 ( #421 )
...
Co-authored-by: Renovate Bot <[email protected] [email protected]
2023-11-24 03:05:07 +00:00
Renovate Bot and techknowlogick
cca7d54117
chore(deps): update docker/setup-qemu-action action to v3 ( #422 )
...
Co-authored-by: Renovate Bot <[email protected] [email protected]
2023-11-24 03:05:00 +00:00
934471813a
Fix #404 : nil map error when reading env file ( #405 )
...
Co-authored-by: Mark Glines <[email protected] https://gitea.com/gitea/act_runner/pulls/405
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-11-24 01:56:27 +00:00
pat-s and techknowlogick
1e940f028b
Add renovate config ( #408 )
...
+ write access required for `renovate-bot` user
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/408
Reviewed-by: techknowlogick <[email protected] [email protected] [email protected]
2023-11-23 20:41:10 +00:00
lng2020 and techknowlogick
2020ce79bf
Remove ACTIONS_RUNTIME_TOKEN workaround ( #396 )
...
The bug has been fixed for a long time in the GitHub version.
The fix commit is https://github.com/docker/build-push-action/commit/d8823bfaed2a82c6f5d4799a2f8e86173c461aba and released in the 4.0.0 and after. The issue also mentions that https://gitea.com/gitea/act_runner/issues/119#issuecomment-738294
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/396
Reviewed-by: techknowlogick <[email protected] [email protected] [email protected]
2023-11-07 05:15:14 +00:00
Jason Song
00e9b3d62b
Bump act ( #394 )
...
Bump act to follow https://gitea.com/gitea/act/pulls/81
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/394
Reviewed-by: Lunny Xiao <[email protected]
2023-10-31 15:03:07 +00:00
130b9f1877
Added environment variable for run.sh to specify runner state file. ( #377 )
...
Added environment variable `RUNNER_STATE_FILE` to let users specify
where `run.sh` looks for the runner JSON file. Defaults to ``.runner``
to preserve the original behavior.
Addresses issue #368 . It's not my preferred solution but it's the least invasive one I can think of.
I'm happy to make any changes you want.
I didn't see an appropriate place to reference the change in the documentation. I will add documentation wherever you think is appropriate.
To test this I did the following:
* Built an image with the `make docker` command and pushed it to my private registry.
* Added that private image as the image in my existing docker stack that was exhibiting the behavior described in #368 .
* Added the RUNNER_STATE_FILE environment variable pointing to the runner JSON file set in my `config.yml` (``/data/runner.json`` in this case).
* Configured a new runner in gitea and added the token as an environment variable in the stack config.
* Deployed the stack and verified the new runner was recognized (in the idle state).
* Force updated the runner service to restart the container.
* Verified the same runner was still recognized in gitea (in the idle state) once it was back up and running.
Here is the relevant config. It's stored as a template so I've left the things that would normally be redacted as they are in the template.
```
runner:
image: hub.hax.in.net/haxwithaxe/act_runner:dev1
networks:
- swarm-net
environment:
RUNNER_STATE_FILE: /data/runner.json
CONFIG_FILE: /data/config.yml
GITEA_INSTANCE_URL: "https://git_gitea "
GITEA_RUNNER_REGISTRATION_TOKEN: "{{ git_runner_reg_token }}"
GITEA_RUNNER_NAME: "git_runner"
volumes:
- runner_data:/data
- /var/run/docker.sock:/var/run/docker.sock
```
`runner_data` is a glusterfs subvolume
Thanks for creating this gitea specific fork. Apart from the issue this pull request addresses it works so well I don't have to think about it once it's set up.
Co-authored-by: haxwithaxe <[email protected] https://gitea.com/gitea/act_runner/pulls/377
Co-authored-by: haxwithaxe <[email protected] [email protected]
2023-10-15 23:21:53 +00:00
4c35288175
Add DIND docker-compose example ( #365 )
...
Co-authored-by: Nikita Vilunov <[email protected] https://gitea.com/gitea/act_runner/pulls/365
Reviewed-by: Lunny Xiao <[email protected] [email protected] [email protected]
2023-10-02 15:12:31 +00:00
Andrew Imeson and techknowlogick
990db1bfc0
Fix k8s rootless Docker filesystem permissions ( #366 )
...
Without this the deployment fails because the runner can't write to
/data/
Credit to @ccureau for identifying the fix
Relates to: gitea/act_runner#264
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/366
Reviewed-by: techknowlogick <[email protected] [email protected] [email protected]
2023-10-02 15:12:14 +00:00
techknowlogick and techknowlogick
d07fbfc8c3
bump act library to v0.2.51 ( #360 )
...
brings in node20 support
Co-authored-by: techknowlogick <[email protected] https://gitea.com/gitea/act_runner/pulls/360
Reviewed-by: delvh <[email protected]
2023-09-24 19:20:40 +00:00
10dc6fb60d
Allow reading registration token from file ( #350 )
...
When deploying the runner in a Docker Swarm setup, it can be useful to
read the potentially sensitive token from a secret instead of having to
pass it from an environment variable.
Co-authored-by: Frederik Ring <[email protected] https://gitea.com/gitea/act_runner/pulls/350
Reviewed-by: Jason Song <[email protected] [email protected] [email protected] [email protected]
2023-09-04 04:12:07 +00:00
TheFox0x7 and Jason Song
ed35b09b8f
change podman socket path ( #341 )
...
port of https://github.com/nektos/act/pull/1961
closes gitea/act_runner#274
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/341
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-08-21 04:01:12 +00:00
Gianni Angelozzi and Jason Song
03f0829d09
Add ForcePull option ( #339 )
...
Close #271
What it does: instead of forcing the value of `ForcePull` to false, the user can now configure it on the runner yaml
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/339
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-08-17 06:51:57 +00:00
Jason Song and Lunny Xiao
7fc1b91ba6
Update to go1.21 in Dockerfile.rootless ( #332 )
...
Follow #330
Fix https://gitea.com/gitea/act_runner/pulls/330#issuecomment-747099
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/332
Reviewed-by: Lunny Xiao <[email protected] [email protected] [email protected] [email protected]
2023-08-12 00:37:08 +00:00
82c3c2df1a
Upgrade Go to 1.21 and bump other dependencies ( #330 )
...
Co-authored-by: harryzcy <[email protected] https://gitea.com/gitea/act_runner/pulls/330
Reviewed-by: Lunny Xiao <[email protected] [email protected] [email protected]
2023-08-10 01:45:25 +00:00
sillyguodong and Jason Song
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 https://gitea.com/gitea/act/src/commit/22d91e3ac3740bb00e8ec044026f028f811cfb61/pkg/container/docker_run.go#L594 ), the actual value of `request.Host` is `"/var/run/docker.sock"`. This does not conform to the specification described in `validhostHeader`.
<details> <summary>ValidHostHeader()</summary>
</details>
So this PR follow upstream: 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 <[email protected] [email protected] [email protected]
2023-08-02 04:34:36 +00:00
sillyguodong and Jason Song
12999b61dd
Reduce unnecessary DB queries for Actions tasks ( #219 )
...
implement: https://github.com/go-gitea/gitea/issues/24544
Changes:
- Add a global variable `tasksVersion` to store the lastest version number which returned by Gitea.
- Pass `tasksVersion` to Gitea when invoking `FetchTask`.
- If there is no task in the `FetchTask` response, store the version from the `FetchTask` response into `tasksVersion` variable.
- If there is a task in the `FetchTask` response, set `tasksVersion` to zero to focre query db in next `FetchTask` request.
Related:
- Protocol: https://gitea.com/gitea/actions-proto-def/pulls/10
- Gitea side: https://github.com/go-gitea/gitea/pull/25199
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/219
Co-authored-by: sillyguodong <[email protected] [email protected]
2023-07-25 03:25:50 +00:00
49a2fcc138
fix endless loop ( #306 )
...
fix endless loop in poll
relate #305
Co-authored-by: CaiCandong <[email protected] https://gitea.com/gitea/act_runner/pulls/306
Co-authored-by: caicandong <[email protected] [email protected]
2023-07-24 07:07:53 +00:00
Konstantin Podsvirov and Jason Song
8f88e4f15a
Update README.md ( #302 )
...
Correct Quick Start section.
Co-authored-by: Jason Song <[email protected] https://gitea.com/gitea/act_runner/pulls/302
Co-authored-by: Konstantin Podsvirov <[email protected] [email protected]
2023-07-24 05:11:42 +00:00
a1bb3b56fd
Catch the panic and print the error ( #305 )
...
refactor # 215 Catch the panic and print the error
close #215
Co-authored-by: CaiCandong <[email protected] https://gitea.com/gitea/act_runner/pulls/305
Co-authored-by: caicandong <[email protected] [email protected]
2023-07-24 04:28:44 +00:00
sillyguodong and Jason Song
1a7ec5f339
Upgrade gitea/act ( #298 )
...
Follow https://gitea.com/gitea/act/pulls/75
Co-authored-by: Jason Song <[email protected] https://gitea.com/gitea/act_runner/pulls/298
Co-authored-by: sillyguodong <[email protected] [email protected]
2023-07-20 05:12:59 +00:00
Jason Song
5d01cb8904
Add tips in config file ( #297 )
...
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/297
2023-07-20 02:16:30 +00:00
Thomas E Lackey and Jason Song
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 <[email protected] [email protected]
2023-07-19 05:12:07 +00:00
Zettat123 and Jason Song
73adae040d
Upgrade act ( #291 )
...
Follow https://gitea.com/gitea/act/pulls/74
Fix #277 , #290
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/291
Co-authored-by: Zettat123 <[email protected] [email protected]
2023-07-17 03:44:42 +00:00
Bo-Yi Wu and Lunny Xiao
db662b3690
ci(lint): refactor code for clarity and linting compliance ( #289 )
...
- Removed `deadcode`, `structcheck`, and `varcheck` linters from `.golangci.yml`
- Fixed a typo in a comment in `daemon.go`
- Renamed `defaultActionsUrl` to `defaultActionsURL` in `exec.go`
- Removed unnecessary else clause in `exec.go` and `runner.go`
- Simplified variable initialization in `exec.go`
- Changed function name from `getHttpClient` to `getHTTPClient` in `http.go`
- Removed unnecessary else clause in `labels_test.go`
Signed-off-by: Bo-Yi Wu <[email protected] https://gitea.com/gitea/act_runner/pulls/289
Co-authored-by: Bo-Yi Wu <[email protected] [email protected]
2023-07-13 01:10:54 +00:00
Bo-Yi Wu and Lunny Xiao
cf92a979e2
refactor(register): refactor registration functions and improve logging ( #288 )
...
- Remove the else clause in the `registerInteractive` function, and unconditionally log "Runner registered successfully."
- Change the return value in the `doRegister` function from `nil` to `ctx.Err()`.
Signed-off-by: Bo-Yi Wu <[email protected] https://gitea.com/gitea/act_runner/pulls/288
Co-authored-by: Bo-Yi Wu <[email protected] [email protected]
2023-07-12 14:12:16 +00:00
Bo-Yi Wu and Lunny Xiao
87058716fb
fix(register): refactor context usage in registration functions ( #286 )
...
- Add context parameter to `registerNoInteractive`, `registerInteractive`, and `doRegister` functions
- Remove the creation of a new context in `doRegister`, now using the passed context instead
Signed-off-by: Bo-Yi Wu <[email protected] https://gitea.com/gitea/act_runner/pulls/286
Co-authored-by: Bo-Yi Wu <[email protected] [email protected]
2023-07-12 13:11:55 +00:00
Lunny Xiao and Jason Song
c701ba4787
Add a quick start runner method in README ( #282 )
...
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/282
Co-authored-by: Lunny Xiao <[email protected] [email protected]
2023-07-12 01:43:26 +00:00
Michael Santos and Jason Song
57ff1df6e0
config: default container workspace set to host path ( #279 )
...
The container workspace path is overwritten by the default host workspace path ($HOME/.cache/act).
Workaround:
```yaml
host:
workdir_parent: workspace
```
Ref: https://gitea.com/gitea/act_runner/commit/34d15f21c2ea8a6dbc8c3b905f332ec98ea0b813
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/279
Co-authored-by: Michael Santos <[email protected] [email protected]
2023-07-10 08:57:55 +00:00
Zettat123 and Jason Song
3dcfd6ea3d
Run as cache server ( #275 )
...
This PR
- adds the `cache-server` command so act_runner can run as a cache server. When running as a cache server, act_runner only processes the requests related to cache and does not run jobs.
- adds the `external_server` configuration for cache. If specified, act_runner will use this URL as the ACTIONS_CACHE_URL instead of starting a cache server itself.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/275
Co-authored-by: Zettat123 <[email protected] [email protected]
2023-07-07 08:28:54 +00:00
Zettat123 and Jason Song
c6006ee699
Upgrade act ( #269 )
...
Follow https://gitea.com/gitea/act/pulls/71 .
Fix https://gitea.com/gitea/act_runner/issues/266
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/269
Co-authored-by: Zettat123 <[email protected] [email protected]
2023-07-03 04:15:46 +00:00
Zettat123 and Lunny Xiao
f2629f2ea3
Add support for finding docker daemon from common socket paths ( #263 )
...
Caused by #260
act_runner will fail to start if user does not set `docker_host` configuration and `DOCKER_HOST` env. This PR adds the support for finding docker daemon from common socket paths so act_runner could detect the docker socket from these paths.
The `commonSocketPaths` is from [nektos/act](https://github.com/nektos/act/blob/e60018a6d9403b27bfd1a1ed6111a6e9de0032fd/cmd/root.go#L124-L131 )
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/263
Co-authored-by: Zettat123 <[email protected] [email protected]
2023-07-01 01:27:54 +00:00
Jason Song
cf48ed88ba
Revert supporting multiple default actions URLs and use github for exec by default ( #262 )
...
## ⚠️ BREAKING ⚠️
Follow https://github.com/go-gitea/gitea/pull/25581 and gitea/act#70 .
- Revert "Parse multiple default actions URLs (#200 )"
- Revert "fix defaultActionsUrls config for exec (#233 )"
- Use `https://github.com ` for exec by default.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/262
2023-06-30 07:53:18 +00:00
Zettat123 and Jason Song
ccc27329dc
Improve the usage of docker_host configuration ( #260 )
...
Follow #242 , #244
Fixes #258
Users could use `docker_host` configuration to specify which docker daemon will be used by act_runner.
- If `docker_host` is **empty**, act_runner will find an available docker host automatically.
- If `docker_host` is **"-"**, act_runner will find an available docker host automatically, but the docker host won't be mounted to the job containers and service containers.
- If `docker_host` is **not empty or "-"**, the specified docker host will be used. An error will be returned if it doesn't work.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/260
Co-authored-by: Zettat123 <[email protected] [email protected]
2023-06-30 04:00:04 +00:00
Tomasz Duda and Jason Song
b0bd503b11
add token support for exec ( #247 )
...
allow to pass token from secrets
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/247
Reviewed-by: Jason Song <[email protected] [email protected] [email protected] [email protected]
2023-06-20 08:41:22 +00:00
Zettat123 and Jason Song
8c14933e70
Upgrade act ( #248 )
...
Follow https://gitea.com/gitea/act/pulls/68
Co-authored-by: Jason Song <[email protected] https://gitea.com/gitea/act_runner/pulls/248
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-20 08:33:42 +00:00
Marius Zwicker and Jason Song
34d15f21c2
Add option to configure workspace on host ( #238 )
...
Adds a new section to the configuration which is used
to control options when running in host mode.
The first option added is to allow configuration
of the location workspaces get created in.
Depends on ~~gitea/act#65~~
Will resolve #235
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/238
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-20 08:29:05 +00:00
Tomasz Duda and Jason Song
32d29f0813
add ACT_EXEC ( #246 )
...
Add env variable to distinguish build run locally from remote one.
Co-authored-by: Jason Song <[email protected] https://gitea.com/gitea/act_runner/pulls/246
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-20 02:08:35 +00:00
Tomasz Duda and Jason Song
2e2c0400c8
add --gitea-instance ( #245 )
...
add --gitea-instance to let user specify address of endpoint of exec
Related to https://gitea.com/gitea/act/pulls/68 . Both can be merged independently though.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/245
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-20 01:57:21 +00:00
Jason Song
054c8d912f
Move docker.host to container.docker_host ( #244 )
...
Follow #242 .
Move `docker.host` to `container.docker_host`.
There are already some options for docker/container in `container`, so developers could get confused about where to add options.
It's breaking, but I think it's OK since `docker.host` was added just two days ago.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/244
Reviewed-by: Lunny Xiao <[email protected]
2023-06-19 09:01:16 +00:00
appleboy and Lunny Xiao
9e4a5f7363
feat: improve Docker configuration and detection handling ( #242 )
...
- Pass `cfg` to `envcheck.CheckIfDockerRunning` function
- Add `Docker` struct to `config.go` for Docker configuration
- Update `config.example.yaml` with `docker` configuration options
- Modify `CheckIfDockerRunning` in `docker.go` to use Docker host from config if provided
Signed-off-by: appleboy <[email protected] https://gitea.com/gitea/act_runner/pulls/242
Reviewed-by: Lunny Xiao <[email protected] [email protected] [email protected] [email protected]
2023-06-18 05:38:38 +00:00
Zettat123 and Jason Song
ec38401097
Add ValidVolumes config ( #226 )
...
Follow https://gitea.com/gitea/act/pulls/60 , https://gitea.com/gitea/act/pulls/64
This PR adds the `valid_volumes` configuration. `valid_volumes` is a sequence containing the volumes (including bind mounts) that can be mounted to the container. By default, `valid_volumes` is empty, which means that no volumes can be mounted. Users can specify multiple valid volumes and [glob](https://github.com/gobwas/glob ) is supported.
All volumes will be allowed when using `exec` to run workflows locally.
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/226
Reviewed-by: Lunny Xiao <[email protected] [email protected] [email protected] [email protected]
2023-06-16 06:07:48 +00:00
Jason Song
45bfe0a9b2
Accept empty file as config ( #241 )
...
Close #240
`yaml.Decoder.Decode` will return EOF when the root node is nil , see https://github.com/go-yaml/yaml/blob/v3/yaml.go#L125
While `yaml.Unmarshal` will accept it, see https://github.com/go-yaml/yaml/blob/v3/yaml.go#L162
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/241
Reviewed-by: Zettat123 <[email protected]
2023-06-15 05:40:37 +00:00
Jason Song
316534996a
Build docker image gitea/act_runner/x.y.z-dind-rootless ( #239 )
...
Follow #208
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/239
Reviewed-by: Zettat123 <[email protected]
2023-06-15 05:27:35 +00:00
sillyguodong and Jason Song
67b1363d25
Support changing labels ( #201 )
...
Implement proposal: https://github.com/go-gitea/gitea/issues/24540
Related:
- Protocol: https://gitea.com/gitea/actions-proto-def/pulls/9
- Gitea side: https://github.com/go-gitea/gitea/pull/24806
Co-authored-by: Jason Song <[email protected] https://gitea.com/gitea/act_runner/pulls/201
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-15 03:59:15 +00:00
Tomasz Duda and Jason Song
946c41cf4f
Improve run.sh to handle empty labels and log to stdout ( #237 )
...
1. Print logs on standard output
2. Don't add labels if GITEA_RUNNER_LABELS not set
Reviewed-on: https://gitea.com/gitea/act_runner/pulls/237
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-13 04:09:11 +00:00
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 <[email protected] https://gitea.com/gitea/act_runner/pulls/208
Reviewed-by: Jason Song <[email protected] [email protected] [email protected]
2023-06-12 06:35:27 +00:00
Renovate Botandtechknowlogick
pat-sandtechknowlogick
lng2020andtechknowlogick
Andrew Imesonandtechknowlogick
TheFox0x7andJason Song
Gianni AngelozziandJason Song
sillyguodongandJason Song
Konstantin PodsvirovandJason Song
Thomas E LackeyandJason Song
Zettat123andJason Song
Bo-Yi WuandLunny Xiao
Michael SantosandJason Song
Zettat123andLunny Xiao
Tomasz DudaandJason Song
Marius ZwickerandJason Song
