Skip to content

Commit e19593b

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

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
@@ -207,7 +207,8 @@ protected Resource<R> prepare(Context<P> context, R desired, P primary, String a
207207
protected void addReferenceHandlingMetadata(R desired, P primary) {
208208
if (addOwnerReference()) {
209209
ReconcilerUtils.checkIfCanAddOwnerReference(primary, desired);
210-
desired.addOwnerReference(primary);
210+
var ref = desired.addOwnerReference(primary);
211+
ref.setController(true);
211212
} else if (useNonOwnerRefBasedSecondaryToPrimaryMapping()) {
212213
addSecondaryToPrimaryMapperAnnotations(desired, primary);
213214
}

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)