martin-g commented on code in PR #20467:
URL: https://github.com/apache/flink/pull/20467#discussion_r990934450
##########
docs/layouts/shortcodes/generated/kubernetes_config_configuration.html:
##########
@@ -206,6 +212,12 @@
<td><p>Enum</p></td>
<td>The exposed type of the rest service. The exposed rest service
could be used to access the Flink’s Web UI and REST endpoint.<br /><br
/>Possible
values:<ul><li>"ClusterIP"</li><li>"NodePort"</li><li>"LoadBalancer"</li><li>"Headless_ClusterIP"</li></ul></td>
</tr>
+ <tr>
+ <td><h5>kubernetes.scheduler-name</h5></td>
+ <td style="word-wrap: break-word;">"default-scheduler"</td>
+ <td>String</td>
+ <td>Specify the kubernetes pod scheduler for Flink pods of
deployment. The default value is using the kubernetes default pod scheduler.
For customerized kubernetes pod scheduler, allow to set pod scheduler for
customerized pod scheduling. Notice that this can be overwritten by config
options 'kubernetes.jobmanager.scheduler-name' and
'kubernetes.taskmanager.scheduler-name' for jobmanager and taskmanager
respectively.</td>
Review Comment:
```suggestion
<td>The name of the Kubernetes pod scheduler for Flink pods of
deployment. The default value is using the kubernetes default pod scheduler.
Notice that this can be overwritten by config options
'kubernetes.jobmanager.scheduler-name' and
'kubernetes.taskmanager.scheduler-name' for JobManager and TaskManager
respectively.</td>
```
##########
docs/layouts/shortcodes/generated/kubernetes_config_configuration.html:
##########
@@ -152,6 +152,12 @@
<td>Integer</td>
<td>Specify how many JobManager pods will be started
simultaneously. Configure the value to greater than 1 to start standby
JobManagers. It will help to achieve faster recovery. Notice that high
availability should be enabled when starting standby JobManagers.</td>
</tr>
+ <tr>
+ <td><h5>kubernetes.jobmanager.scheduler-name</h5></td>
+ <td style="word-wrap: break-word;">"default-scheduler"</td>
+ <td>String</td>
+ <td>Specify the kubernetes pod scheduler for jobmanager pods of
deployment. The default value is using the kubernetes default pod scheduler.
For customerized kubernetes pod scheduler, allow to set pod scheduler for
customerized pod scheduling. If not explicitly configured, config option
'kubernetes.scheduler-name' will be used.</td>
Review Comment:
```suggestion
<td>The name of the Kubernetes pod scheduler for JobManager
pods. If not explicitly configured then config option
'kubernetes.scheduler-name' will be used.</td>
```
##########
docs/layouts/shortcodes/generated/kubernetes_config_configuration.html:
##########
@@ -260,6 +272,12 @@
<td>Map</td>
<td>The node selector to be set for TaskManager pods. Specified as
key:value pairs separated by commas. For example,
environment:production,disk:ssd.</td>
</tr>
+ <tr>
+ <td><h5>kubernetes.taskmanager.scheduler-name</h5></td>
+ <td style="word-wrap: break-word;">"default-scheduler"</td>
+ <td>String</td>
+ <td>Specify the kubernetes pod scheduler for taskmanager pods of
deployment. The default value is using the kubernetes default pod scheduler.
For customerized kubernetes pod scheduler, allow to set pod scheduler for
customerized pod scheduling. If not explicitly configured, config option
'kubernetes.scheduler-name' will be used.</td>
Review Comment:
```suggestion
<td>The name of the Kubernetes pod scheduler for taskmanager
pods. If not explicitly configured then config option
'kubernetes.scheduler-name' will be used.</td>
```
##########
flink-kubernetes/src/main/java/org/apache/flink/kubernetes/configuration/KubernetesConfigOptions.java:
##########
@@ -444,6 +445,46 @@ public class KubernetesConfigOptions {
public static final ConfigOption<String> TASK_MANAGER_POD_TEMPLATE;
+ public static final ConfigOption<String> JOB_MANAGER_POD_SCHEDULER_NAME =
+ key("kubernetes.jobmanager.scheduler-name")
+ .stringType()
+ .defaultValue("default-scheduler")
+ .withFallbackKeys(KUBERNETES_POD_SCHEDULER_NAME_KEY)
+ .withDescription(
+ "Specify the kubernetes pod scheduler for
jobmanager pods of deployment. "
Review Comment:
The description should be in sync with the documentation. That is, if my
suggestions for the `.html` file are accepted then they should be applied here
as well.
--
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]
