1658f51fcb
Make Runner{Deployment,ReplicaSet} replicas actually optional ( #186 )
...
If omitted, it properly defaults to 1.
Fixes #64
2020-11-14 22:06:33 +09:00
9a22bb5086
Initial version of Helm chart ( #187 )
...
Acceptance tests are passing with the chart. In addition to standard chart values, syncPeriod is supported.
Please use it as a foundation for further collaboration.
Ref #184
Inspired by #91
Related #61
2020-11-14 22:06:14 +09:00
71436f0466
Final fixes before merging
2020-11-14 22:00:41 +09:00
b63879f59f
Ensure the chart is passing acceptance tests
2020-11-14 21:58:16 +09:00
c623ce0765
docs: spelling 🙃
2020-11-14 12:32:14 +00:00
01117041b8
chart: controller-manager should not be autoscaled
2020-11-14 20:37:40 +09:00
42a272051d
chart: Use the correct image
2020-11-14 20:37:22 +09:00
6a4c29d30e
Set `ACCEPTANCE_TEST_DEPLOYMENT_TOOL=helm` to run acceptance tests with chart
2020-11-14 20:31:37 +09:00
ce48dc58e6
Add crds to the chart
2020-11-14 20:31:21 +09:00
04dde518f0
docs: adding mroe detail for sync period config
2020-11-14 11:22:09 +00:00
1b98c8f811
refactor: adding explicit sync-period flag
2020-11-14 11:19:16 +00:00
14b34efa77
Start collaborating to develop a Helm chart
...
Ref #184
2020-11-14 20:15:42 +09:00
bbfe03f02b
Add acceptance test ( #168 )
...
To ease verifying the controller to work before submitting/merging PRs and releasing a new version of the controller.
2020-11-14 20:07:14 +09:00
8ccf64080c
Merge pull request #182 from callum-tait-pbx/docs/improvements
...
docs/improvements
2020-11-14 12:02:24 +09:00
492ea4b583
docs: adding a common errors section
2020-11-13 08:38:25 +00:00
3446d4d761
docs: reordering to be more logical
2020-11-13 08:27:18 +00:00
1f82032fe3
Improvements in the documentation for permissions and replication ( #175 )
2020-11-13 09:36:22 +09:00
5b2272d80a
Add read-only permissions to actions for being able to autoscale ( #180 )
2020-11-13 09:27:01 +09:00
0870250f9b
Enable matrix build for runner image ( #179 )
2020-11-13 09:24:12 +09:00
a4061d0625
gofmt ed
2020-11-12 09:20:06 +09:00
7846f26199
Increase memory limit ( #173 )
...
We saw controller quite often OOMing, so first help to increase limit a bit.
2020-11-12 09:14:33 +09:00
8217ebcaac
Updated Runner to 2.274.1 ( #176 )
...
* Updated Runner to 2.274.1
* Update image tag in workflow
2020-11-12 09:14:06 +09:00
83857ba7e0
use tcp DOCKER_HOST instead of sharing docker.sock
2020-11-12 08:07:52 +09:00
e613219a89
Fix token registration broken since v0.11.0 ( #167 )
...
Fixes #166
2020-11-11 09:38:05 +09:00
bc35bdfa85
fixed label argument in entrypoint ( #162 )
...
* fixed label argument in entrypoint
* Removed quotes from RUNNER_GROUP_ARG
2020-11-11 08:44:51 +09:00
ece8fd8fe4
Bump Go to 1.15 ( #160 )
...
Closes #104
2020-11-10 17:16:04 +09:00
dcf8524b5c
Adds RUNNER_GROUP argument to the runner registration ( #157 )
...
* Adds RUNNER_GROUP argument to the runner registration
Adds the ability to register a runner to a predefined runner_group
Resolves #137
* Update README with runner group example
- Updates the README with instructions of how to add the runner to a
group
- Fix code fencing for shell and yaml blocks in the README
- Use consistent bullet points (dash not asterisk)
2020-11-10 17:15:54 +09:00
4eb45d3c7f
Fix build error
2020-11-10 17:09:16 +09:00
1c30bdf35b
Add GHE URL to transport ( #152 )
...
Fixes #149
2020-11-10 17:05:09 +09:00
3f335ca628
Fix panic on startup when misconfigured ( #154 )
...
Fixes #153
2020-11-10 17:03:33 +09:00
f2a2ab7ede
Check token validity only when creating new pod ( #159 )
...
Fixes #143
2020-11-10 17:02:30 +09:00
40c5050978
Added support for other than public GitHub URL ( #146 )
...
Refactoring a bit
2020-10-28 22:15:53 +09:00
99a53a6e79
Releasing latest controller from master push ( #147 )
...
Fixes #135
2020-10-28 22:13:35 +09:00
6d78fb07b3
Fix permission error with the default setup since v0.9.4 ( #142 )
...
Fixes #138
2020-10-25 11:25:48 +09:00
faaca10fba
Rename Runner.Spec.dockerWithinRunnerContainer to docker"d"WithinRunnerContainer ( #134 )
...
* Rename Runner.Spec.dockerWithinRunnerContainer to dockerdWithinRunnerContainer
Ref https://github.com/summerwind/actions-runner-controller/pull/126#issuecomment-712501790
2020-10-21 21:32:40 +09:00
d16dfac0f8
Restart if pod ends up succeeded ( #136 )
...
Fixes #132
2020-10-21 21:32:26 +09:00
af483d83da
Possibility to define resources for DIND container ( #125 )
...
Ref #119
2020-10-21 10:26:19 +09:00
92920926fe
Configurable "runner and DinD in a single container" ( #126 )
2020-10-20 08:48:28 +09:00
7d0bfb77e3
Inject Env Vars into Runner defined Container Spec ( #127 )
...
The runner token is now injected into the `runner` container defined within Runner.Spec.Containers[]
2020-10-20 08:43:53 +09:00
c4074130e8
Patch additional protocol instances during manifest generation ( #129 )
...
Fixes #128
2020-10-19 09:57:53 +09:00
be2e61f209
Add "alternatives" section to README ( #124 )
...
Grow together :)
2020-10-15 08:40:02 +09:00
da818a898a
Manifests valid for K8s 1.18 and 1.19 ( #123 )
...
Fixes #113
Fixes #116
2020-10-15 08:39:46 +09:00
2d250d5e06
Fix release build ( #122 )
2020-10-14 08:43:14 +09:00
231cde1531
Update build-runner workflow to be compatible with forks, fix image push ( #117 )
...
Partly revert and enhances #115
This is a follow-up to #115 that replaces the hardcoded `summerwind` portion of the image name with `${{ github.repository_owner }}` to enable contributors to test the image pushing behavior and fixes image building by conditionally passing `--push` to the build step based on the event that triggered the workflow.
After setting the `DOCKER_ACCESS_TOKEN` Secret on my fork of this repository, I was able to use this updated workflow to [build and push](https://github.com/urcomputeringpal/actions-runner-controller/runs/1242793758?check_suite_focus=true ) a [set of images](https://hub.docker.com/r/urcomputeringpal/actions-runner/tags ) and confirm their functionality. I imagine this will be useful to future contributors who wish to help with the chore of keeping up with https://github.com/actions/runner/releases .
2020-10-13 21:14:36 +09:00
c986c5553d
Fixing Docker Build and Push for Runner Image ( #115 )
...
A new image tag for the runner stopped being published on master merges from changes in #86 .
This fixes that in the following ways:
- Tests the GH workflow on PRs w/o pushing the images
- Runner and Docker version are moved from Makefile to GH Actions workflow and are passed in as build args
- GHA workflow runs on PRs, and if the workflow file itself is changed (i.e. version bump) or the runner Docker source changes (excluding the Makefile since thats just for local dev)
- Images are pushed on push (i.e. a merge)
2020-10-09 09:16:24 +09:00
f12bb76fd1
Update runner to v2.273.5 ( #111 )
2020-10-08 09:02:01 +09:00
a63860029a
Prefer autoscaling based on jobs rather than workflows if available ( #114 )
...
Adds the ability to autoscale on jobs in addition to workflows. We fall back to using workflow metrics if job details are not present.
Resolves #89
2020-10-08 09:00:44 +09:00
1bc6809c1b
Merge pull request #110 from summerwind/ensure-controller-gen-code-manifests-sync
...
Ensure controller-gen is up-to-date and the code and the manifests are in-sync
2020-10-06 09:44:25 +09:00
2e7b77321d
Verify manifests on pull request build
2020-10-06 09:30:25 +09:00
1e466ad3df
Ensure controller-gen is up-to-date and the code and the manifests are in-sync
...
Follow-up for #95 that added /finalizers subresource permission and #103 that upgraded controller-gen from 0.2.4 from 0.3.0
2020-10-06 09:23:03 +09:00
Yusuke Kuoka
Yusuke Kuoka
Callum Tait
Javier de Pedro López
Elliot Maincourt
Reinier Timmer
Shinnosuke Sawada
Juho Saarinen
Reinier Timmer
Dan Webb
Brendan Galloway
Jesse Newland
Hayden Fuss
Harry Gogonis
Dominic LoBue