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

Revert "update provider-aws to v0.44.2" #206

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Conversation

ssuman2-infoblox
Copy link
Contributor

Reverts #204

panic in webhook-dsnexec.go, RCA inprogress:

2023/11/21 09:58:01 http: panic serving 172.60.4.78:41468: runtime error: invalid memory address or nil pointer dereference
goroutine 121 [running]:
net/http.(*conn).serve.func1()
	/usr/local/go/src/net/http/server.go:1868 +0xb9
panic({0x1a6bc40?, 0x307b920?})
	/usr/local/go/src/runtime/panic.go:920 +0x270
sigs.k8s.io/controller-runtime/pkg/webhook/admission.(*Decoder).DecodeRaw(0x0, {{0xc0005f0a00, 0x2227, 0x2500}, {0x0, 0x0}}, {0x21fce58, 0xc00057c900})
	/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/webhook/admission/decode.go:76 +0x12e
sigs.k8s.io/controller-runtime/pkg/webhook/admission.(*Decoder).Decode(_, {{{0xc0007525a0, 0x24}, {{0x0, 0x0}, {0xc0005643a8, 0x2}, {0xc0005643aa, 0x3}}, {{0x0, ...}, ...}, ...}}, ...)
	/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/webhook/admission/decode.go:49 +0x85
github.com/infobloxopen/db-controller/webhook.(*DsnExecInjector).Handle(_, {_, _}, {{{0xc0007525a0, 0x24}, {{0x0, 0x0}, {0xc0005643a8, 0x2}, {0xc0005643aa, ...}}, ...}})
	/workspace/webhook/webhook-dsnexec.go:58 +0xa8
sigs.k8s.io/controller-runtime/pkg/webhook/admission.(*Webhook).Handle(_, {_, _}, {{{0xc0007525a0, 0x24}, {{0x0, 0x0}, {0xc0005643a8, 0x2}, {0xc0005643aa, ...}}, ...}})
	/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/webhook/admission/webhook.go:169 +0x1e9
sigs.k8s.io/controller-runtime/pkg/webhook/admission.(*Webhook).ServeHTTP(0xc00080d2c0, {0x7f448d2253b8?, 0xc00015cdc0}, 0xc00074c900)
	/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/webhook/admission/http.go:98 +0xc10
sigs.k8s.io/controller-runtime/pkg/webhook/internal/metrics.InstrumentedHook.InstrumentHandlerInFlight.func1({0x7f448d2253b8, 0xc00015cdc0}, 0x2204c00?)
	/workspace/vendor/github.com/prometheus/client_golang/prometheus/promhttp/instrument_server.go:60 +0xcb
net/http.HandlerFunc.ServeHTTP(0x2204cf8?, {0x7f448d2253b8?, 0xc00015cdc0?}, 0xc00084b8a0?)
	/usr/local/go/src/net/http/server.go:2136 +0x29
github.com/prometheus/client_golang/prometheus/promhttp.InstrumentHandlerCounter.func1({0x2204cf8?, 0xc0000f21c0?}, 0xc00074c900)
	/workspace/vendor/github.com/prometheus/client_golang/prometheus/promhttp/instrument_server.go:147 +0xb6
net/http.HandlerFunc.ServeHTTP(0x71e1a6?, {0x2204cf8?, 0xc0000f21c0?}, 0xc00048e3c0?)
	/usr/local/go/src/net/http/server.go:2136 +0x29
github.com/prometheus/client_golang/prometheus/promhttp.InstrumentHandlerDuration.func2({0x2204cf8, 0xc0000f21c0}, 0xc00074c900)
	/workspace/vendor/github.com/prometheus/client_golang/prometheus/promhttp/instrument_server.go:109 +0xc2
net/http.HandlerFunc.ServeHTTP(0x10?, {0x2204cf8?, 0xc0000f21c0?}, 0xc00048e56c?)
	/usr/local/go/src/net/http/server.go:2136 +0x29
net/http.(*ServeMux).ServeHTTP(0x410725?, {0x2204cf8, 0xc0000f21c0}, 0xc00074c900)
	/usr/local/go/src/net/http/server.go:2514 +0x142
net/http.serverHandler.ServeHTTP({0x21fd488?}, {0x2204cf8?, 0xc0000f21c0?}, 0x6?)
	/usr/local/go/src/net/http/server.go:2938 +0x8e
net/http.(*conn).serve(0xc000756ab0, {0x2212420, 0xc00057a450})
	/usr/local/go/src/net/http/server.go:2009 +0x5f4
created by net/http.(*Server).Serve in goroutine 95
	/usr/local/go/src/net/http/server.go:3086 +0x5cb

@ssuman2-infoblox ssuman2-infoblox marked this pull request as ready for review November 21, 2023 18:10
@bjeevan-ib bjeevan-ib merged commit 5f5fa4a into main Nov 21, 2023
3 checks passed
@bjeevan-ib bjeevan-ib deleted the revert-204-sujay branch November 21, 2023 23:47
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.

2 participants