Skip to content

Commit 17c3be1

Browse files
metacosmcsviri
authored andcommitted
feat: handle clustered resource on secondary to primary mapper init (#2321)
Fixes #2311 Overriding getPrimaryResourceType should allow to make things work even in deeper hierarchies. Signed-off-by: Chris Laprun <[email protected]> Signed-off-by: Attila Mészáros <[email protected]>
1 parent 1ed539b commit 17c3be1

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

operator-framework-core/src/test/java/io/javaoperatorsdk/operator/processing/dependent/workflow/AbstractWorkflowExecutorTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,11 @@ public TestDependent(String name) {
4545
super(ConfigMap.class, name);
4646
}
4747

48+
@Override
49+
protected Class<TestCustomResource> getPrimaryResourceType() {
50+
return TestCustomResource.class;
51+
}
52+
4853
@Override
4954
public ReconcileResult<ConfigMap> reconcile(TestCustomResource primary,
5055
Context<TestCustomResource> context) {

operator-framework/src/test/java/io/javaoperatorsdk/operator/dependent/bulkdependent/ConfigMapDeleterBulkDependentResource.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,11 @@ public ConfigMapDeleterBulkDependentResource() {
2828
super(ConfigMap.class);
2929
}
3030

31+
@Override
32+
protected Class<BulkDependentTestCustomResource> getPrimaryResourceType() {
33+
return BulkDependentTestCustomResource.class;
34+
}
35+
3136
@Override
3237
public Map<String, ConfigMap> desiredResources(BulkDependentTestCustomResource primary,
3338
Context<BulkDependentTestCustomResource> context) {

0 commit comments

Comments
 (0)