Add helpful comments

Signed-off-by: Carlisia <[email protected]>

internal/util/managercontroller/managercontroller.go

@@ -14,6 +14,8 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+// TODO(2.0) After converting all controllers to runttime-controller,
+// the functions in this file will no longer be needed and should be removed.
 package managercontroller
 
 import (

pkg/apis/velero/v1/backupstoragelocation_types.go

@@ -70,6 +70,8 @@ type BackupStorageLocationStatus struct {
 	AccessMode BackupStorageLocationAccessMode `json:"accessMode,omitempty"`
 }
 
+// TODO(2.0) After converting all resources to use the runttime-controller client,
+// the genclient and k8s:deepcopy markers will no longer be needed and should be removed.
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 // +kubebuilder:object:root=true
@@ -88,6 +90,8 @@ type BackupStorageLocation struct {
 	Status BackupStorageLocationStatus `json:"status,omitempty"`
 }
 
+// TODO(2.0) After converting all resources to use the runttime-controller client,
+// the k8s:deepcopy marker will no longer be needed and should be removed.
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 // +kubebuilder:object:root=true
 

pkg/cmd/server/server.go

@@ -469,6 +469,8 @@ func (s *server) validateBackupStorageLocations() error {
 	pluginManager := clientmgmt.NewManager(s.logger, s.logLevel, s.pluginRegistry)
 	defer pluginManager.CleanupClients()
 
+	// Fetching from the server directly since at this point
+	// the cache has not yet started
 	locations := &velerov1api.BackupStorageLocationList{}
 	if err := s.mgr.GetAPIReader().List(context.Background(), locations, &kbclient.ListOptions{
 		Namespace: s.namespace,