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.

Sign up for GitHub

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

Jump to bottom

cursed panic #2845

Closed
worstell opened this issue Sep 26, 2024 · 1 comment · Fixed by #3192
Closed

cursed panic #2845

worstell opened this issue Sep 26, 2024 · 1 comment · Fixed by #3192
Labels
bug Something isn't working

Comments

@worstell
Copy link
Contributor 

> panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x1008f555c]

goroutine 414 [running]:
github.com/TBD54566975/ftl/go-runtime/schema.analyzersWithDependencies()
	/Users/dli/Development/ftl/go-runtime/schema/extract.go:303 +0x120
github.com/TBD54566975/ftl/go-runtime/schema.Extract({0x14000b5a2d0?, 0x101ab454c?})
	/Users/dli/Development/ftl/go-runtime/schema/extract.go:119 +0xd8
github.com/TBD54566975/ftl/go-runtime/compile.Build({0x10257b838, 0x14000d920f0}, {0x14000634b10, 0x1a}, {0x14000b5a2d0, 0x30}, 0x14000e2f750, {0x14a84bf20, 0x14001212020}, {0x0, ...}, ...)
	/Users/dli/Development/ftl/go-runtime/compile/build.go:273 +0x910
github.com/TBD54566975/ftl/internal/buildengine.buildGoModule({0x10257b838?, _}, {_, _}, _, {{{0x14000b5a2d0, 0x30}, {0x140008af748, 0x2}, {0x140008af74c, ...}, ...}, ...}, ...)
	/Users/dli/Development/ftl/internal/buildengine/build_go.go:12 +0x84
github.com/TBD54566975/ftl/internal/buildengine.buildModule({0x10257b838, _}, {_, _}, _, {{{0x14000b5a2d0, 0x30}, {0x140008af748, 0x2}, {0x140008af74c, ...}, ...}, ...}, ...)
	/Users/dli/Development/ftl/internal/buildengine/build.go:49 +0x588
github.com/TBD54566975/ftl/internal/buildengine.Build(...)
	/Users/dli/Development/ftl/internal/buildengine/build.go:26
github.com/TBD54566975/ftl/internal/buildengine.(*Engine).build(0x14000a20180, {0x10257b838, 0x140012f00c0}, {0x140008ae865, 0x5}, 0x140001374d0, 0x140006584d0)
	/Users/dli/Development/ftl/internal/buildengine/engine.go:739 +0x354
github.com/TBD54566975/ftl/internal/buildengine.(*Engine).tryBuild(0x14000a20180, {0x10257b838, 0x140012f00c0}, 0x14000ee0d50, {0x140008ae865, 0x5}, 0x140001374d0, 0x140006584d0, 0x14000ee0d20)
	/Users/dli/Development/ftl/internal/buildengine/engine.go:706 +0x294
github.com/TBD54566975/ftl/internal/buildengine.(*Engine).buildWithCallback.func2()
	/Users/dli/Development/ftl/internal/buildengine/engine.go:641 +0x3a0
golang.org/x/sync/errgroup.(*Group).Go.func1()
	/Users/dli/go/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:78 +0x54
created by golang.org/x/sync/errgroup.(*Group).Go in goroutine 422
	/Users/dli/go/pkg/mod/golang.org/x/[email protected]/errgroup/errgroup.go:75 +0x98
	```
@worstell worstell added the bug Something isn't working label Sep 26, 2024
@worstell worstell self-assigned this Sep 26, 2024
@ftl-robot ftl-robot mentioned this issue Sep 26, 2024
Open
stuartwdouglas added a commit that referenced this issue Sep 26, 2024
@stuartwdouglas
chore: figure out what is going on with the panic
76321c4 
diagnoses: #2845
@stuartwdouglas stuartwdouglas mentioned this issue Sep 26, 2024
Merged
stuartwdouglas added a commit that referenced this issue Sep 26, 2024
@stuartwdouglas
chore: figure out what is going on with the panic (#2851)
4ad4d48 
diagnoses: #2845
@worstell
Copy link
Contributor Author

going to unassign this with @stuartwdouglas's change in, and wait for the next recurrence

@worstell worstell removed their assignment Oct 10, 2024
stuartwdouglas added a commit that referenced this issue Oct 25, 2024
@stuartwdouglas
fix: cursed panic
432adc7 
The requires slice was being appended to concurrently multiple times, this should only happen once.

fixes: #2845 #3162
@stuartwdouglas stuartwdouglas mentioned this issue Oct 25, 2024
Merged
stuartwdouglas added a commit that referenced this issue Oct 25, 2024
@stuartwdouglas
fix: cursed panic
a28bb78 
The requires slice was being appended to concurrently multiple times, this should only happen once.

fixes: #2845 #3162
stuartwdouglas added a commit that referenced this issue Oct 25, 2024
@stuartwdouglas
fix: cursed panic (#3192)
1b6d741 
The requires slice was being appended to concurrently multiple times,
this should only happen once.

fixes: #2845 #3162
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: cursed panic block/ftl
1 participant
@worstell