Question for 5.2 migration

[jonesbusy]

Hi,

I understand that the 5.2 has some breaking changes, but it's not clear in my situation (Even after reading https://javaoperatorsdk.io/docs/migration/v5-2-migration/)

I have multiple Kubernetes Bulk dependent defined as

public class TeamSyncBulkDependent extends KubernetesDependentResource<CloudTeam, CloudTeamSync>
        implements CRUDBulkDependentResource<CloudTeam, CloudTeamSync> {

   // desiredResources and getSecondaryResources implemented here

After 5.2 migration I understand I need to pass a String resource ID

public class TeamSyncBulkDependent extends KubernetesDependentResource<CloudTeam, CloudTeamSync>
        implements CRUDBulkDependentResource<CloudTeam, CloudTeamSync, String> {

But then I'm force to implement the following

    @Override
    public void deleteTargetResource(
            CloudTeamSync primary, CloudTeam resource, String s, Context<CloudTeamSync> context) {
        //
    }

I'm not sure if I'm missing something on the class hierarchy but I don't need to change anything related to delete behavior of dependent

I guess one of the solution is to implement

    @Override
    public void deleteTargetResource(
            CloudTeamSync primary, CloudTeam resource, String s, Context<CloudTeamSync> context) {
        context.getClient().resource(resource).delete();
    }

Thanks for the help!

[csviri]

Hi @jonesbusy

there is a prepared CRUDKubernetesBulkDependentResource

That uses ResourceID (not string) for you.