Skip to content

Commit 3e6a5fc

Browse files
committed
chore(all): Move from types.* to cli.ServiceContext, refs go-orb/go-orb#33
1 parent 9e71807 commit 3e6a5fc

File tree

46 files changed

+638
-324
lines changed

Some content is hidden

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

46 files changed

+638
-324
lines changed

Taskfile.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,20 @@
11
version: "3"
22

33
tasks:
4+
local-lint:
5+
desc: Run "golangci-lint" in all packages with the go.work
6+
cmds:
7+
- |
8+
for i in $(find . -name 'go.mod'); do
9+
if [[ "$i" == "./.github/go.mod" ]]; then
10+
continue
11+
fi
12+
pushd ${i:0:-6} 1>/dev/null
13+
echo $i
14+
golangci-lint run || true
15+
popd 1>/dev/null
16+
done
17+
418
lint:
519
desc: Run "golangci-lint" in all packages
620
cmds:

benchmarks/event/cmd/handler/wire.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ func run(
7373
cli.ProvideSingleServiceContext,
7474
types.ProvideComponents,
7575

76-
cli.ProvideConfigData,
77-
cli.ProvideServiceName,
76+
cli.ProvideAppConfigData,
77+
cli.ProvideServiceConfigData,
7878

7979
log.ProvideNoOpts,
8080

benchmarks/event/cmd/handler/wire_gen.go

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

benchmarks/event/cmd/request/main.go

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import (
1717
"github.com/go-orb/go-orb/config"
1818
"github.com/go-orb/go-orb/event"
1919
"github.com/go-orb/go-orb/log"
20-
"github.com/go-orb/go-orb/types"
2120
_ "github.com/go-orb/plugins/codecs/goccyjson"
2221
_ "github.com/go-orb/plugins/codecs/proto"
2322
_ "github.com/go-orb/plugins/event/natsjs"
@@ -96,9 +95,7 @@ func connection(
9695
//
9796
//nolint:funlen
9897
func bench(
99-
ctx context.Context,
100-
sn types.ServiceName,
101-
configs types.ConfigData,
98+
svcCtx *cli.ServiceContext,
10299
logger log.Logger,
103100
eventHandler event.Type,
104101
) error {
@@ -110,8 +107,7 @@ func bench(
110107
PackageSize: defaultPackageSize,
111108
}
112109

113-
sections := append(types.SplitServiceName(sn), configSection)
114-
if err := config.Parse(sections, configs, &cfg); err != nil {
110+
if err := config.Parse(nil, event.DefaultConfigSection, svcCtx.Config, &cfg); err != nil {
115111
return err
116112
}
117113

@@ -126,7 +122,7 @@ func bench(
126122

127123
runtime.GOMAXPROCS(cfg.Threads)
128124

129-
wCtx, wCancel := context.WithCancel(ctx)
125+
wCtx, wCancel := context.WithCancel(svcCtx.Context())
130126

131127
// Create random bytes to ping-pong on each request.
132128
msg := make([]byte, cfg.PackageSize)
@@ -169,7 +165,7 @@ func bench(
169165
//
170166
logger.Info("Now running the benchmark")
171167

172-
ctx, cancel := context.WithCancel(ctx)
168+
ctx, cancel := context.WithCancel(svcCtx.Context())
173169

174170
// Timer to end requests
175171
time.AfterFunc(time.Second*time.Duration(cfg.Duration), func() {

benchmarks/event/cmd/request/wire.go

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,13 @@ type wireRunResult struct{}
2222

2323
// wireRunCallback is the actual code that runs the business logic.
2424
type wireRunCallback func(
25-
ctx context.Context,
26-
serviceName types.ServiceName,
27-
configs types.ConfigData,
25+
svcCtx *cli.ServiceContext,
2826
logger log.Logger,
2927
eventHandler event.Type,
3028
) error
3129

3230
func wireRun(
33-
serviceName types.ServiceName,
34-
configs types.ConfigData,
31+
serviceContext *cli.ServiceContext,
3532
components *types.Components,
3633
logger log.Logger,
3734
event event.Type,
@@ -49,9 +46,8 @@ func wireRun(
4946
}
5047
}
5148

52-
//
5349
// Actual code
54-
runErr := cb(ctx, serviceName, configs, logger, event)
50+
runErr := cb(serviceContext, logger, event)
5551

5652
// Orb shutdown.
5753
ctx = context.Background()
@@ -78,8 +74,8 @@ func run(
7874
cli.ProvideSingleServiceContext,
7975
types.ProvideComponents,
8076

81-
cli.ProvideConfigData,
82-
cli.ProvideServiceName,
77+
cli.ProvideAppConfigData,
78+
cli.ProvideServiceConfigData,
8379

8480
log.ProvideNoOpts,
8581

benchmarks/event/cmd/request/wire_gen.go

Lines changed: 11 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

benchmarks/rps/cmd/orb-rps-client/wire.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ func wireRun(
6767
return wireRunResult{}, runErr
6868
}
6969

70-
func provideClientConfig(serviceName types.ServiceName, configs types.ConfigData) (*clientConfig, error) {
70+
func provideClientConfig(svcCtx *cli.ServiceContext) (*clientConfig, error) {
7171
cfg := &clientConfig{
7272
BypassRegistry: defaultBypassRegistry,
7373
PoolSize: defaultPoolSize,
@@ -80,7 +80,7 @@ func provideClientConfig(serviceName types.ServiceName, configs types.ConfigData
8080
ContentType: defaultContentType,
8181
}
8282

83-
if err := config.Parse([]string{configSection}, configs, &cfg); err != nil {
83+
if err := config.Parse(nil, configSection, svcCtx.Config, &cfg); err != nil {
8484
return nil, err
8585
}
8686

@@ -103,9 +103,8 @@ func run(
103103
cli.ProvideSingleServiceContext,
104104
types.ProvideComponents,
105105

106-
cli.ProvideConfigData,
107-
cli.ProvideServiceName,
108-
cli.ProvideServiceVersion,
106+
cli.ProvideAppConfigData,
107+
cli.ProvideServiceConfigData,
109108

110109
log.ProvideNoOpts,
111110

benchmarks/rps/cmd/orb-rps-client/wire_gen.go

Lines changed: 10 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

benchmarks/rps/cmd/orb-rps-server/wire.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,8 @@ func run(
126126
cli.ProvideSingleServiceContext,
127127
types.ProvideComponents,
128128

129-
cli.ProvideConfigData,
130-
cli.ProvideServiceName,
131-
cli.ProvideServiceVersion,
129+
cli.ProvideAppConfigData,
130+
cli.ProvideServiceConfigData,
132131

133132
log.ProvideNoOpts,
134133
registry.ProvideNoOpts,

benchmarks/rps/cmd/orb-rps-server/wire_gen.go

Lines changed: 5 additions & 9 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)