Merge branch 'feat/container-port-ranges' of https://github.com/topfr…

…eegames/maestro into feat/container-port-ranges
topfreegames · Jun 27, 2024 · d3fe3d9 · d3fe3d9
2 parents 9f59d1d + 331184d
commit d3fe3d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/internal/core/entities/scheduler.go b/internal/core/entities/scheduler.go
@@ -53,7 +53,7 @@ const (
 
 var (
 	ErrNoPortRangeConfigured     = errors.New("must configure scheduler.PortRange or scheduler.Spec.Container.Ports.HostPortRange")
-	ErrBothPortRangesConfigured  = errors.New("cannot configure both scheduler.PortRange and scheduler.Spec.Container.Ports.HostPortRange")
+	ErrBothPortRangesConfigured  = errors.New("scheduler.PortRange and scheduler.Spec.Container.Ports.HostPortRange are mutually exclusive configurations; please choose only one")
 	ErrMissingContainerPortRange = errors.New("must configure HostPortRange for all scheduler.Spec.Container.Ports")
 )
 

diff --git a/proto/api/v1/messages.proto b/proto/api/v1/messages.proto
@@ -115,7 +115,7 @@ message ContainerPort {
   int32 port = 3;
   // Optional container host port.
   int32 host_port = 4;
-  // Optional host port range, replaces the PortRange in the Scheduler object
+  // The host port range object describes what is the port range used to allocate game rooms.
   optional PortRange host_port_range = 5;
 }