[YUNIKORN-3212] auto update GOMEMLIMIT setting

Document the new percentage setting mark the existing fixed value as
deprecated.

Author: wilfred-s

docs/user_guide/service_config.md

@@ -357,39 +357,39 @@ Additionally, garbage collection and memory usage can be tuned as follows
 ```yaml
 # Scheduler container GC configuration
 goGC: 100
-goMemoryLimit: 1536MiB
+goMemoryLimitPercentage: 80
 
 # Web UI container resources
 web:
   goGC: 100
-  goMemoryLimit: 200MiB
+  goMemoryLimitPercentage: 60
 
 # Admission controller GC configuration
 admissionController:
   goGC: 100
-  goMemoryLimit: 200MiB
+  goMemoryLimitPercentage: 60
 ```
 
 The `goGC`, `web.goGC`, and `admissionController.goGC` settings control the
 percentage of new, unallocated heap memory vs. current live memory at which
-the Go runtime initates garbage collection for the scheduler, Web UI, and
+the Go runtime initiates garbage collection for the scheduler, Web UI, and
 Admission Controller, respectively.
 
 The default value of `100` matches the Go runtime default. This value is
 passed into the container via the `GOGC` environment variable.
 
-The `goMemoryLimit`, `web.goMemoryLimit`, and
-`admissionController.goMemoryLimit` is used to set a soft limit on the
-total amount of memory the Go runtime is allowed to consume. This limit
-applies to memory managed by the Go runtime. This value should be set
-lower than the respective value of `resources.limits.memory` to allow
-for transient spikes in memory usage as well as memory usage which is
-not managed by the Go runtime.
+The `goMemoryLimitPercentage`, `web.goMemoryLimitPercentage`, and
+`admissionController.goMemoryLimitPercentage` are used to set a soft limit on
+the total amount of memory the Go runtime is allowed to consume. This limit
+applies to memory managed by the Go runtime. The setting helps manage transient
+spikes in memory usage as well as memory usage which is not managed by the Go
+runtime.
 
-Legal values consist of a numeric value in bytes with an optional unit
-suffix. The supported suffixes include `B`, `KiB`, `MiB`, `GiB`, and
-`TiB`. This value is passsed into the container via the `GOMEMLIMIT`
-environment variable.
+The value is applied to the `resources.limits.memory` setting for the container
+when the pod is created. A value of 80 is interpreted as 80% and is applied to
+the `resources.limits.memory`. 
+The resulting value is the memory limit in bytes, and is passed into the 
+container via the `GOMEMLIMIT` environment variable.
 
 ### Optional features
 
@@ -447,6 +447,26 @@ yunikornDefaults:
 ```
 
 ### Deprecated settings
+
+#### goMemoryLimit setting (specified value)
+
+**_DEPRECATED in 1.9.0:_** replaced by `goMemoryLimitPercentage`
+
+The `goMemoryLimit`, `web.goMemoryLimit`, and
+`admissionController.goMemoryLimit` are used to set a soft limit on the
+total amount of memory the Go runtime is allowed to consume. This limit
+applies to memory managed by the Go runtime. This value should be set
+lower than the respective value of `resources.limits.memory` to allow
+for transient spikes in memory usage as well as memory usage which is
+not managed by the Go runtime.
+
+Legal values consist of a numeric value in bytes with an optional unit
+suffix. The supported suffixes include `B`, `KiB`, `MiB`, `GiB`, and
+`TiB`. This value is passsed into the container via the `GOMEMLIMIT`
+environment variable.
+
+#### Moved to ConfigMap
+
 The following settings are deprecated, and will be removed from a future
 YuniKorn release. They should now be specified in the `yunikorn-configs` ConfigMap
 or via the Helm `yunikornDefaults` section: