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

fix: Create redis by default in public dns with cloud dns example #139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

levinandrew
Copy link
Contributor

after terraform apply succeeded... wandb-app kept failing with the following at commit hash 00fefa79c305004a1dda3470bf58950e35b5df98

panic: dial tcp 10.152.243.127:0: i/o timeout

goroutine 1 [running]:
github.com/wandb/core/services/gorilla/cmd.RequiredCache(...)
	/home/circleci/go/src/github.com/wandb/core/services/gorilla/cmd/setup.go:1485
github.com/wandb/core/services/gorilla/cmd.(*gorillaCommander).MainCmd(0xc0014c0c00, {0xc0014c0d00, 0x1, 0x1})
	/home/circleci/go/src/github.com/wandb/core/services/gorilla/cmd/gorilla.go:656 +0xca05
main.main()
	/home/circleci/go/src/github.com/wandb/core/services/gorilla/cmd/megabinary/main.go:77 +0x431

the cacheConfig env var (passed into the the function call that returns the error that we panic with) was set to redis://wandb-redis-master:0?ttlInSeconds=60480

which got resolved to

wandb@wandb-app-76bd48749-rx6lz:~$ getent hosts wandb-redis-master
10.152.243.127  wandb-redis-master.default.svc.cluster.local

after setting that create_redis to true, wandb-app could start up successfully

@levinandrew levinandrew requested a review from a team as a code owner June 25, 2024 20:38
@levinandrew levinandrew changed the title fix: create redis by default in public dns with cloud dns example fix: Create redis by default in public dns with cloud dns example Jun 26, 2024
@levinandrew levinandrew force-pushed the levinandrew/fix-public-dns-with-cloud-dns-ex branch from e3ca828 to 6abe811 Compare June 26, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant