refactor: 💡 add note to change the logic in updateVolumes()

pkg/controller/servicebindingrequest/binder.go

@@ -4,10 +4,11 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"time"
+
 	"k8s.io/apimachinery/pkg/fields"
 	"k8s.io/apimachinery/pkg/labels"
 	"k8s.io/apimachinery/pkg/runtime/schema"
-	"time"
 
 	"gotest.tools/assert/cmp"
 	corev1 "k8s.io/api/core/v1"
@@ -141,6 +142,8 @@ func (b *Binder) removeSpecVolumes(
 func (b *Binder) updateVolumes(volumes []interface{}) ([]interface{}, error) {
 	name := b.sbr.GetName()
 	log := b.logger
+
+	// FIXME(isuttonl): update should not bail out here since b.volumeKeys might change
 	log.Debug("Checking if binding volume is already defined...")
 	for _, v := range volumes {
 		volume := v.(corev1.Volume)