-
Notifications
You must be signed in to change notification settings - Fork 405
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FLINK-35357][docs] Add kubernetes.operator.plugins.listeners config #845
Conversation
@@ -182,5 +182,11 @@ | |||
<td>Map</td> | |||
<td>Custom HTTP header for HttpArtifactFetcher. The header will be applied when getting the session job artifacts. Expected format: headerKey1:headerValue1,headerKey2:headerValue2.</td> | |||
</tr> | |||
<tr> | |||
<td><h5>kubernetes.operator.plugins.listeners.<listener-name>.class</h5></td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, these html files are generated instead of changing manually.
As I understand, you need to update the KubernetesOperatorConfigOptions
and run mvn clean install -DskipTests -Pgenerate-docs
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your advice. It seems I have some misconceptions about its construction. I will fix it。
@@ -333,6 +333,14 @@ public static String operatorConfigKey(String key) { | |||
+ "WARNING: not intended to be used together with the cron-based " | |||
+ "periodic checkpoint triggering"); | |||
|
|||
@Documentation.Section(SECTION_DYNAMIC) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Documentation.Section(SECTION_DYNAMIC) | |
@SuppressWarnings("unused") | |
@Documentation.Section(SECTION_DYNAMIC) |
Adding the @SuppressWarnings("unused")
to avoid warning.
public static final ConfigOption<String> PLUGINS_LISTENERS_CLASS = | ||
operatorConfig("plugins.listeners.<listener-name>.class") | ||
.stringType() | ||
.defaultValue("") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.defaultValue("") | |
.noDefaultValue() |
hi @1996fanrui . Thank you very much for your advice, I have optimized it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update!
LGTM, I will merge it if no objections within 2 days.
What is the purpose of the change
Brief change log
1.Edit docs/layouts/shortcodes/generated/dynamic_section.html
2.Add kubernetes.operator.plugins.listeners config
Verifying this change
Does this pull request potentially affect one of the following parts:
CustomResourceDescriptors
: (yes / no) noDocumentation