Skip to content

Commit d70f734

Browse files
committed
feat: dependent resource set controller flag on owner reference
Signed-off-by: Attila Mészáros <[email protected]>
1 parent 9057d5f commit d70f734

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -238,4 +238,8 @@ protected boolean creatable() {
238238
protected boolean updatable() {
239239
return updatable;
240240
}
241+
242+
protected Boolean readonly() {
243+
return !isCreatable() && !isUpdatable() && !isDeletable();
244+
}
241245
}

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

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
import java.util.Optional;
66
import java.util.Set;
77

8+
import io.fabric8.kubernetes.api.model.OwnerReference;
9+
import io.fabric8.kubernetes.api.model.OwnerReferenceBuilder;
810
import org.slf4j.Logger;
911
import org.slf4j.LoggerFactory;
1012

@@ -171,7 +173,15 @@ protected Resource<R> prepare(Context<P> context, R desired, P primary, String a
171173
protected void addReferenceHandlingMetadata(R desired, P primary) {
172174
if (addOwnerReference()) {
173175
ReconcilerUtils.checkIfCanAddOwnerReference(primary, desired);
174-
desired.addOwnerReference(primary);
176+
177+
OwnerReferenceBuilder ownerRef = new OwnerReferenceBuilder()
178+
.withApiVersion(primary.getApiVersion())
179+
.withKind(primary.getKind())
180+
.withName(primary.getMetadata().getName())
181+
.withUid(primary.getMetadata().getUid())
182+
.withController(!readonly());
183+
184+
desired.addOwnerReference(ownerRef.build());
175185
} else if (useNonOwnerRefBasedSecondaryToPrimaryMapping()) {
176186
addSecondaryToPrimaryMapperAnnotations(desired, primary);
177187
}

0 commit comments

Comments
 (0)