Skip to content

Commit 5061759

Browse files
authored
fix: error with java docs validation (#2336)
Signed-off-by: Attila Mészáros <[email protected]>
1 parent 1b849dd commit 5061759

File tree

2 files changed

+13
-5
lines changed

2 files changed

+13
-5
lines changed

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/ObservedGenerationAware.java

+6-4
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import io.fabric8.kubernetes.api.model.HasMetadata;
44
import io.fabric8.kubernetes.client.CustomResource;
5+
import io.javaoperatorsdk.operator.api.config.ConfigurationService;
56
import io.javaoperatorsdk.operator.api.reconciler.UpdateControl;
67

78
/**
@@ -11,10 +12,11 @@
1112
* In order for this automatic handling to work the status object returned by
1213
* {@link CustomResource#getStatus()} should not be null.
1314
* <p>
14-
* The observed generation is updated even when {@link UpdateControl#noUpdate()} or
15-
* {@link UpdateControl#updateResource(HasMetadata)} is called. Although those results call normally
16-
* does not result in a status update, there will be a subsequent status update Kubernetes API call
17-
* in this case.
15+
* The observed generation is updated with SSA mode only if
16+
* {@link UpdateControl#patchStatus(HasMetadata)} or
17+
* {@link UpdateControl#patchResourceAndStatus(HasMetadata)} is called. In non-SSA mode (see
18+
* {@link ConfigurationService#useSSAToPatchPrimaryResource()}) observed generation is update even
19+
* if patch is not called.
1820
*
1921
* @see ObservedGenerationAwareStatus
2022
*/

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ConfigurationService.java

+7-1
Original file line numberDiff line numberDiff line change
@@ -327,6 +327,8 @@ default ExecutorServiceManager getExecutorServiceManager() {
327327
* method of Kubernetes Dependent Resource.
328328
*
329329
* @since 4.4.0
330+
*
331+
* @return if SSA should be used for dependent resources
330332
*/
331333
default boolean ssaBasedCreateUpdateMatchForDependentResources() {
332334
return true;
@@ -355,6 +357,8 @@ default Set<Class<? extends HasMetadata>> defaultNonSSAResource() {
355357
* Disable this if you want to react to your own dependent resource updates
356358
*
357359
* @since 4.5.0
360+
*
361+
* @return if special annotation should be used for dependent resource to filter events
358362
*/
359363
default boolean previousAnnotationForDependentResourcesEventFiltering() {
360364
return true;
@@ -366,10 +370,12 @@ default boolean previousAnnotationForDependentResourcesEventFiltering() {
366370
* <p>
367371
* Disabled by default as Kubernetes does not support, and discourages, this interpretation of
368372
* resourceVersions. Enable only if your api server event processing seems to lag the operator
369-
* logic and you want to further minimize the the amount of work done / updates issued by the
373+
* logic, and you want to further minimize the amount of work done / updates issued by the
370374
* operator.
371375
*
372376
* @since 4.5.0
377+
*
378+
* @return if resource version should be parsed (as integer)
373379
*/
374380
default boolean parseResourceVersionsForEventFilteringAndCaching() {
375381
return false;

0 commit comments

Comments
 (0)