oracle / oracle/oracle-database-operator

panic: runtime error: invalid memory address or nil pointer dereference

Open
#49 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
192
Forks
69
Avg merge
16h 13m
Merged PRs (30d)
1

Description

oracle-database-operator keeps CrashLoopBackOff for a SingleInstanceDatabase runtime.

Stack Trace:

21.3.0.0.0
{"reconciler group": "database.oracle.com", "reconciler kind": "SingleInstanceDatabase", "name": "maximodb", "namespace": "oracle-database-operator", "GetDatabaseVersion": "oracle-database-operator/maximodb"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1a1639e]
goroutine 1630 [running]:
github.com/oracle/oracle-database-operator/controllers/database.(*SingleInstanceDatabaseReconciler).deleteWallet(0xc000665180, 0x3, {0x224ee98, 0xc0033667b0}, {{{0xc0002b4618, 0xe}, {0xc000383678, 0x0}}})
/workspace/controllers/database/singleinstancedatabase_controller.go:1665 +0xde
github.com/oracle/oracle-database-operator/controllers/database.(*SingleInstanceDatabaseReconciler).Reconcile(0xc000665180, {0x224ee98, 0xc0033667b0}, {{{0xc0002b4618, 0x1e6bac0}, {0xc000383678, 0x30}}})
/workspace/controllers/database/singleinstancedatabase_controller.go:188 +0x877
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile(0xc000648420, {0x224ee98, 0xc003366720}, {{{0xc0002b4618, 0x1e6bac0}, {0xc000383678, 0x413c14}}})
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:114 +0x26f
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler(0xc000648420, {0x224edf0, 0xc000a1df80}, {0x1d15660, 0xc000662960})
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:311 +0x33e
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem(0xc000648420, {0x224edf0, 0xc000a1df80})
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:266 +0x205
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2()
/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.2/pkg/internal/controller/controller.go:227 +0x85
created by sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in controllers/database/singleinstancedatabase_controller.go at deleteWallet, line 1665, then follow its call from Reconcile at line 188. Reproduce the reported SingleInstanceDatabase runtime and inspect the nil value involved in that path. Done means reconciliation no longer panics and the operator does not remain in CrashLoopBackOff.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, kubernetes
Domain
databases, devops
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.