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

ci: fix GOPATH, PATH, and GOCACHE #512

Merged
merged 1 commit into from
Apr 14, 2023
Merged

ci: fix GOPATH, PATH, and GOCACHE #512

merged 1 commit into from
Apr 14, 2023

Conversation

jakubgs
Copy link
Contributor

@jakubgs jakubgs commented Apr 14, 2023

Using $HOME/go is bad and you should feel bad.
Also, adding /usr/loca/go/bin is a terrible idea.
And GOCACHE is poisoning stuff for linter.

@jakubgs jakubgs self-assigned this Apr 14, 2023
@status-im-auto
Copy link

status-im-auto commented Apr 14, 2023

Jenkins Builds

Click to see older builds (24)
Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ d33f7aa #1 2023-04-14 13:47:04 ~58 sec tests 📄log
✔️ d33f7aa #1 2023-04-14 13:47:08 ~1 min linux 📦deb
✔️ d33f7aa #1 2023-04-14 13:48:00 ~1 min nix-flake 📄log
✔️ d33f7aa #1 2023-04-14 13:49:01 ~2 min ios 📦tgz
✔️ d33f7aa #1 2023-04-14 13:49:17 ~3 min android 📦tgz
✖️ d33f7aa #2 2023-04-14 13:50:07 ~22 sec tests 📄log
✖️ 2759a99 #3 2023-04-14 13:55:27 ~53 sec tests 📄log
✔️ 2759a99 #2 2023-04-14 13:55:44 ~1 min linux 📦deb
✔️ 2759a99 #2 2023-04-14 13:56:29 ~1 min nix-flake 📄log
✔️ 2759a99 #2 2023-04-14 13:56:55 ~2 min ios 📦tgz
✔️ 2759a99 #2 2023-04-14 13:57:38 ~3 min android 📦tgz
✖️ 25a81cf #4 2023-04-14 14:00:50 ~25 sec tests 📄log
✔️ 25a81cf #3 2023-04-14 14:00:51 ~30 sec linux 📦deb
✔️ 25a81cf #3 2023-04-14 14:02:19 ~1 min nix-flake 📄log
✔️ 25a81cf #3 2023-04-14 14:02:43 ~2 min ios 📦tgz
✖️ 25a81cf #5 2023-04-14 14:03:26 ~1 min tests 📄log
✔️ 25a81cf #3 2023-04-14 14:03:39 ~3 min android 📦tgz
✖️ 25a81cf #7 2023-04-14 14:05:19 ~38 sec tests 📄log
✖️ 25a81cf #8 2023-04-14 15:33:47 ~25 sec tests 📄log
✔️ a27119f #4 2023-04-14 15:42:32 ~1 min linux 📦deb
✔️ a27119f #9 2023-04-14 15:42:54 ~1 min tests 📄log
✔️ a27119f #4 2023-04-14 15:43:19 ~2 min nix-flake 📄log
✔️ a27119f #4 2023-04-14 15:44:12 ~2 min ios 📦tgz
✔️ a27119f #4 2023-04-14 15:45:27 ~4 min android 📦tgz
Commit #️⃣ Finished (UTC) Duration Platform Result
9a94358 #5 2023-04-14 15:51:46 ~14 sec android 📄log
✖️ 9a94358 #10 2023-04-14 15:51:46 ~11 sec tests 📄log
9a94358 #5 2023-04-14 15:51:51 ~15 sec ios 📄log
9a94358 #5 2023-04-14 15:51:53 ~17 sec linux 📄log
✔️ 9a94358 #5 2023-04-14 15:53:31 ~1 min nix-flake 📄log
✔️ fd5865b #6 2023-04-14 15:55:54 ~1 min linux 📦deb
✔️ fd5865b #11 2023-04-14 15:56:22 ~1 min tests 📄log
✔️ fd5865b #6 2023-04-14 15:56:54 ~2 min nix-flake 📄log
✔️ fd5865b #6 2023-04-14 15:57:11 ~2 min ios 📦tgz
✔️ fd5865b #6 2023-04-14 15:57:58 ~3 min android 📦tgz

@jakubgs
Copy link
Contributor Author

jakubgs commented Apr 14, 2023

What the FUCK is happening?

image

This PR is #512, why is it trying to get files form other PR builds?

@jakubgs
Copy link
Contributor Author

jakubgs commented Apr 14, 2023

I believe this is caused by things cached in ~/.cache/golangci-lint:

jenkins@linux-03:~/.cache/golangci-lint$ du -hsc .
374M	.
374M	total

We can customize it using GOLANGCI_LINT_CACHE: https://golangci-lint.run/usage/configuration/#cache

@jakubgs
Copy link
Contributor Author

jakubgs commented Apr 14, 2023

Oh, there is another cache in place here:

bash-5.1$ grep .cache/go-build strace.log | head -n5
911842 newfstatat(AT_FDCWD, "/home/jenkins/.cache/go-build",  <unfinished ...>
911842 newfstatat(AT_FDCWD, "/home/jenkins/.cache/go-build/README",  <unfinished ...>
911842 newfstatat(AT_FDCWD, "/home/jenkins/.cache/go-build",  <unfinished ...>
911842 newfstatat(AT_FDCWD, "/home/jenkins/.cache/go-build/00",  <unfinished ...>
911842 newfstatat(AT_FDCWD, "/home/jenkins/.cache/go-build/01",  <unfinished ...>

@jakubgs jakubgs force-pushed the ci/fix-tests-gopath branch 2 times, most recently from a27119f to 9a94358 Compare April 14, 2023 15:51
@jakubgs jakubgs added the infra Infra, devops, CI and related tasks label Apr 14, 2023
Using `$HOME/go` is bad and you should feel bad.
Also, adding `/usr/loca/go/bin` is a terrible idea.
And `GOCACHE` is poisoning stuff for linter.

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs changed the title ci: fix GOPATH and PATH for tests and other ci: fix GOPATH, PATH, and GOCACHE Apr 14, 2023
@jakubgs jakubgs merged commit fd5865b into master Apr 14, 2023
@jakubgs jakubgs deleted the ci/fix-tests-gopath branch April 14, 2023 15:58
jakubgs added a commit to logos-co/nomos-node that referenced this pull request Apr 26, 2023
We've had this issue before with some `go-waku` builds:
waku-org/go-waku#512

Signed-off-by: Jakub Sokołowski <jakub@status.im>
jakubgs added a commit to logos-co/nomos-node that referenced this pull request Apr 26, 2023
We've had this issue before with some `go-waku` builds:
waku-org/go-waku#512

Signed-off-by: Jakub Sokołowski <jakub@status.im>
jakubgs added a commit to logos-co/nomos-node that referenced this pull request Apr 26, 2023
We've had this issue before with some `go-waku` builds:
waku-org/go-waku#512

Signed-off-by: Jakub Sokołowski <jakub@status.im>
jakubgs added a commit to logos-co/nomos-node that referenced this pull request Jun 12, 2023
We've had this issue before with some `go-waku` builds:
waku-org/go-waku#512

Signed-off-by: Jakub Sokołowski <jakub@status.im>
bacv pushed a commit to logos-co/nomos-node that referenced this pull request Jun 13, 2023
We've had this issue before with some `go-waku` builds:
waku-org/go-waku#512

Signed-off-by: Jakub Sokołowski <jakub@status.im>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Infra, devops, CI and related tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants