Remove GameServer UpdateStatus() since its not being used.

Also included documentation on why we don't provide a status
subresource.

pkg/apis/agones/v1/gameserver.go

@@ -92,9 +92,14 @@ var (
 )
 
 // +genclient
+// +genclient:noStatus
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 
-// GameServer is the data structure for a GameServer resource
+// GameServer is the data structure for a GameServer resource.
+// It is worth noting that while there is a `GameServerStatus` Status entry for the `GameServer`, it is not
+// defined as a subresource - unlike `Fleet` and other Agones resources.
+// This is so that we can retain the ability to change multiple aspects of a `GameServer` in a single atomic operation,
+// which is particularly useful for operations such as allocation.
 type GameServer struct {
 	metav1.TypeMeta   `json:",inline"`
 	metav1.ObjectMeta `json:"metadata,omitempty"`

site/content/en/docs/Reference/agones_crd_api_reference.html

@@ -2497,7 +2497,11 @@ <h3 id="Fleet">Fleet
 <h3 id="GameServer">GameServer
 </h3>
 <p>
-<p>GameServer is the data structure for a GameServer resource</p>
+<p>GameServer is the data structure for a GameServer resource.
+It is worth noting that while there is a <code>GameServerStatus</code> Status entry for the <code>GameServer</code>, it is not
+defined as a subresource - unlike <code>Fleet</code> and other Agones resources.
+This is so that we can retain the ability to change multiple aspects of a <code>GameServer</code> in a single atomic operation,
+which is particularly useful for operations such as allocation.</p>
 </p>
 <table>
 <thead>