temporalio / temporalio/temporal
import of github.com/cactus/go-statsd-client is ambiguous in go.temporal.io/server
@Ardagan is already working on this.
Since Jan 21, 2022.
- Dominant language
- Go
- Stars
- 23.2k
- Forks
- 1.9k
- Avg merge
- 2d 8h
- Merged PRs (30d)
- 228
Description
Expected Behavior
updating packages for embedded temporal , from go.temporal.io/server v1.9.2 to go.temporal.io/server v1.14.3
go mod tidy works fine
Actual Behavior
go mod tidy fails
temp_test imports
go.temporal.io/server/common/authorization imports
go.temporal.io/server/common/metrics imports
github.com/cactus/go-statsd-client/statsd loaded from github.com/cactus/go-statsd-client/statsd@v0.0.0-20200423205355-cb0885a1018c,
but go 1.16 would fail to locate it:
ambiguous import: found package github.com/cactus/go-statsd-client/statsd in multiple modules:
github.com/cactus/go-statsd-client v3.1.1+incompatible (.../go/pkg/mod/github.com/cactus/go-statsd-client@v3.1.1+incompatible/statsd)
github.com/cactus/go-statsd-client/statsd v0.0.0-20200423205355-cb0885a1018c (../go/pkg/mod/github.com/cactus/go-statsd-client/statsd@v0.0.0-20200423205355-cb0885a1018c)
Steps to Reproduce the Problem
-
use go.temporal.io/server v1.9.2 in code
sample:
package mainimport (
"go.temporal.io/server/common/authorization"
"go.temporal.io/server/common/config"
"log"
)func main() {
cfg := config.Authorization{}
_, err := authorization.GetAuthorizerFromConfig(&cfg)
if err != nil {
log.Println(err)
}
} -
go get -u go.temporal.io/server
-
go mod tidy
Specifications
- Version: go version go1.17.5
- Platform: ubuntu 20 (same on windows)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.