Skip to content

Commit e71afa6

Browse files
committed
docs
Signed-off-by: Attila Mészáros <csviri@gmail.com>
1 parent fd2f8a7 commit e71afa6

File tree

5 files changed

+2
-13
lines changed

5 files changed

+2
-13
lines changed

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ It makes it easy to implement best practices and patterns for an Operator. Featu
4040
* Handling dependent resources, related events, and caching.
4141
* Automatic Retries
4242
* Smart event scheduling
43-
* Handling Observed Generations automatically
4443
* Easy to use Error Handling
4544
* ... and everything that a batteries included framework needs
4645

docs/documentation/features.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,9 +203,7 @@ To turn off this feature, set `generationAwareEventProcessing` to `false` for th
203203
## Support for Well Known (non-custom) Kubernetes Resources
204204

205205
A Controller can be registered for a non-custom resource, so well known Kubernetes resources like (
206-
`Ingress`, `Deployment`,...). Note that automatic observed generation handling is not supported
207-
for these resources, though, in this case, the handling of the observed generation is probably
208-
handled by the primary controller.
206+
`Ingress`, `Deployment`,...).
209207

210208
See
211209
the [integration test](https://github.com/java-operator-sdk/java-operator-sdk/blob/main/operator-framework/src/test/java/io/javaoperatorsdk/operator/sample/deployment/DeploymentReconciler.java)

docs/documentation/v5-0-migration.md

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,6 @@ permalink: /docs/v5-0-migration
4444
where it is demonstrated. Also, the related part of
4545
a [workaround](https://github.com/operator-framework/java-operator-sdk/blob/main/operator-framework/src/test/java/io/javaoperatorsdk/operator/StatusPatchSSAMigrationIT.java#L110-L116).
4646

47-
Related automatic observed generation handling changes:
48-
Automated Observed Generation (see features in docs), is automatically handled for non-SSA, even if
49-
the status sub-resource is not instructed to be updated. This is not true for SSA, observed generation is updated
50-
only when patch status is instructed by `UpdateControl`.
51-
5247
6. `ManagedDependentResourceContext` has been renamed to `ManagedWorkflowAndDependentResourceContext` and is accessed
5348
via the accordingly renamed `managedWorkflowAndDependentResourceContext` method.
5449
7. `ResourceDiscriminator` was removed. In most of the cases you can just delete the discriminator, everything should

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ default boolean parseResourceVersionsForEventFilteringAndCaching() {
395395
/**
396396
* {@link io.javaoperatorsdk.operator.api.reconciler.UpdateControl} patch resource or status can
397397
* either use simple patches or SSA. Setting this to {@code true}, controllers will use SSA for
398-
* adding finalizers, managing observed generation, patching resources and status.
398+
* adding finalizers, patching resources and status.
399399
*
400400
* @return {@code true} if Server-Side Apply (SSA) should be used when patching the primary
401401
* resources, {@code false} otherwise

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/ReconciliationDispatcher.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -168,9 +168,6 @@ private PostExecutionControl<P> reconcileExecution(ExecutionScope<P> executionSc
168168
}
169169
}
170170

171-
172-
173-
// if using SSA the observed generation is updated only if user instructs patching the status
174171
if (updateControl.isPatchStatus()) {
175172
customResourceFacade.patchStatus(toUpdate, originalResource);
176173
}

0 commit comments

Comments
 (0)