Skip to content

update Go version for nvcheck building stability #1128

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

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

koron
Copy link
Member

@koron koron commented Aug 13, 2022

nvcheckのbuildがgopkg.inのせいで不安定なのを解消する試み。

Go 1.15がgopkg.inにアクセスした際、gopkg.inがgithub.comにアクセスに行ってそこでrejectされてるっぽい。
(アクセス制限かトークンの利用上限に引っかかってるのかもしれない。それ以上は調べられない)

ただ最新のGoはパッケージの取得にgopkg.inに直接リクエストするわけではなくキャッシュサーバーに問い合わせるはずで、
かつ対象が yaml.v2 という有名なモジュールなのでキャッシュミスは確率が低い。

なのでGoが1.15と古いこと
および取得方法が go install ...@latest ではなく go get の古い方法であること
が上記のトリガーであると推定し、
それを検証するためGoを1.18にし go install ...@latest で取得するように変更してみた。

@koron koron requested a review from tsuyoshicho August 13, 2022 07:44
@koron koron self-assigned this Aug 13, 2022
@koron
Copy link
Member Author

koron commented Aug 13, 2022

とりあえず通った。

直後に以前失敗したモノを Re-run してみたがそちらは失敗した。
https://github.com/vim-jp/vimdoc-ja-working/runs/7817850944?check_suite_focus=true

なので原因はともかく、この書き換えで現象を緩和できる可能性を示せた。

@koron koron marked this pull request as ready for review August 13, 2022 07:48
@tsuyoshicho tsuyoshicho requested review from h-east and k-takata August 13, 2022 07:49
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

nvcheck 側には go.mod 対応していただいているので、この修正を行うのが良いと考えます

@h-east san, @k-takata san
どちらかApproveしていただけたらマージ、でどうでしょうか?

追記:
他でも出ています(がたまに通ったりしますが)
現在は以下のPRが、これにより失敗しています。
#1112

@tsuyoshicho
Copy link
Contributor

@k-takata もしご意見があれば8/15日内におねがいします。
とくなければマージしようと考えています。

@k-takata k-takata merged commit 5317acc into master Aug 15, 2022
@k-takata k-takata deleted the update-go-for-nvcheck-stability branch August 15, 2022 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants