Skip to content

Commit 6cca5ae

Browse files
committed
refactor for updating
1 parent 7751945 commit 6cca5ae

File tree

18 files changed

+244
-37
lines changed

18 files changed

+244
-37
lines changed

3rdmocks/ServerStream.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Makefile

+1-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ grpc:
1818
./rpc/gen/core.proto
1919

2020
deps:
21-
env GO111MODULE=on go mod download
22-
env GO111MODULE=on go mod vendor
21+
go mod vendor
2322

2423
binary:
2524
CGO_ENABLED=0 go build -ldflags "$(GO_LDFLAGS)" -gcflags=all=-G=3 -o eru-core

cluster/mocks/Cluster.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

engine/docker/mocks/APIClient.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

engine/mocks/API.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lock/mocks/DistributedLock.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

metrics/metrics.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -93,22 +93,22 @@ func (m *Metrics) SendMetrics(metrics ...*resources.Metrics) {
9393
log.Errorf(nil, "[SendMetrics] Collector not found: %s", metric.Name) //nolint
9494
continue
9595
}
96-
switch collector.(type) { // nolint: gocritic
96+
switch collector.(type) { // nolint
9797
case *prometheus.GaugeVec:
9898
value, err := strconv.ParseFloat(metric.Value, 64)
9999
if err != nil {
100100
log.Errorf(nil, "[SendMetrics] Error occurred while parsing %v value %v: %v", metric.Name, metric.Value, err) //nolint
101101
}
102-
collector.(*prometheus.GaugeVec).WithLabelValues(metric.Labels...).Set(value)
102+
collector.(*prometheus.GaugeVec).WithLabelValues(metric.Labels...).Set(value) // nolint
103103
if err := m.gauge(metric.Key, value); err != nil {
104104
log.Errorf(nil, "[SendMetrics] Error occurred while sending %v data to statsd: %v", metric.Name, err) //nolint
105105
}
106106
case *prometheus.CounterVec:
107-
value, err := strconv.ParseInt(metric.Value, 10, 32)
107+
value, err := strconv.ParseInt(metric.Value, 10, 32) // nolint
108108
if err != nil {
109109
log.Errorf(nil, "[SendMetrics] Error occurred while parsing %v value %v: %v", metric.Name, metric.Value, err) //nolint
110110
}
111-
collector.(*prometheus.CounterVec).WithLabelValues(metric.Labels...).Add(float64(value))
111+
collector.(*prometheus.CounterVec).WithLabelValues(metric.Labels...).Add(float64(value)) // nolint
112112
if err := m.count(metric.Key, int(value), 1.0); err != nil {
113113
log.Errorf(nil, "[SendMetrics] Error occurred while sending %v data to statsd: %v", metric.Name, err) //nolint
114114
}

resources/manager.go

-14
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ func NewPluginManager(config types.Config) (*PluginManager, error) {
2525
config: config,
2626
plugins: []Plugin{},
2727
}
28-
2928
return pm, nil
3029
}
3130

@@ -56,19 +55,6 @@ func (pm *PluginManager) GetPlugins() []Plugin {
5655
return pm.plugins
5756
}
5857

59-
func (pm *PluginManager) callPlugins(plugins []Plugin, f func(Plugin)) {
60-
wg := &sync.WaitGroup{}
61-
wg.Add(len(plugins))
62-
63-
for _, plugin := range plugins {
64-
go func(p Plugin) {
65-
defer wg.Done()
66-
f(p)
67-
}(plugin)
68-
}
69-
wg.Wait()
70-
}
71-
7258
func callPlugins[T any](ctx context.Context, plugins []Plugin, f func(Plugin) (T, error)) (map[Plugin]T, error) {
7359
resMap := sync.Map{}
7460
combinedErr := types.NewCombinedErr()

resources/mocks/Plugin.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

rpc/mocks/CoreRPC_RunAndWaitServer.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

source/mocks/Source.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

store/etcdv3/meta/mocks/ETCDClientV3.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

store/etcdv3/meta/mocks/KV.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

store/etcdv3/meta/mocks/Txn.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

store/mocks/Store.go

+16-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

utils/file.go

+1-4
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,7 @@ func ListAllExecutableFiles(basedir string) ([]string, error) {
2323
return nil
2424
})
2525

26-
if err != nil {
27-
return nil, err
28-
}
29-
return files, nil
26+
return files, err
3027
}
3128

3229
func isExecutable(perm fs.FileMode) bool {

0 commit comments

Comments
 (0)