Closed kolbe closed 2 years ago
I meet another one:
make
gofmt (simplify)
goimports (if installed)
linting
./tools/check/check-lint.sh
./docker/control/tiup-cluster/cmd/clean.go:18: "io/ioutil" ./docker/control/tiup-cluster/cmd/uninstall.go:18: "io/ioutil" ./docker/control/tiup-cluster/cmd/status.go:17: "io/ioutil" ./docker/control/tiup-cluster/tools/migrate/main.go:19: "io/ioutil" ./docker/control/tiup-cluster/tools/pkger/main.go:20: "io/ioutil" ./docker/control/tiup-cluster/server/package/package.go:22: "io/ioutil" ./docker/control/tiup-cluster/components/cluster/command/scale_out.go:18: "io/ioutil" ./docker/control/tiup-cluster/components/cluster/command/deploy.go:18: "io/ioutil" ./docker/control/tiup-cluster/components/errdoc/errdoc-gen/main.go:98: "io/ioutil" ./docker/control/tiup-cluster/components/playground/monitor.go:20: "io/ioutil" ./docker/control/tiup-cluster/components/playground/playground.go:21: "io/ioutil" ./docker/control/tiup-cluster/components/playground/grafana.go:19: "io/ioutil" ./docker/control/tiup-cluster/components/playground/grafana_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/components/playground/main.go:20: "io/ioutil" ./docker/control/tiup-cluster/components/dm/spec/topology_dm_test.go:18: "io/ioutil" ./docker/control/tiup-cluster/components/dm/spec/alertmanager.go:19: "io/ioutil" ./docker/control/tiup-cluster/components/dm/spec/grafana_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/components/dm/ansible/import.go:20: "io/ioutil" ./docker/control/tiup-cluster/components/dm/ansible/import_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/components/dm/command/import.go:18: "io/ioutil" ./docker/control/tiup-cluster/components/client/main.go:21: "io/ioutil" ./docker/control/tiup-cluster/pkg/cliutil/ssh.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/crypto/ca.go:21: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/embed/embed_test.go:4: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/install/local_install.sh.go:16:import "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/grafana.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/alertmanager.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/blackbox.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/prometheus.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/dm/prometheus.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/dashboard.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/datasource.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/config/tispark.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/systemd/system.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/systemd/tispark.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/drainer.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/tidb.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/pump.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/cdc.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/dm_worker.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/grafana.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/blackbox_exporter.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/tikv.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/alertmanager.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/prometheus.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/dm_master.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/tispark.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/node_exporter.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/tiflash.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/template/scripts/pd.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/drainer.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/tidb.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/pump.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/parse_topology_test.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/spec_manager.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/instance.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/tikv.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/spec_manager_test.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/grafana_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/parse_topology.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/tispark.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/tiflash.go:21: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/spec/pd.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/ansible/import_test.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/ansible/inventory.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/audit/audit.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/task/ssh_keygen.go:22: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/task/env_init.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/api/binlog.go:21: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/executor/local_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/cluster/manager.go:24: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/file_source.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/mirror.go:22: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/v1manifest/repo.go:24: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/v1manifest/local_manifests.go:20: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/store/sync.go:18: "io/ioutil" ./docker/control/tiup-cluster/pkg/repository/store/txn.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/logger/debug.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/localdata/profile.go:22: "io/ioutil" ./docker/control/tiup-cluster/pkg/file/file_test.go:5: "io/ioutil" ./docker/control/tiup-cluster/pkg/file/file.go:4: "io/ioutil" ./docker/control/tiup-cluster/pkg/utils/http_client.go:20: "io/ioutil" ./docker/control/tiup-cluster/pkg/telemetry/telemetry_test.go:19: "io/ioutil" ./docker/control/tiup-cluster/pkg/telemetry/meta_test.go:4: "io/ioutil" ./docker/control/tiup-cluster/pkg/telemetry/scrub_test.go:17: "io/ioutil" ./docker/control/tiup-cluster/pkg/telemetry/meta.go:17: "io/ioutil"
use of "io/ioutil" is deprecated in Go 1.16, please update to use "io" and "os"
make: *** [lint] Error 1
Could you try in the root path of cloned tiup repo: rm -rf *
, and then git checkout .
, and try make
again?
It might due to dirty files in paths excluded by .gitignore
, so that you can't see them with git status
. I think some test files might go into those dirs.
I got this error after completely re-cloning from scratch. Also get it again after following your suggested steps.
On Apr 14, 2021, at 8:37 PM, Allen Zhong @.***> wrote:
Could you try in the root path of cloned tiup repo: rm -rf *, and then git checkout ., and try make again?
It might due to dirty files in paths excluded by .gitignore, so that you can't see them with git status. I think some test files might go into those dirs.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/pingcap/tiup/issues/1310#issuecomment-820033009, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJKEWTLVX4GLUAY2O3JPUTTIZNQTANCNFSM426SCUDA.
Wierd. Then could you show us the git diff
after it complaints about go.sum difference? And retry make
could bypass this check, or using make build
directly without linting stuff could also do the trick.
Yes, make build
worked fine.
Here's the status after the failure:
Files go.sum and /tmp/go.sum.before differ
make: *** [tidy] Error 1
$ git status
On branch master
Your branch is up to date with 'origin/master'.
Changes not staged for commit:
(use "git add <file>..." to update what will be committed)
(use "git restore <file>..." to discard changes in working directory)
modified: components/dm/ansible/import.go
modified: components/dm/command/import.go
modified: components/dm/spec/topology_dm_test.go
modified: components/errdoc/spec/spec.go
modified: go.mod
modified: go.sum
modified: pkg/cliutil/progress/multi_bar.go
modified: pkg/cliutil/progress/single_bar.go
modified: pkg/cluster/ansible/import_test.go
modified: pkg/cluster/ansible/inventory.go
modified: pkg/cluster/api/typeutil/size.go
modified: pkg/cluster/executor/ssh.go
modified: pkg/cluster/manager/edit_config.go
modified: pkg/cluster/manager/manager_test.go
modified: pkg/cluster/spec/parse_topology.go
modified: pkg/cluster/spec/server_config.go
modified: pkg/cluster/spec/server_config_test.go
modified: pkg/cluster/spec/spec_manager.go
modified: pkg/cluster/spec/spec_test.go
modified: pkg/cluster/spec/tiflash.go
modified: pkg/cluster/spec/validate_test.go
modified: pkg/crypto/rsa.go
modified: pkg/repository/progress.go
modified: pkg/telemetry/meta.go
modified: pkg/telemetry/scrub.go
modified: pkg/utils/diff.go
modified: pkg/utils/diff_test.go
no changes added to commit (use "git add" and/or "git commit -a")
[git.diff.txt](https://github.com/pingcap/tiup/files/6315221/git.diff.txt)
Bug Report
Please answer these questions before submitting your issue. Thanks!
make
🙂A successful build!
tiup --version
)?