yangwwei commented on a change in pull request #29:
URL:
https://github.com/apache/incubator-yunikorn-site/pull/29#discussion_r504312845
##########
File path: docs/api/scheduler.md
##########
@@ -434,6 +434,76 @@ Reponse
}
```
+## Configuration update
+
+Endpoint to override scheduler configuration.
+
+**URL** : `/ws/v1/config`
+
+**Method** : `PUT`
+
+**Auth required** : NO
+
+### Success response
+
+**Code** : `200 OK`
+
+**Content example**
+
+```yaml
+partitions:
+ -
+ name: default
+ placementrules:
+ - name: tag
+ value: namespace
+ create: true
+ queues:
+ - name: root
+ submitacl: '*'
+ properties:
+ application.sort.policy: stateaware
+checksum:
[173,7,83,162,71,12,24,54,61,72,195,227,207,95,85,14,128,101,107,7,112,214,125,208,136,131,212,118,44,191,90,222]
+```
+**Note:** the checksum is the base configuration checksum on top of what we
want to make the changes.
+If the provisioned checksum differs from the actual scheduler checksum, the
configuration update will fail.
+
+### Failure response
+
+The configuration update can fail due to different reasons such as:
+- invalid configuration,
+- outdated base checksum,
+- failure while saving the configuration to the scheduler.
Review comment:
this line is quite confusing... I think this is to outline what could be
the failures
this will be hard for users to understand why the saving fails..
##########
File path: docs/api/scheduler.md
##########
@@ -434,6 +434,76 @@ Reponse
}
```
+## Configuration update
+
+Endpoint to override scheduler configuration.
+
+**URL** : `/ws/v1/config`
+
+**Method** : `PUT`
+
+**Auth required** : NO
+
+### Success response
+
+**Code** : `200 OK`
+
+**Content example**
+
+```yaml
+partitions:
+ -
+ name: default
+ placementrules:
+ - name: tag
+ value: namespace
+ create: true
+ queues:
+ - name: root
+ submitacl: '*'
+ properties:
+ application.sort.policy: stateaware
+checksum:
[173,7,83,162,71,12,24,54,61,72,195,227,207,95,85,14,128,101,107,7,112,214,125,208,136,131,212,118,44,191,90,222]
+```
+**Note:** the checksum is the base configuration checksum on top of what we
want to make the changes.
+If the provisioned checksum differs from the actual scheduler checksum, the
configuration update will fail.
Review comment:
> If the provisioned checksum differs from the actual scheduler
checksum, the configuration update will fail.
if the provided checksum in the request differs from the base version, the
configuration update will fail. This is to avoid applying any unintentional
changes to the running version. Subsequently, the client needs to retrieve the
latest version from the scheduler and send another update with the up-to-date
checksum.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]