chia7712 commented on code in PR #369:
URL: https://github.com/apache/yunikorn-site/pull/369#discussion_r1468510708


##########
docs/user_guide/admission_controller.md:
##########
@@ -0,0 +1,200 @@
+---
+id: admission_controller
+title: Admission Controller
+---
+
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## What is the admission controller
+
+The pod's yaml file can be easily configured through the admission controller. 
The admission controller automatically mutates the pod spec so that it can be 
scheduled by the Yunikorn Scheduler. The admission controller is also 
responsible for validating configuration additions and updates in the configmap.
+
+The admission controller runs in a separate pod, it runs a [mutation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook)
+and a [validation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook).
+
+### Mutation webhook
+The `mutation webhook` will add `schedulerName:yunikorn` to the pod spec so 
that the pod can be scheduled by the Yunikorn scheduler.
+
+In addition, The `mutation webhook` will also add the following labels:

Review Comment:
   could you include the annotation also?  
https://github.com/apache/yunikorn-k8shim/blob/master/pkg/common/constants/constants.go#L95



##########
docs/user_guide/admission_controller.md:
##########
@@ -0,0 +1,200 @@
+---
+id: admission_controller
+title: Admission Controller
+---
+
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## What is the admission controller
+
+The pod's yaml file can be easily configured through the admission controller. 
The admission controller automatically mutates the pod spec so that it can be 
scheduled by the Yunikorn Scheduler. The admission controller is also 
responsible for validating configuration additions and updates in the configmap.
+
+The admission controller runs in a separate pod, it runs a [mutation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook)
+and a [validation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook).
+
+### Mutation webhook
+The `mutation webhook` will add `schedulerName:yunikorn` to the pod spec so 
that the pod can be scheduled by the Yunikorn scheduler.
+
+In addition, The `mutation webhook` will also add the following labels:
+- `applicationId` label. If applicationId or spark-app-selector already 
exists, it will be used. Otherwise, an applicationId will be generated for the 
pod. The default is `yunikorn-<namespace>-autogen`. Pods in the same namespace 
will generate the same applicationId. For other generation methods, please 
refer to [Service configuration](service_config.md). 
+- `queue` label. If there is a queue label, it will be used. Note that if 
placement rule is enabled, the value set in label will be ignored. Otherwise, 
`root.default` will be assigned by default. If you want to change the default 
queue name, please refer to [Service configuration](service_config.md).
+- `disableStateAware` label, if the admission controller generates an 
applicationId for the pod, `disableStateAware: true` must also be set. This 
will cause the application to immediately transition from "Starting" to 
"Running" state so that it will not block other applications
+
+When updating a created pod, the admission controller will check whether the 
userinfo annotation before and after the update is the same. If it is 
different, the change will not be allowed.
+### Validation webhook
+The `validation webhook` validates the configuration set in the configmap.This 
is used to prevent writing malformed configuration into the configmap.
+The `validation webhook` calls scheduler [validation REST 
API](https://yunikorn.apache.org/docs/api/scheduler#configuration-validation) 
to validate configmap updates.
+
+## Admission controller deployment
+By default, the admission controller is deployed as part of the YuniKorn Helm 
chart installation. This can be disabled if necessary (though not recommended) 
by setting the Helm parameter `embedAdmissionController` to `false`.

Review Comment:
   "necessary (though not recommended)" could we have a section to discuss the 
reasons why we `discourage` users from doing this?



##########
docs/user_guide/admission_controller.md:
##########
@@ -0,0 +1,200 @@
+---
+id: admission_controller
+title: Admission Controller
+---
+
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## What is the admission controller
+
+The pod's yaml file can be easily configured through the admission controller. 
The admission controller automatically mutates the pod spec so that it can be 
scheduled by the Yunikorn Scheduler. The admission controller is also 
responsible for validating configuration additions and updates in the configmap.
+
+The admission controller runs in a separate pod, it runs a [mutation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook)
+and a [validation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook).
+
+### Mutation webhook
+The `mutation webhook` will add `schedulerName:yunikorn` to the pod spec so 
that the pod can be scheduled by the Yunikorn scheduler.
+
+In addition, The `mutation webhook` will also add the following labels:
+- `applicationId` label. If applicationId or spark-app-selector already 
exists, it will be used. Otherwise, an applicationId will be generated for the 
pod. The default is `yunikorn-<namespace>-autogen`. Pods in the same namespace 
will generate the same applicationId. For other generation methods, please 
refer to [Service configuration](service_config.md). 
+- `queue` label. If there is a queue label, it will be used. Note that if 
placement rule is enabled, the value set in label will be ignored. Otherwise, 
`root.default` will be assigned by default. If you want to change the default 
queue name, please refer to [Service configuration](service_config.md).
+- `disableStateAware` label, if the admission controller generates an 
applicationId for the pod, `disableStateAware: true` must also be set. This 
will cause the application to immediately transition from "Starting" to 
"Running" state so that it will not block other applications
+
+When updating a created pod, the admission controller will check whether the 
userinfo annotation before and after the update is the same. If it is 
different, the change will not be allowed.

Review Comment:
   pardon me, could you share reference to me? did not noticed this 
functionality before.



##########
docs/user_guide/admission_controller.md:
##########
@@ -0,0 +1,200 @@
+---
+id: admission_controller
+title: Admission Controller
+---
+
+<!--
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+  http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under the License is distributed on an
+"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+KIND, either express or implied.  See the License for the
+specific language governing permissions and limitations
+under the License.
+-->
+
+## What is the admission controller
+
+The pod's yaml file can be easily configured through the admission controller. 
The admission controller automatically mutates the pod spec so that it can be 
scheduled by the Yunikorn Scheduler. The admission controller is also 
responsible for validating configuration additions and updates in the configmap.
+
+The admission controller runs in a separate pod, it runs a [mutation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook)
+and a [validation 
webhook](https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#validatingadmissionwebhook).
+
+### Mutation webhook
+The `mutation webhook` will add `schedulerName:yunikorn` to the pod spec so 
that the pod can be scheduled by the Yunikorn scheduler.
+
+In addition, The `mutation webhook` will also add the following labels:
+- `applicationId` label. If applicationId or spark-app-selector already 
exists, it will be used. Otherwise, an applicationId will be generated for the 
pod. The default is `yunikorn-<namespace>-autogen`. Pods in the same namespace 
will generate the same applicationId. For other generation methods, please 
refer to [Service configuration](service_config.md). 
+- `queue` label. If there is a queue label, it will be used. Note that if 
placement rule is enabled, the value set in label will be ignored. Otherwise, 
`root.default` will be assigned by default. If you want to change the default 
queue name, please refer to [Service configuration](service_config.md).
+- `disableStateAware` label, if the admission controller generates an 
applicationId for the pod, `disableStateAware: true` must also be set. This 
will cause the application to immediately transition from "Starting" to 
"Running" state so that it will not block other applications
+
+When updating a created pod, the admission controller will check whether the 
userinfo annotation before and after the update is the same. If it is 
different, the change will not be allowed.
+### Validation webhook
+The `validation webhook` validates the configuration set in the configmap.This 
is used to prevent writing malformed configuration into the configmap.
+The `validation webhook` calls scheduler [validation REST 
API](https://yunikorn.apache.org/docs/api/scheduler#configuration-validation) 
to validate configmap updates.
+
+## Admission controller deployment
+By default, the admission controller is deployed as part of the YuniKorn Helm 
chart installation. This can be disabled if necessary (though not recommended) 
by setting the Helm parameter `embedAdmissionController` to `false`.
+
+On startup, the admission controller performs a series of tasks to ensure that 
it is properly registered with Kubernetes:
+
+Loads a Kubernetes secret called `admission-controller-secrets`. This secret 
stores a pair of CA certificates which are used to sign the TLS server 
certificate used by the admission controller.
+If the secret cannot be found or either CA certificate is within 90 days of 
expiration, generates new certificate(s). If a certificate is expiring, a new 
one is generated with an expiration of 12 months in the future. If both 
certificates are missing or expiring, the second certificate is generated with 
an expiration of 6 months in the future. This ensures that both certificates do 
not expire at the same time, and that there is an overlap of trusted 
certificates.
+If the CA certificates were created or updated, writes the secrets back to 
Kubernetes.
+Generates an ephemeral TLS server certificate signed by the CA certificate 
with the latest expiration date.
+Validates, and if necessary, creates or updates the Kubernetes webhook 
configurations named `yunikorn-admission-controller-validations` and 
`yunikorn-admission-controller-mutations`. If the CA certificates have changed, 
the webhooks will also be updated. These webhooks allow the Kubernetes API 
server to connect to the admission controller service to perform configmap 
validations and pod mutations.
+Starts up the admission controller HTTPS server.
+
+Additionally, the admission controller also starts a background task to wait 
for CA certificates to expire. Once either certificate is expiring within the 
next 30 days, new CA and server certificates are generated, the webhook 
configurations are updated, and the HTTPS server is quickly restarted. This 
ensures that certificates rotate properly without downtime.
+
+In production clusters, it is recommended to deploy the admission controller 
with two replicas by setting the Helm parameter 
`admissionController.replicaCount` to `2`. This will ensure that at least one 
admission controller webhook is reachable by the Kubernetes API server at all 
times. In this configuration, the CA certificates and webhook configurations 
are shared between the instances.
+
+## Configure the admission controller
+The admission controller can be configured with the `yunikorn-configs` 
configmap.
+
+Through the configuration of the admission controller, you can control 
multiple settings, including specifying the namespace you want or not want 
Yunikorn to schedule, whether to generate a unique applicationId and default 
queue name for each application, etc. In addition, the admission controller can 
be used to control the access of users and groups.
+
+For more details on admission controller configuration, please refer to 
[Service configuration](service_config.md).
+
+For more information on user and group access management, please view 
[User&Group Resolutiona](usergroup_resolution.md).
+
+## Example
+The following YAML does not set any information related to YuniKorn
+```yaml
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+       labels:
+               app: nginx
+       name: nginx
+spec:
+       replicas: 1
+       selector:
+               matchLabels:
+                       app: nginx
+       template:
+               metadata:
+                       labels:
+                               app: nginx
+                       name: nginx
+               spec:
+                       container:
+                               - name: nginx
+                                       image: "nginx:stable-alpine

Review Comment:
   there are some invalid format, please fix it.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to