Skip to content

Commit e661c53

Browse files
committed
[Dependent Resources] Set controller flag to true for owner reference
Signed-off-by: Attila Mészáros <[email protected]>
1 parent a86fc2c commit e661c53

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/kubernetes/KubernetesDependentResource.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,8 @@ protected Resource<R> prepare(Context<P> context, R desired, P primary, String a
205205

206206
protected void addReferenceHandlingMetadata(R desired, P primary) {
207207
if (addOwnerReference()) {
208-
desired.addOwnerReference(primary);
208+
var ref = desired.addOwnerReference(primary);
209+
ref.setController(true);
209210
} else if (useNonOwnerRefBasedSecondaryToPrimaryMapping()) {
210211
addSecondaryToPrimaryMapperAnnotations(desired, primary);
211212
}

operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/generickubernetesresource/GenericKubernetesDependentTestBase.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ void testReconciliation() {
3131
assertThat(cm).isNotNull();
3232
assertThat(cm.getData())
3333
.containsEntry(ConfigMapGenericKubernetesDependent.KEY, INITIAL_DATA);
34+
assertThat(cm.getMetadata().getOwnerReferences().get(0).getController()).isTrue();
3435
});
3536

3637
resource.getSpec().setValue(CHANGED_DATA);

0 commit comments

Comments
 (0)