Commit graph

253 commits

Author SHA1 Message Date
Gabriel
c219d4fd84
Merge pull request #72 from gabriel-samfira/fix-tls-listener
Fix TLS listener
2023-01-31 15:49:52 +02:00
Gabriel Adrian Samfira
27a523f133 Fix TLS listener
The TLS listener was not being set up correctly. The TLSConfig was changed
to include only cert and key. The cert now needs to be a full chain bundle
including intermediary CA certificates. The ca_cert config option was removed.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-31 13:42:39 +00:00
Gabriel
1ad52e87ef
Merge pull request #71 from gabriel-samfira/add-extra-specs
Add extra specs on pools
2023-01-30 18:09:50 +02:00
Gabriel Adrian Samfira
040cb0f5f6 Add extra-specs flag
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-30 16:03:50 +00:00
Gabriel Adrian Samfira
d00da32375 Deduplicate some code
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-30 14:29:55 +00:00
Gabriel Adrian Samfira
f25951decb Add extra specs on pools
Extra specs is an opaque valid JSON that can be set on a pool and which
will be passed along to the provider as part of instance bootstrap params.

This field is meant to allow operators to send extra configuration values
to external or built-in providers. The extra specs is not interpreted or
useful in any way to garm itself, but it may be useful to the provider
which interacts with the IaaS.

The extra specs are not meant to be used for secrets. Adding sensitive
information to this field is highly discouraged. This field is meant as a    
means to add fine tuning knobs to the providers, on a per pool basis.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-30 13:10:21 +00:00
Gabriel
1867bbaad6
Merge pull request #70 from gabriel-samfira/move-some-code-around
Move some code around
2023-01-29 17:58:22 +02:00
Gabriel Adrian Samfira
8d17498ab8
Remove caches, retry fetching the hostname
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-29 16:03:20 +02:00
Gabriel Adrian Samfira
77f96d2761 Drive-by fix: Get propper runner prefix for pool
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-27 15:00:06 +00:00
Gabriel Adrian Samfira
4d071b7d10 Return only alpha numeric characters as an ID
On some providers the default character set used by shortid may lead to
errors when creating runners, due to the fact that underscores are not
allowed in their names.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-27 14:57:25 +00:00
Gabriel Adrian Samfira
8f56f51598 Move some code around
Move the metrics code into its own package.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-27 14:57:25 +00:00
Gabriel
a3ddb5e63a
Merge pull request #69 from mercedes-benz/fix-update-prefix
fix: don't update empty runner prefix
2023-01-27 10:54:11 +02:00
Michael Kuhnt
ddae6095a1
fix: don't update unset runner prefix 2023-01-27 09:05:58 +01:00
Gabriel
9dcd9df95f
Merge pull request #60 from mercedes-benz/metrics
feat: add prometheus metrics & endpoint
2023-01-26 23:58:57 +02:00
Michael Kuhnt
6cd18ff1fd
improve access to controller info 2023-01-26 22:13:18 +01:00
Michael Kuhnt
6a032bfaa2
metrics: fix review findings 2023-01-26 15:46:27 +01:00
Michael Kuhnt
ee659f509f
feat: add prometheus metrics & endpoint 2023-01-26 14:15:16 +01:00
Gabriel
a0670c6503
Merge pull request #61 from mercedes-benz/fix-webhook-signature-validation
Fix webhook signature validation
2023-01-24 15:08:03 +02:00
mgoeppe
0ebd20620d added secret check in param validation 2023-01-24 08:56:57 +01:00
mgoeppe
dce1808860 fixed runner tests 2023-01-24 08:51:25 +01:00
mgoeppe
f9f917ba05 aligned code enterprises,organizations and repositories and fixed sql tests 2023-01-24 08:51:25 +01:00
mgoeppe
5813dce051 fix webhook validation for enterprises
* also list needs to decrypt the secret
2023-01-24 08:47:54 +01:00
Gabriel
03d632c2b6
Merge pull request #68 from gabriel-samfira/sanitize-log-entries
Sanitize log entries
2023-01-23 18:06:06 +02:00
Gabriel Adrian Samfira
e93b6d73e5
Sanitize log entries
While most of these log entries come from either github or our own
database, it's still a good idea to sanitize them.
2023-01-23 18:01:46 +02:00
Gabriel
114f17718b
Merge pull request #67 from gabriel-samfira/disable-windows-build
Disable Windows binary build
2023-01-21 01:45:51 +02:00
Gabriel Adrian Samfira
b8e02ecbe1
Disable Windows binary build
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-21 01:32:19 +02:00
Gabriel
baa02dde98
Create codeql.yml 2023-01-21 01:21:03 +02:00
Gabriel Adrian Samfira
ee06fe7384
Add lint, fmt and install-lint-deps targets
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-21 00:41:33 +02:00
Gabriel Adrian Samfira
3aef31d0f0
Make verify-vendor message more friendly
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-21 00:26:20 +02:00
Gabriel
e96caf4711
Merge pull request #66 from gabriel-samfira/add-aditional-checks
Fix linting errors and add workflow to check
2023-01-21 00:01:55 +02:00
Gabriel Adrian Samfira
70fd3f47d0
Remove unused functions
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 23:57:03 +02:00
Gabriel Adrian Samfira
4063265478
Add checks for gofmt
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 23:43:48 +02:00
Gabriel Adrian Samfira
38b2ef5966
Add linters workflow
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 22:26:52 +02:00
Gabriel Adrian Samfira
b354cedf7e
Fixed a bunch of linting issues
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 22:21:22 +02:00
Gabriel Adrian Samfira
687c48127b
Fix some linting issues and add verify-vendor
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 21:55:52 +02:00
Gabriel
9c50674cc6
Merge pull request #65 from icadariu/markdown_lint
Markdown lint
2023-01-20 21:34:15 +02:00
Ionut Cadariu
87529281dd
fix: issues after go mod tidy
Signed-off-by: Ionut Cadariu <ionut.cadariu@gmail.com>
2023-01-20 20:24:13 +01:00
Ionut Cadariu
b95e8a394f
doc: fixing grammar and mdlint 2023-01-20 20:20:56 +01:00
Gabriel
4ff7df8f6f
Merge pull request #64 from cloudbase/move-pool-type
Move pool type in params
2023-01-20 17:14:38 +02:00
Gabriel Adrian Samfira
f2cf947c00
Move pool type in params
Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 17:08:15 +02:00
Gabriel
f985dc5379
Merge pull request #63 from gabriel-samfira/add-runner-prefix-to-common-type
Add a common RunnerPrefix type
2023-01-20 12:15:01 +02:00
Gabriel Adrian Samfira
abcc9569bd
Add a common RunnerPrefix type
There are several fields that are common among some of the data
structures in garm. The RunnerPrefix is just one of them. Perhaps we
should move some of the rest in a common type and embed that into the
types that share those fields.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-20 12:12:15 +02:00
Gabriel
4f2808eb23
Merge pull request #62 from mercedes-benz/fix-shortid
fix: shortid generates correctly now
2023-01-19 15:30:43 +02:00
Michael Kuhnt
49762d7f9e
fix: shortid generates correctly now 2023-01-19 14:23:40 +01:00
Gabriel
c92dfb5d20
Merge pull request #57 from mercedes-benz/runner-prefix
feat: allow to configure the runner name
2023-01-19 13:33:59 +02:00
Michael Kuhnt
6af3025743
feat: allow to configure the runner name 2023-01-19 11:13:36 +01:00
Gabriel
b2a22e1e79
Merge pull request #56 from mercedes-benz/scale-down
feat: scale down idle runners
2023-01-12 12:33:12 +02:00
Michael Kuhnt
3a46a9d127
feat: scale down idle runners 2023-01-10 18:18:28 +01:00
Gabriel
e28f8e6b9d
Merge pull request #55 from gabriel-samfira/properly-exit-loop
Ensure loop closes properly and provider update
2023-01-08 18:45:45 +02:00
Gabriel Adrian Samfira
b954038624 Ensure loop closes properly and provider update
* Ensure the pool loop exits properly when the pool is not yet in
a running state.
  * Use ListInstances() when cleaning orphaned runners. This ensures
We only run one API call per pool to list instances, instead of running
a GetInstance() for each individual instance we are checking.

Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
2023-01-08 16:40:42 +00:00