fix name of binary to forgejo-runner for rootless docker image #43

Merged
earl-warren merged 1 commit from :fix_rootless_script_bin_name into main 2023-07-08 10:02:03 +00:00
Contributor

While testing, I noticed that the rootless image can not be build because of errors while building the docker image. The errors are coming from the renamed binary, which is not found during the container build.

While testing, I noticed that the rootless image can not be build because of errors while building the docker image. The errors are coming from the renamed binary, which is not found during the container build.
louis9902 added 1 commit 2023-07-08 09:49:05 +00:00
All checks were successful
checks / check and test (pull_request) Successful in 1m2s
8fa9c5df20
fix name of binary to forgejo-runner for rootless docker image
Owner

It would be nice to publish the rootless image at https://code.forgejo.org/forgejo/-/packages/container/runner/versions. The focus has been on produce the binaries for now. And the container images are primarily a vessel to compile them. But in a k8s context there is a need for rootless images, AFAIU. Is this your target?

It would be nice to publish the rootless image at https://code.forgejo.org/forgejo/-/packages/container/runner/versions. The focus has been on produce the binaries for now. And the container images are primarily a vessel to compile them. But in a k8s context there is a need for rootless images, AFAIU. Is this your target?
earl-warren merged commit cfb20e8131 into main 2023-07-08 10:02:03 +00:00
Author
Contributor

No, not really, I'm just trying to get the runner running as rootless container (using podman with a rootless user and root inside the container), but so far without success.

No, not really, I'm just trying to get the runner running as rootless container (using podman with a rootless user and root inside the container), but so far without success.
Owner

I think it would be a simpler if https://codeberg.org/forgejo/forgejo/issues/983 was implemented. You're trying with docker compose or something different? I'm deploying binaries myself but I see the appeal for running from within a container.

I think it would be a simpler if https://codeberg.org/forgejo/forgejo/issues/983 was implemented. You're trying with docker compose or something different? I'm deploying binaries myself but I see the appeal for running from within a container.
Owner

By docker compose I mean whatever is equivalent to that with podman.

By `docker compose` I mean whatever is equivalent to that with podman.
Author
Contributor

I'm currently using ansible and podman quadlet (systemd unit generator). I deploy the *.kube file and the corresponding kube.yml file with ansible create a systemd unit and start and enable the created unit. So I can get auto restart if my server restarts as well as auto updating if I enable the podman service.
And if the forgejo subcommand is coming this will make a lot of things a lot easier in terms of deployment.

I'm currently using ansible and podman quadlet (systemd unit generator). I deploy the *.kube file and the corresponding kube.yml file with ansible create a systemd unit and start and enable the created unit. So I can get auto restart if my server restarts as well as auto updating if I enable the podman service. And if the forgejo subcommand is coming this will make a lot of things a lot easier in terms of deployment.
Owner

And if the forgejo subcommand is coming this will make a lot of things a lot easier in terms of deployment.

I'll work on that tomorrow.

> And if the forgejo subcommand is coming this will make a lot of things a lot easier in terms of deployment. I'll work on that tomorrow.
Menelion reviewed 2023-07-25 11:32:16 +00:00
caesar reviewed 2023-07-25 11:54:09 +00:00
caesar left a comment
Owner

Edit: sorry for the noise on this issue, just using it to test the accessibility improvements in Forgejo v1.20

_Edit: sorry for the noise on this issue, just using it to test the accessibility improvements in Forgejo v1.20_
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: forgejo/runner#43
No description provided.