Skip to content

Commit 6ff9722

Browse files
committed
chore: align tooling and add package docs
- bump common to v0.2.1 (gofumpt v0.10 + k8s pkg doc) - bump go directive to 1.25.6 (matches go.work) - add // Package X docs to admission, affinity, metrics, main - gofumpt v0.10: split first arg of multi-line admission/workload_validator call
1 parent 44b5288 commit 6ff9722

7 files changed

Lines changed: 16 additions & 3 deletions

File tree

admission/server.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// Package admission implements the cocoon-webhook mutate and validate
2+
// handlers (pods, workloads, and CocoonSet CRs).
13
package admission
24

35
import (

admission/workload_validator.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,8 @@ func checkScaleDown(ctx context.Context, req *admissionv1.AdmissionRequest, oldR
156156
msg := fmt.Sprintf(
157157
"cocoon-webhook: scale-down blocked for cocoon %s %s/%s (%d -> %d). "+
158158
"Use a CocoonHibernation CR to suspend individual agents.",
159-
req.Kind.Kind, req.Namespace, req.Name, oldReplicas, newReplicas)
159+
req.Kind.Kind, req.Namespace, req.Name, oldReplicas, newReplicas,
160+
)
160161
logger.Warn(ctx, msg)
161162
metrics.RecordAdmission(metrics.HandlerValidate, metrics.DecisionDeny)
162163
return commonadmission.Deny(msg)

affinity/store.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// Package affinity reserves and releases cocoon node-affinity slots,
2+
// persisted as ConfigMaps so the assignment survives webhook restarts.
13
package affinity
24

35
import (

go.mod

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
module github.com/cocoonstack/cocoon-webhook
22

3-
go 1.25.0
3+
go 1.25.6
44

55
require (
6-
github.com/cocoonstack/cocoon-common v0.1.10-0.20260423175920-c8cd308978cb
6+
github.com/cocoonstack/cocoon-common v0.2.1-0.20260506143712-eaa6e94cf830
77
github.com/projecteru2/core v0.0.0-20241016125006-ff909eefe04c
88
github.com/prometheus/client_golang v1.23.2
99
k8s.io/api v0.35.3

go.sum

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@ github.com/cockroachdb/redact v1.1.3 h1:AKZds10rFSIj7qADf0g46UixK8NNLwWTNdCIGS5w
2929
github.com/cockroachdb/redact v1.1.3/go.mod h1:BVNblN9mBWFyMyqK1k3AAiSxhvhfK2oOZZ2lK+dpvRg=
3030
github.com/cocoonstack/cocoon-common v0.1.10-0.20260423175920-c8cd308978cb h1:s31sByTK+6ik1BOaAirpKZ1O9q3OSzT8kAiECci5T0k=
3131
github.com/cocoonstack/cocoon-common v0.1.10-0.20260423175920-c8cd308978cb/go.mod h1:cjBGOeZvsw2J5X7RIQJ44PJx0QiwgypEJVdd5V38arU=
32+
github.com/cocoonstack/cocoon-common v0.2.1-0.20260506141504-3f9473d6ec7c h1:lko3TWi0tqiraB4sK0urz2lxxVjaFNzx2QvRHdlynOw=
33+
github.com/cocoonstack/cocoon-common v0.2.1-0.20260506141504-3f9473d6ec7c/go.mod h1:xIXbJ83vngQ2mrLC6q0Tw7h21M9BYBBqqYTcHaUrm1Y=
34+
github.com/cocoonstack/cocoon-common v0.2.1-0.20260506143712-eaa6e94cf830 h1:L7OIAjWwvBiFH3QwLTJRF1khFW2KjyyZyPKDfotP5NY=
35+
github.com/cocoonstack/cocoon-common v0.2.1-0.20260506143712-eaa6e94cf830/go.mod h1:xIXbJ83vngQ2mrLC6q0Tw7h21M9BYBBqqYTcHaUrm1Y=
3236
github.com/codegangsta/inject v0.0.0-20150114235600-33e0aa1cb7c0/go.mod h1:4Zcjuz89kmFXt9morQgcfYZAYZ5n8WHjt81YYWIwtTM=
3337
github.com/coreos/etcd v3.3.10+incompatible/go.mod h1:uF7uidLiAD3TWHmW31ZFd/JWoc32PjwdhPthX9715RE=
3438
github.com/coreos/go-etcd v2.0.0+incompatible/go.mod h1:Jez6KQU2B/sWsbdaef3ED8NzMklzPG4d5KIOhIy30Tk=

main.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
// Package main is the cocoon-webhook entry point. The webhook handles
2+
// admission review for cocoon pods, workloads, and CocoonSet CRs, and
3+
// owns the per-pool node-affinity reservation store.
14
package main
25

36
import (

metrics/metrics.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// Package metrics exposes cocoon-webhook Prometheus counters.
12
package metrics
23

34
import (

0 commit comments

Comments
 (0)