Skip to content

Commit 65cf712

Browse files
upgrade nats libs and golint fixes (#660)
1 parent 93bae99 commit 65cf712

224 files changed

Lines changed: 25952 additions & 6538 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.golangci.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ linters:
1616
- legacy
1717
- std-error-handling
1818
paths:
19-
- third_party$
19+
- third_party
2020
- builtin$
2121
- examples$
2222
formatters:

go.mod

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
module knative.dev/eventing-natss
22

3-
go 1.23.0
3+
go 1.24.4
44

55
require (
66
github.com/cloudevents/sdk-go/protocol/nats_jetstream/v2 v2.15.2
@@ -10,9 +10,9 @@ require (
1010
github.com/hashicorp/go-cleanhttp v0.5.2
1111
github.com/hashicorp/golang-lru v1.0.2
1212
github.com/kelseyhightower/envconfig v1.4.0
13-
github.com/nats-io/nats-server/v2 v2.10.18
14-
github.com/nats-io/nats.go v1.36.0
15-
github.com/nats-io/nkeys v0.4.7
13+
github.com/nats-io/nats-server/v2 v2.11.6
14+
github.com/nats-io/nats.go v1.43.0
15+
github.com/nats-io/nkeys v0.4.11
1616
github.com/pkg/errors v0.9.1 // indirect
1717
go.opencensus.io v0.24.0
1818
go.uber.org/zap v1.27.0
@@ -64,18 +64,19 @@ require (
6464
github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
6565
github.com/golang/protobuf v1.5.4 // indirect
6666
github.com/google/gnostic-models v0.6.9 // indirect
67+
github.com/google/go-tpm v0.9.5 // indirect
6768
github.com/google/gofuzz v1.2.0 // indirect
6869
github.com/grpc-ecosystem/grpc-gateway/v2 v2.26.1 // indirect
6970
github.com/inconshreveable/mousetrap v1.1.0 // indirect
7071
github.com/josharian/intern v1.0.0 // indirect
7172
github.com/json-iterator/go v1.1.12 // indirect
72-
github.com/klauspost/compress v1.17.11 // indirect
73+
github.com/klauspost/compress v1.18.0 // indirect
7374
github.com/mailru/easyjson v0.9.0 // indirect
7475
github.com/minio/highwayhash v1.0.3 // indirect
7576
github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
7677
github.com/modern-go/reflect2 v1.0.2 // indirect
7778
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
78-
github.com/nats-io/jwt/v2 v2.5.8 // indirect
79+
github.com/nats-io/jwt/v2 v2.7.4 // indirect
7980
github.com/nats-io/nuid v1.0.1 // indirect
8081
github.com/openzipkin/zipkin-go v0.4.3 // indirect
8182
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
@@ -93,15 +94,15 @@ require (
9394
go.uber.org/atomic v1.10.0 // indirect
9495
go.uber.org/automaxprocs v1.6.0 // indirect
9596
go.uber.org/multierr v1.11.0 // indirect
96-
golang.org/x/crypto v0.38.0 // indirect
97-
golang.org/x/mod v0.24.0 // indirect
97+
golang.org/x/crypto v0.39.0 // indirect
98+
golang.org/x/mod v0.25.0 // indirect
9899
golang.org/x/net v0.40.0 // indirect
99100
golang.org/x/oauth2 v0.26.0 // indirect
100-
golang.org/x/sync v0.14.0 // indirect
101+
golang.org/x/sync v0.15.0 // indirect
101102
golang.org/x/sys v0.33.0 // indirect
102103
golang.org/x/term v0.32.0 // indirect
103-
golang.org/x/text v0.25.0 // indirect
104-
golang.org/x/time v0.10.0 // indirect
104+
golang.org/x/text v0.26.0 // indirect
105+
golang.org/x/time v0.12.0 // indirect
105106
golang.org/x/tools v0.33.0 // indirect
106107
gomodules.xyz/jsonpatch/v2 v2.5.0 // indirect
107108
google.golang.org/api v0.198.0 // indirect

go.sum

Lines changed: 24 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,8 @@ github.com/alecthomas/units v0.0.0-20190717042225-c3de453c63f4/go.mod h1:ybxpYRF
6565
github.com/alecthomas/units v0.0.0-20190924025748-f65c72e2690d/go.mod h1:rBZYJk541a8SKzHPHnH3zbiI+7dagKZ0cgpgrD7Fyho=
6666
github.com/alecthomas/units v0.0.0-20211218093645-b94a6e3cc137/go.mod h1:OMCwj8VM1Kc9e19TLln2VL61YJF0x1XFtfdL4JdbSyE=
6767
github.com/antihax/optional v1.0.0/go.mod h1:uupD/76wgC+ih3iEmQUL+0Ugr19nfwCT1kdvxnR2qWY=
68+
github.com/antithesishq/antithesis-sdk-go v0.4.3-default-no-op h1:+OSa/t11TFhqfrX0EOSqQBDJ0YlpmK0rDSiB19dg9M0=
69+
github.com/antithesishq/antithesis-sdk-go v0.4.3-default-no-op/go.mod h1:IUpT2DPAKh6i/YhSbt6Gl3v2yvUZjmKncl7U91fup7E=
6870
github.com/antlr/antlr4/runtime/Go/antlr v1.4.10 h1:yL7+Jz0jTC6yykIK/Wh74gnTJnrGr5AyrNMXuA0gves=
6971
github.com/antlr/antlr4/runtime/Go/antlr v1.4.10/go.mod h1:F7bn7fEU90QkQ3tnmaTx3LTKLEDqnwWODIYppRQ5hnY=
7072
github.com/armon/circbuf v0.0.0-20150827004946-bbbad097214e/go.mod h1:3U/XgcO3hCbHZ8TKRvWD2dDTCfh9M9ya+I9JpbB7O8o=
@@ -254,6 +256,8 @@ github.com/google/go-cmp v0.5.8/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeN
254256
github.com/google/go-cmp v0.5.9/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
255257
github.com/google/go-cmp v0.7.0 h1:wk8382ETsv4JYUZwIsn6YpYiWiBsYLSJiTsyBybVuN8=
256258
github.com/google/go-cmp v0.7.0/go.mod h1:pXiqmnSA92OHEEa9HXL2W4E7lf9JzCmGVUdgjX3N/iU=
259+
github.com/google/go-tpm v0.9.5 h1:ocUmnDebX54dnW+MQWGQRbdaAcJELsa6PqZhJ48KwVU=
260+
github.com/google/go-tpm v0.9.5/go.mod h1:h9jEsEECg7gtLis0upRBQU+GhYVH6jMjrFxI8u6bVUY=
257261
github.com/google/gofuzz v1.0.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg=
258262
github.com/google/gofuzz v1.2.0 h1:xRy4A+RhZaiKjJ1bPfwQ8sedCA+YS2YcCHW6ec7JMi0=
259263
github.com/google/gofuzz v1.2.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg=
@@ -352,8 +356,8 @@ github.com/kelseyhightower/envconfig v1.4.0 h1:Im6hONhd3pLkfDFsbRgu68RDNkGF1r3dv
352356
github.com/kelseyhightower/envconfig v1.4.0/go.mod h1:cccZRl6mQpaq41TPp5QxidR+Sa3axMbJDNb//FQX6Gg=
353357
github.com/kisielk/errcheck v1.5.0/go.mod h1:pFxgyoBC7bSaBwPgfKdkLd5X25qrDl4LWUI2bnpBCr8=
354358
github.com/kisielk/gotool v1.0.0/go.mod h1:XhKaO+MFFWcvkIS/tQcRk01m1F5IRFswLeQ+oQHNcck=
355-
github.com/klauspost/compress v1.17.11 h1:In6xLpyWOi1+C7tXUUWv2ot1QvBjxevKAaI6IXrJmUc=
356-
github.com/klauspost/compress v1.17.11/go.mod h1:pMDklpSncoRMuLFrf1W9Ss9KT+0rH90U12bZKk7uwG0=
359+
github.com/klauspost/compress v1.18.0 h1:c/Cqfb0r+Yi+JtIEq73FWXVkRonBlf0CRNYc8Zttxdo=
360+
github.com/klauspost/compress v1.18.0/go.mod h1:2Pp+KzxcywXVXMr50+X0Q/Lsb43OQHYWRCY2AiWywWQ=
357361
github.com/konsorten/go-windows-terminal-sequences v1.0.1/go.mod h1:T0+1ngSBFLxvqU3pZ+m/2kptfBszLMUkC4ZK/EgS/cQ=
358362
github.com/konsorten/go-windows-terminal-sequences v1.0.3/go.mod h1:T0+1ngSBFLxvqU3pZ+m/2kptfBszLMUkC4ZK/EgS/cQ=
359363
github.com/kr/fs v0.1.0/go.mod h1:FFnZGqtBN9Gxj7eW1uZ42v5BccTP0vu6NEaFoC2HwRg=
@@ -414,14 +418,14 @@ github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 h1:C3w9PqII01/Oq
414418
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822/go.mod h1:+n7T8mK8HuQTcFwEeznm/DIxMOiR9yIdICNftLE1DvQ=
415419
github.com/mwitkow/go-conntrack v0.0.0-20161129095857-cc309e4a2223/go.mod h1:qRWi+5nqEBWmkhHvq77mSJWrCKwh8bxhgT7d/eI7P4U=
416420
github.com/mwitkow/go-conntrack v0.0.0-20190716064945-2f068394615f/go.mod h1:qRWi+5nqEBWmkhHvq77mSJWrCKwh8bxhgT7d/eI7P4U=
417-
github.com/nats-io/jwt/v2 v2.5.8 h1:uvdSzwWiEGWGXf+0Q+70qv6AQdvcvxrv9hPM0RiPamE=
418-
github.com/nats-io/jwt/v2 v2.5.8/go.mod h1:ZdWS1nZa6WMZfFwwgpEaqBV8EPGVgOTDHN/wTbz0Y5A=
419-
github.com/nats-io/nats-server/v2 v2.10.18 h1:tRdZmBuWKVAFYtayqlBB2BuCHNGAQPvoQIXOKwU3WSM=
420-
github.com/nats-io/nats-server/v2 v2.10.18/go.mod h1:97Qyg7YydD8blKlR8yBsUlPlWyZKjA7Bp5cl3MUE9K8=
421-
github.com/nats-io/nats.go v1.36.0 h1:suEUPuWzTSse/XhESwqLxXGuj8vGRuPRoG7MoRN/qyU=
422-
github.com/nats-io/nats.go v1.36.0/go.mod h1:Ubdu4Nh9exXdSz0RVWRFBbRfrbSxOYd26oF0wkWclB8=
423-
github.com/nats-io/nkeys v0.4.7 h1:RwNJbbIdYCoClSDNY7QVKZlyb/wfT6ugvFCiKy6vDvI=
424-
github.com/nats-io/nkeys v0.4.7/go.mod h1:kqXRgRDPlGy7nGaEDMuYzmiJCIAAWDK0IMBtDmGD0nc=
421+
github.com/nats-io/jwt/v2 v2.7.4 h1:jXFuDDxs/GQjGDZGhNgH4tXzSUK6WQi2rsj4xmsNOtI=
422+
github.com/nats-io/jwt/v2 v2.7.4/go.mod h1:me11pOkwObtcBNR8AiMrUbtVOUGkqYjMQZ6jnSdVUIA=
423+
github.com/nats-io/nats-server/v2 v2.11.6 h1:4VXRjbTUFKEB+7UoaKL3F5Y83xC7MxPoIONOnGgpkHw=
424+
github.com/nats-io/nats-server/v2 v2.11.6/go.mod h1:2xoztlcb4lDL5Blh1/BiukkKELXvKQ5Vy29FPVRBUYs=
425+
github.com/nats-io/nats.go v1.43.0 h1:uRFZ2FEoRvP64+UUhaTokyS18XBCR/xM2vQZKO4i8ug=
426+
github.com/nats-io/nats.go v1.43.0/go.mod h1:iRWIPokVIFbVijxuMQq4y9ttaBTMe0SFdlZfMDd+33g=
427+
github.com/nats-io/nkeys v0.4.11 h1:q44qGV008kYd9W1b1nEBkNzvnWxtRSQ7A8BoqRrcfa0=
428+
github.com/nats-io/nkeys v0.4.11/go.mod h1:szDimtgmfOi9n25JpfIdGw12tZFYXqhGxjhVxsatHVE=
425429
github.com/nats-io/nuid v1.0.1 h1:5iA8DT8V7q8WK2EScv2padNa/rTESc1KdnPw4TC2paw=
426430
github.com/nats-io/nuid v1.0.1/go.mod h1:19wcPz3Ph3q0Jbyiqsd0kePYG7A95tJPxeL+1OSON2c=
427431
github.com/nxadm/tail v1.4.4/go.mod h1:kenIhsEOeOJmVchQTgglprH7qJGnHDVpk1VPCcaMI8A=
@@ -605,8 +609,8 @@ golang.org/x/crypto v0.0.0-20200622213623-75b288015ac9/go.mod h1:LzIPMQfyMNhhGPh
605609
golang.org/x/crypto v0.0.0-20210817164053-32db794688a5/go.mod h1:GvvjBRRGRdwPK5ydBHafDWAxML/pGHZbMvKqRZ5+Abc=
606610
golang.org/x/crypto v0.0.0-20210921155107-089bfa567519/go.mod h1:GvvjBRRGRdwPK5ydBHafDWAxML/pGHZbMvKqRZ5+Abc=
607611
golang.org/x/crypto v0.19.0/go.mod h1:Iy9bg/ha4yyC70EfRS8jz+B6ybOBKMaSxLj6P6oBDfU=
608-
golang.org/x/crypto v0.38.0 h1:jt+WWG8IZlBnVbomuhg2Mdq0+BBQaHbtqHEFEigjUV8=
609-
golang.org/x/crypto v0.38.0/go.mod h1:MvrbAqul58NNYPKnOra203SB9vpuZW0e+RRZV+Ggqjw=
612+
golang.org/x/crypto v0.39.0 h1:SHs+kF4LP+f+p14esP5jAoDpHU8Gu/v9lFRK6IT5imM=
613+
golang.org/x/crypto v0.39.0/go.mod h1:L+Xg3Wf6HoL4Bn4238Z6ft6KfEpN0tJGo53AAPC632U=
610614
golang.org/x/exp v0.0.0-20190121172915-509febef88a4/go.mod h1:CJ0aWSM057203Lf6IL+f9T1iT9GByDxfZKAQTCR3kQA=
611615
golang.org/x/exp v0.0.0-20190306152737-a1d7652674e8/go.mod h1:CJ0aWSM057203Lf6IL+f9T1iT9GByDxfZKAQTCR3kQA=
612616
golang.org/x/exp v0.0.0-20190510132918-efd6b22b2522/go.mod h1:ZjyILWgesfNpC6sMxTJOJm9Kp84zZh5NQWvqDGG3Qr8=
@@ -645,8 +649,8 @@ golang.org/x/mod v0.4.2/go.mod h1:s0Qsj1ACt9ePp/hMypM3fl4fZqREWJwdYDEqhRiZZUA=
645649
golang.org/x/mod v0.5.0/go.mod h1:5OXOZSfqPIIbmVBIIKWRFfZjPR0E5r58TLhUjH0a2Ro=
646650
golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4/go.mod h1:jJ57K6gSWd91VN4djpZkiMVwK6gcyfeH4XE8wZrZaV4=
647651
golang.org/x/mod v0.8.0/go.mod h1:iBbtSCu2XBx23ZKBPSOrRkjjQPZFPuis4dIYUhu/chs=
648-
golang.org/x/mod v0.24.0 h1:ZfthKaKaT4NrhGVZHO1/WDTwGES4De8KtWO0SIbNJMU=
649-
golang.org/x/mod v0.24.0/go.mod h1:IXM97Txy2VM4PJ3gI61r1YEk/gAj6zAHN3AdZt6S9Ww=
652+
golang.org/x/mod v0.25.0 h1:n7a+ZbQKQA/Ysbyb0/6IbB1H/X41mKgbhfv7AfG/44w=
653+
golang.org/x/mod v0.25.0/go.mod h1:IXM97Txy2VM4PJ3gI61r1YEk/gAj6zAHN3AdZt6S9Ww=
650654
golang.org/x/net v0.0.0-20180724234803-3673e40ba225/go.mod h1:mL1N/T3taQHkDXs73rZJwtUhF3w3ftmwwsq0BUmARs4=
651655
golang.org/x/net v0.0.0-20180826012351-8a410e7b638d/go.mod h1:mL1N/T3taQHkDXs73rZJwtUhF3w3ftmwwsq0BUmARs4=
652656
golang.org/x/net v0.0.0-20180906233101-161cd47e91fd/go.mod h1:mL1N/T3taQHkDXs73rZJwtUhF3w3ftmwwsq0BUmARs4=
@@ -733,8 +737,8 @@ golang.org/x/sync v0.0.0-20210220032951-036812b2e83c/go.mod h1:RxMgew5VJxzue5/jJ
733737
golang.org/x/sync v0.0.0-20220601150217-0de741cfad7f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
734738
golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
735739
golang.org/x/sync v0.1.0/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
736-
golang.org/x/sync v0.14.0 h1:woo0S4Yywslg6hp4eUFjTVOyKt0RookbpAHG4c1HmhQ=
737-
golang.org/x/sync v0.14.0/go.mod h1:1dzgHSNfp02xaA81J2MS99Qcpr2w7fw1gpm99rleRqA=
740+
golang.org/x/sync v0.15.0 h1:KWH3jNZsfyT6xfAfKiz6MRNmd46ByHDYaZ7KSkCtdW8=
741+
golang.org/x/sync v0.15.0/go.mod h1:1dzgHSNfp02xaA81J2MS99Qcpr2w7fw1gpm99rleRqA=
738742
golang.org/x/sys v0.0.0-20180823144017-11551d06cbcc/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
739743
golang.org/x/sys v0.0.0-20180830151530-49385e6e1522/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
740744
golang.org/x/sys v0.0.0-20180905080454-ebe1bf3edb33/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
@@ -838,13 +842,13 @@ golang.org/x/text v0.3.7/go.mod h1:u+2+/6zg+i71rQMx5EYifcz6MCKuco9NR6JIITiCfzQ=
838842
golang.org/x/text v0.7.0/go.mod h1:mrYo+phRRbMaCq/xk9113O4dZlRixOauAjOtrjsXDZ8=
839843
golang.org/x/text v0.9.0/go.mod h1:e1OnstbJyHTd6l/uOt8jFFHp6TRDWZR/bV3emEE/zU8=
840844
golang.org/x/text v0.14.0/go.mod h1:18ZOQIKpY8NJVqYksKHtTdi31H5itFRjB5/qKTNYzSU=
841-
golang.org/x/text v0.25.0 h1:qVyWApTSYLk/drJRO5mDlNYskwQznZmkpV2c8q9zls4=
842-
golang.org/x/text v0.25.0/go.mod h1:WEdwpYrmk1qmdHvhkSTNPm3app7v4rsT8F2UD6+VHIA=
845+
golang.org/x/text v0.26.0 h1:P42AVeLghgTYr4+xUnTRKDMqpar+PtX7KWuNQL21L8M=
846+
golang.org/x/text v0.26.0/go.mod h1:QK15LZJUUQVJxhz7wXgxSy/CJaTFjd0G+YLonydOVQA=
843847
golang.org/x/time v0.0.0-20181108054448-85acf8d2951c/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=
844848
golang.org/x/time v0.0.0-20190308202827-9d24e82272b4/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=
845849
golang.org/x/time v0.0.0-20191024005414-555d28b269f0/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=
846-
golang.org/x/time v0.10.0 h1:3usCWA8tQn0L8+hFJQNgzpWbd89begxN66o1Ojdn5L4=
847-
golang.org/x/time v0.10.0/go.mod h1:3BpzKBy/shNhVucY/MWOyx10tF3SFh9QdLuxbVysPQM=
850+
golang.org/x/time v0.12.0 h1:ScB/8o8olJvc+CQPWrK3fPZNfh7qgwCrY0zJmoEQLSE=
851+
golang.org/x/time v0.12.0/go.mod h1:CDIdPxbZBQxdj6cxyCIdrNogrJKMJ7pr37NYpMcMDSg=
848852
golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
849853
golang.org/x/tools v0.0.0-20190114222345-bf090417da8b/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
850854
golang.org/x/tools v0.0.0-20190226205152-f727befe758c/go.mod h1:9Yl7xja0Znq3iFh3HoIrodX9oNMXvdceNzlUR8zjMvY=

pkg/channel/jetstream/controller/resources/dispatcher.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -86,15 +86,15 @@ func (b *DispatcherDeploymentBuilder) WithArgs(args *DispatcherDeploymentArgs) *
8686
func (b *DispatcherDeploymentBuilder) Build() *v1.Deployment {
8787
replicas := b.args.Replicas
8888

89-
b.deployment.ObjectMeta.Namespace = b.args.DispatcherNamespace
90-
b.deployment.ObjectMeta.OwnerReferences = []metav1.OwnerReference{b.args.OwnerRef}
91-
b.deployment.ObjectMeta.Annotations = commonconfig.JoinStringMaps(b.deployment.ObjectMeta.Annotations, b.args.DeploymentAnnotations)
92-
b.deployment.ObjectMeta.Labels = commonconfig.JoinStringMaps(b.deployment.ObjectMeta.Labels, b.args.DeploymentLabels)
89+
b.deployment.Namespace = b.args.DispatcherNamespace
90+
b.deployment.OwnerReferences = []metav1.OwnerReference{b.args.OwnerRef}
91+
b.deployment.Annotations = commonconfig.JoinStringMaps(b.deployment.Annotations, b.args.DeploymentAnnotations)
92+
b.deployment.Labels = commonconfig.JoinStringMaps(b.deployment.Labels, b.args.DeploymentLabels)
9393

9494
b.deployment.Spec.Replicas = &replicas
9595
defaultAnnotations := map[string]string{constants.ConfigMapHashAnnotationKey: b.args.ConfigMapHash}
96-
b.deployment.Spec.Template.ObjectMeta.Annotations = commonconfig.JoinStringMaps(defaultAnnotations, b.args.PodAnnotations)
97-
b.deployment.Spec.Template.ObjectMeta.Labels = commonconfig.JoinStringMaps(b.deployment.Spec.Template.ObjectMeta.Labels, b.args.PodLabels)
96+
b.deployment.Spec.Template.Annotations = commonconfig.JoinStringMaps(defaultAnnotations, b.args.PodAnnotations)
97+
b.deployment.Spec.Template.Labels = commonconfig.JoinStringMaps(b.deployment.Spec.Template.Labels, b.args.PodLabels)
9898
b.deployment.Spec.Template.Spec.ServiceAccountName = b.args.ServiceAccount
9999

100100
b.deployment.Spec.Template.Spec.NodeSelector = b.args.DeploymentNodeSelector
@@ -117,7 +117,7 @@ func (b *DispatcherDeploymentBuilder) Build() *v1.Deployment {
117117
if b.args.ConfigMapName != "" {
118118
for i, v := range b.deployment.Spec.Template.Spec.Volumes {
119119
if v.Name == constants.SettingsConfigMapName {
120-
b.deployment.Spec.Template.Spec.Volumes[i].VolumeSource.ConfigMap.Name = b.args.ConfigMapName
120+
b.deployment.Spec.Template.Spec.Volumes[i].ConfigMap.Name = b.args.ConfigMapName
121121

122122
break
123123
}

pkg/channel/jetstream/controller/resources/dispatcher_service.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,9 @@ func (b *DispatcherServiceBuilder) WithArgs(args *DispatcherServiceArgs) *Dispat
5959
}
6060

6161
func (b *DispatcherServiceBuilder) Build() *corev1.Service {
62-
b.service.ObjectMeta.Namespace = b.args.DispatcherNamespace
63-
b.service.ObjectMeta.Annotations = commonconfig.JoinStringMaps(b.service.Annotations, b.args.ServiceAnnotations)
64-
b.service.ObjectMeta.Labels = commonconfig.JoinStringMaps(b.service.Labels, b.args.ServiceLabels)
62+
b.service.Namespace = b.args.DispatcherNamespace
63+
b.service.Annotations = commonconfig.JoinStringMaps(b.service.Annotations, b.args.ServiceAnnotations)
64+
b.service.Labels = commonconfig.JoinStringMaps(b.service.Labels, b.args.ServiceLabels)
6565
return b.service
6666
}
6767

pkg/channel/jetstream/controller/resources/service.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ func MakeK8sService(kc *v1alpha1.NatsJetStreamChannel, opts ...ServiceOption) (*
6565
Kind: "Service",
6666
},
6767
ObjectMeta: metav1.ObjectMeta{
68-
Name: MakeJSMChannelServiceName(kc.ObjectMeta.Name),
68+
Name: MakeJSMChannelServiceName(kc.Name),
6969
Namespace: kc.Namespace,
7070
Labels: map[string]string{
7171
MessagingRoleLabel: MessagingRole,

pkg/channel/jetstream/dispatcher/dispatcher_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,11 @@ import (
4343

4444
"knative.dev/eventing-natss/pkg/apis/messaging/v1alpha1"
4545
"knative.dev/eventing-natss/pkg/channel/jetstream/utils"
46-
reconciletesting "knative.dev/eventing-natss/pkg/reconciler/testing"
4746
"knative.dev/eventing/pkg/channel"
4847
)
4948

5049
func TestDispatcher_RegisterChannelHost(t *testing.T) {
51-
nc := reconciletesting.NewNatsJetStreamChannel(testNS, ncName)
50+
nc := reconcilertesting.NewNatsJetStreamChannel(testNS, ncName)
5251
config := createChannelConfig(nc)
5352

5453
d := &Dispatcher{}
@@ -116,7 +115,7 @@ func TestDispatcher_ReconcileConsumers(t *testing.T) {
116115
eventRecorder := record.NewFakeRecorder(10)
117116
ctx = controller.WithEventRecorder(ctx, eventRecorder)
118117

119-
nc := reconciletesting.NewNatsJetStreamChannel(testNS, ncName, reconciletesting.WithNatsJetStreamChannelSubscribers(subscribers))
118+
nc := reconcilertesting.NewNatsJetStreamChannel(testNS, ncName, reconcilertesting.WithNatsJetStreamChannelSubscribers(subscribers))
120119
sub := fanout.Subscription{
121120
RetryConfig: &kncloudevents.RetryConfig{
122121
RequestTimeout: time.Second,

pkg/channel/jetstream/dispatcher/reconciler.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -285,9 +285,9 @@ func (r *Reconciler) newConfigFromChannel(nc *v1alpha1.NatsJetStreamChannel) Cha
285285
HostName: nc.Status.Address.URL.Host,
286286
ConsumerConfigTemplate: nc.Spec.ConsumerConfigTemplate,
287287
}
288-
if nc.Spec.SubscribableSpec.Subscribers != nil {
289-
newSubs := make([]Subscription, len(nc.Spec.SubscribableSpec.Subscribers))
290-
for i, source := range nc.Spec.SubscribableSpec.Subscribers {
288+
if nc.Spec.Subscribers != nil {
289+
newSubs := make([]Subscription, len(nc.Spec.Subscribers))
290+
for i, source := range nc.Spec.Subscribers {
291291
innerSub, _ := fanout.SubscriberSpecToFanoutConfig(source)
292292

293293
// This functionality cannot be configured via the Subscription CRD. The default implementation is

pkg/reconciler/testing/factory.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ import (
3636
"knative.dev/pkg/logging"
3737
logtesting "knative.dev/pkg/logging/testing"
3838
"knative.dev/pkg/reconciler"
39-
. "knative.dev/pkg/reconciler/testing"
39+
reconcilertesting "knative.dev/pkg/reconciler/testing"
4040
)
4141

4242
const (
@@ -49,8 +49,8 @@ const (
4949
type Ctor func(context.Context, *Listers) controller.Reconciler
5050

5151
// MakeFactory creates a reconciler factory with fake clients and controller created by `ctor`.
52-
func MakeFactory(ctor Ctor) Factory {
53-
return func(t *testing.T, r *TableRow) (controller.Reconciler, ActionRecorderList, EventList) {
52+
func MakeFactory(ctor Ctor) reconcilertesting.Factory {
53+
return func(t *testing.T, r *reconcilertesting.TableRow) (controller.Reconciler, reconcilertesting.ActionRecorderList, reconcilertesting.EventList) {
5454
ls := NewListers(r.Objects)
5555

5656
ctx := logging.WithLogger(context.Background(), logtesting.TestLogger(t))
@@ -77,14 +77,14 @@ func MakeFactory(ctor Ctor) Factory {
7777

7878
// Validate all Create operations through the eventing client.
7979
client.PrependReactor("create", "*", func(action clientgotesting.Action) (handled bool, ret runtime.Object, err error) {
80-
return ValidateCreates(context.Background(), action)
80+
return reconcilertesting.ValidateCreates(context.Background(), action)
8181
})
8282
client.PrependReactor("update", "*", func(action clientgotesting.Action) (handled bool, ret runtime.Object, err error) {
83-
return ValidateUpdates(context.Background(), action)
83+
return reconcilertesting.ValidateUpdates(context.Background(), action)
8484
})
8585

86-
actionRecorderList := ActionRecorderList{dynamicClient, client, kubeClient}
87-
eventList := EventList{Recorder: eventRecorder}
86+
actionRecorderList := reconcilertesting.ActionRecorderList{dynamicClient, client, kubeClient}
87+
eventList := reconcilertesting.EventList{Recorder: eventRecorder}
8888

8989
if r.Ctx == nil {
9090
r.Ctx = ctx

pkg/reconciler/testing/natsjetstreamchannel.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ func WithNatsJetStreamChannelFinalizer(nc *v1alpha1.NatsJetStreamChannel) {
7878

7979
func WithNatsJetStreamChannelDeleted(nc *v1alpha1.NatsJetStreamChannel) {
8080
deleteTime := metav1.NewTime(time.Unix(1e9, 0))
81-
nc.ObjectMeta.SetDeletionTimestamp(&deleteTime)
81+
nc.SetDeletionTimestamp(&deleteTime)
8282
}
8383

8484
func WithNatsJetStreamDeploymentSpecTemplate(nc *v1alpha1.NatsJetStreamChannel) {

0 commit comments

Comments
 (0)