Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GOのバージョンをあげる #1046

Closed
xxarupakaxx opened this issue Jul 26, 2023 · 4 comments
Closed

GOのバージョンをあげる #1046

xxarupakaxx opened this issue Jul 26, 2023 · 4 comments
Assignees

Comments

@xxarupakaxx
Copy link
Contributor

1.17 -> 1.20に挙げる

わからなかったらSysAd/randomかxxarupakaxxにきく

@kaitoyama
Copy link
Contributor

1.18で以下の変更が加わったようなので、
golang/go#47641
以下と同様に
prometheus/procfs#431
%w%vに変更する必要がありそうです。

@kaitoyama
Copy link
Contributor

t.Errorfで使われていた%wの置き換えをしました。
6999548

現状の手元でのmake testでは1.17ではFAILしないのですが1.20ではTestQuestionnaires/InsertQuestionnaire(たまにFAILする)とTestQuestionnaires/UpdateQuestionnaire(基本 FAILする)がFAILしました。
FAILの原因はどちらも

test-anke-to-1  |               Error Trace:    questionnaires_test.go:479
test-anke-to-1  |               Error:          Max difference between 2023-08-04 23:40:22.494758867 +0900 JST m=+30.140549571 and 2023-08-04 23:40:19 +0900 JST allowed is 2s, but difference was 3.494758867s

みたいな感じで、現在時刻とDBの時刻にズレがありすぎるのが理由みたいです。

対応については次のMTGで相談したいです。

@xxarupakaxx
Copy link
Contributor Author

DRAFTでもいいのでPR立ててもらえると嬉しいかも

@xxarupakaxx
Copy link
Contributor Author

使ってるライブラリのバージョン上げたりしたらうまくいったりしないかな

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants