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

vcenter null pointer due to irresponsive node #131

Closed
TaLoN1x opened this issue May 3, 2024 · 9 comments · Fixed by #132
Closed

vcenter null pointer due to irresponsive node #131

TaLoN1x opened this issue May 3, 2024 · 9 comments · Fixed by #132
Assignees
Labels
bug Something isn't working

Comments

@TaLoN1x
Copy link
Contributor

TaLoN1x commented May 3, 2024

If ESXi node is not available, I get this nullpointer:

[signal SIGSEGV: segmentation violation code=0x1 addr=0x170 pc=0xa56b73]

goroutine 13 [running]:
github.com/bl4ko/netbox-ssot/internal/source/vmware.(*VmwareSource).syncHostPhysicalNics(_, , {{{{{...}, {...}, {...}}, {0x0, 0x0, 0x0}, {0x0, 0x0, ...}}, ...}, ...}, ...)
/Users/jurikononov/src/netbox-ssot/internal/source/vmware/vmware_sync.go:285 +0x93
github.com/bl4ko/netbox-ssot/internal/source/vmware.(*VmwareSource).syncHostNics(
, _, {{{{{...}, {...}, {...}}, {0x0, 0x0, 0x0}, {0x0, 0x0, ...}}, ...}, ...}, ...)
/Users/jurikononov/src/netbox-ssot/internal/source/vmware/vmware_sync.go:261 +0x65
github.com/bl4ko/netbox-ssot/internal/source/vmware.(*VmwareSource).syncHosts(0xc000dae000, 0xc0000da000)
/Users/jurikononov/src/netbox-ssot/internal/source/vmware/vmware_sync.go:246 +0xb6a
github.com/bl4ko/netbox-ssot/internal/source/vmware.(*VmwareSource).Sync(0xc000dae000, 0xc0000da000)
/Users/jurikononov/src/netbox-ssot/internal/source/vmware/vmware.go:209 +0x2fb
main.main.func1({0xf3bbb8, 0xc000da0570}, {0xf39e70, 0xc000dae000})
/Users/jurikononov/src/netbox-ssot/cmd/netbox-ssot/main.go:98 +0x2f1
created by main.main in goroutine 1
/Users/jurikononov/src/netbox-ssot/cmd/netbox-ssot/main.go:78 +0x67e

@bl4ko bl4ko added the bug Something isn't working label May 3, 2024
@bl4ko
Copy link
Owner

bl4ko commented May 3, 2024

Thanks for reporting the issue. Could you provide the full log with DEBUG level?

@TaLoN1x
Copy link
Contributor Author

TaLoN1x commented May 3, 2024

Please find batch of last logs below. I did obfuscate some values. someserver.local is p
hysical node of cluster that is unavailable (disconnected).

sample.log

@bl4ko
Copy link
Owner

bl4ko commented May 3, 2024

This log seems odd to me. How is it possible that IP addresses and interfaces for eevcenter source are synced before device data. Did you rename more than one source to eevcenter?

@TaLoN1x
Copy link
Contributor Author

TaLoN1x commented May 3, 2024

I had some nodes defined manually in netbox. It wasnät the first run for bath either, I had some null pointers I mentioned earlier, that I worked around with if smth == nil {return} ugly hack :)

@bl4ko bl4ko self-assigned this May 3, 2024
@bl4ko
Copy link
Owner

bl4ko commented May 3, 2024

Oh, I see so you are altering source code. The error says the panic occurs on

/Users/jurikononov/src/netbox-ssot/internal/source/vmware/vmware_sync.go:443 +0x4d

Which is empty line in main branch.

Could you first report issues in the main source code so I can test them and fix them?

@TaLoN1x
Copy link
Contributor Author

TaLoN1x commented May 3, 2024

yeah, my bad. let me revert it and provide new log.

@TaLoN1x TaLoN1x closed this as completed May 3, 2024
@TaLoN1x TaLoN1x reopened this May 3, 2024
@TaLoN1x
Copy link
Contributor Author

TaLoN1x commented May 3, 2024

sample.log
this should make more sence now. Sorry for creating a confusion.

bl4ko added a commit that referenced this issue May 3, 2024
@bl4ko bl4ko linked a pull request May 3, 2024 that will close this issue
@bl4ko
Copy link
Owner

bl4ko commented May 3, 2024

Could you test again on 131-vcenter-null-pointer-due-to-irresponsive-node branch.

bl4ko added a commit that referenced this issue May 3, 2024
@TaLoN1x
Copy link
Contributor Author

TaLoN1x commented May 3, 2024

this fix resolved null pointer.

@bl4ko bl4ko closed this as completed May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants