config: Remove unnecessary nil checks

Signed-off-by: Daniel Kleinstein <daniel@scaleops.com>
NVIDIA · Oct 28, 2024 · d1bb887 · d1bb887
1 parent e6ec8d3
commit d1bb887
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/api/config/v1/config.go b/api/config/v1/config.go
@@ -97,13 +97,8 @@ func DisableResourceNamingInConfig(logger logger, config *Config) {
 	config.Resources.MIGs = nil
 
 	// Disable renaming / device selection in Sharing.TimeSlicing.Resources
-	if (config.sharing.TimeSlicing != nil) {
-		config.Sharing.TimeSlicing.disableResoureRenaming(logger, "timeSlicing")
-	}
-	// Disable renaming / device selection in Sharing.MPS.Resources
-	if (config.Sharing.MPS != nil) {
-		config.Sharing.MPS.disableResoureRenaming(logger, "mps")
-	}
+	config.Sharing.TimeSlicing.disableResoureRenaming(logger, "timeSlicing")
+	config.Sharing.MPS.disableResoureRenaming(logger, "mps")
 }
 
 // parseConfig parses a config file as either YAML of JSON and unmarshals it into a Config struct.

diff --git a/api/config/v1/sharing.go b/api/config/v1/sharing.go
@@ -38,7 +38,7 @@ func (s *Sharing) SharingStrategy() SharingStrategy {
 		return SharingStrategyMPS
 	}
 
-	if s.TimeSlicing != nil && s.TimeSlicing.isReplicated() {
+	if s.TimeSlicing.isReplicated() {
 		return SharingStrategyTimeSlicing
 	}
 	return SharingStrategyNone